From ef591e7ee21435da9314c5f7f6ea983c6f423898 Mon Sep 17 00:00:00 2001
From: Guillaume Virlet <github@virlet.org>
Date: Wed, 2 Sep 2015 13:55:39 +0200
Subject: Url: introduce global helper functions for cleanup and scheme
 detection

Relates to #314 & #326

Additions:
 - add global `cleanup_url()` and `get_url_scheme()` functions

Modifications:
 - replace `Url` usage in `index.php` by calls to global functions
 - fix `Url` tests not being run: PHPUnit expects a single test class per file
   - move classes to separate files
---
 tests/UrlTest.php | 170 ------------------------------------------------------
 1 file changed, 170 deletions(-)
 delete mode 100755 tests/UrlTest.php

(limited to 'tests/UrlTest.php')

diff --git a/tests/UrlTest.php b/tests/UrlTest.php
deleted file mode 100755
index c848e88e..00000000
--- a/tests/UrlTest.php
+++ /dev/null
@@ -1,170 +0,0 @@
-<?php
-/**
- * Url's tests
- */
-
-require_once 'application/Url.php';
-
-/**
- * Unitary tests for unparse_url()
- */
-class UnparseUrlTest extends PHPUnit_Framework_TestCase
-{
-    /**
-     * Thanks for building nothing
-     */
-    public function testUnparseEmptyArray()
-    {
-        $this->assertEquals('', unparse_url(array()));
-    }
-
-    /**
-     * Rebuild a full-featured URL
-     */
-    public function testUnparseFull()
-    {
-        $ref = 'http://username:password@hostname:9090/path'
-              .'?arg1=value1&arg2=value2#anchor';
-        $this->assertEquals($ref, unparse_url(parse_url($ref)));
-    }
-}
-
-/**
- * Unitary tests for URL utilities
- */
-class UrlTest extends PHPUnit_Framework_TestCase
-{
-    // base URL for tests
-    protected static $baseUrl = 'http://domain.tld:3000';
-
-    /**
-     * Helper method
-     */
-    private function assertUrlIsCleaned($query='', $fragment='')
-    {
-        $url = new Url(self::$baseUrl.$query.$fragment);
-        $url->cleanup();
-        $this->assertEquals(self::$baseUrl, $url->__toString());
-    }
-
-    /**
-     * Instantiate an empty URL
-     */
-    public function testEmptyConstruct()
-    {
-        $this->assertEquals('', new Url(''));
-    }
-
-    /**
-     * Instantiate a URL
-     */
-    public function testConstruct()
-    {
-        $ref = 'http://username:password@hostname:9090/path'
-              .'?arg1=value1&arg2=value2#anchor';
-        $this->assertEquals($ref, new Url($ref));
-    }
-
-    /**
-     * URL cleanup - nothing to do
-     */
-    public function testNoCleanup()
-    {
-        // URL with no query nor fragment
-        $this->assertUrlIsCleaned();
-
-        // URL with no annoying elements
-        $ref = self::$baseUrl.'?p1=val1&p2=1234#edit';
-        $url = new Url($ref);
-        $this->assertEquals($ref, $url->cleanup());
-    }
-
-    /**
-     * URL cleanup - annoying fragment
-     */
-    public function testCleanupFragment()
-    {
-        $this->assertUrlIsCleaned('', '#tk.rss_all');
-        $this->assertUrlIsCleaned('', '#xtor=RSS-');
-        $this->assertUrlIsCleaned('', '#xtor=RSS-U3ht0tkc4b');
-    }
-
-    /**
-     * URL cleanup - single annoying query parameter
-     */
-    public function testCleanupSingleQueryParam()
-    {
-        $this->assertUrlIsCleaned('?action_object_map=junk');
-        $this->assertUrlIsCleaned('?action_ref_map=Cr4p!');
-        $this->assertUrlIsCleaned('?action_type_map=g4R84g3');
-
-        $this->assertUrlIsCleaned('?fb_stuff=v41u3');
-        $this->assertUrlIsCleaned('?fb=71m3w4573');
-
-        $this->assertUrlIsCleaned('?utm_campaign=zomg');
-        $this->assertUrlIsCleaned('?utm_medium=numnum');
-        $this->assertUrlIsCleaned('?utm_source=c0d3');
-        $this->assertUrlIsCleaned('?utm_term=1n4l');
-
-        $this->assertUrlIsCleaned('?xtor=some-url');
-    }
-
-    /**
-     * URL cleanup - multiple annoying query parameters
-     */
-    public function testCleanupMultipleQueryParams()
-    {
-        $this->assertUrlIsCleaned('?xtor=some-url&fb=som3th1ng');
-        $this->assertUrlIsCleaned(
-            '?fb=stuff&utm_campaign=zomg&utm_medium=numnum&utm_source=c0d3'
-        );
-    }
-
-    /**
-     * URL cleanup - multiple annoying query parameters, annoying fragment
-     */
-    public function testCleanupMultipleQueryParamsAndFragment()
-    {
-        $this->assertUrlIsCleaned('?xtor=some-url&fb=som3th1ng', '#tk.rss_all');
-    }
-
-    /**
-     * Nominal case - the URL contains both useful and annoying parameters
-     */
-    public function testCleanupMixedContent()
-    {
-        // ditch annoying query params and fragment, keep useful params
-        $url = new Url(
-            self::$baseUrl
-            .'?fb=zomg&my=stuff&utm_medium=numnum&is=kept#tk.rss_all'
-        );
-        $this->assertEquals(self::$baseUrl.'?my=stuff&is=kept', $url->cleanup());
-
-
-        // ditch annoying query params, keep useful params and fragment
-        $url = new Url(
-            self::$baseUrl
-            .'?fb=zomg&my=stuff&utm_medium=numnum&is=kept#again'
-        );
-        $this->assertEquals(
-            self::$baseUrl.'?my=stuff&is=kept#again',
-            $url->cleanup()
-        );
-    }
-
-    /**
-     * Test default http scheme.
-     */
-    public function testDefaultScheme() {
-        $url = new Url(self::$baseUrl);
-        $this->assertEquals('http', $url->getScheme());
-        $url = new Url('domain.tld');
-        $this->assertEquals('http', $url->getScheme());
-        $url = new Url('ssh://domain.tld');
-        $this->assertEquals('ssh', $url->getScheme());
-        $url = new Url('ftp://domain.tld');
-        $this->assertEquals('ftp', $url->getScheme());
-        $url = new Url('git://domain.tld/push?pull=clone#checkout');
-        $this->assertEquals('git', $url->getScheme());
-    }
-}
-- 
cgit v1.2.3