Merge pull request #1 from Crecket/develop

Added

Unit tests
Removed files from url, this means all file names are completely hidden
Auto prefixer for css values
Better synthax and logic
Improved performance

.gitignore

@@ -3,4 +3,6 @@
 /index.php
 /minify.php
 /cache
-/phpunit.phar
+/phpunit.phar
+/phpunit-4.5.0.phar
+/tests/cache

.travis.yml

@@ -1,10 +1,17 @@
 language: php
+
 php:
-    - "5.6"
-    - "7.0"
+  - 5.6
+  - 7.0
+
 script: phpunit tests/test.php
+
 install:
-    - composer self-update
     - composer update -o
     - npm install -g bower
     - bower install
+    - phpunit --version
+
+matrix:
+    allow_failures:
+        - php: 5.6

composer.json

@@ -16,11 +16,10 @@
     "tedivm/jshrink": "^1.1",
     "twig/twig": "^1.23",
     "oyejorge/less.php": "v1.7.0.10",
-    "css-crush/css-crush": "*",
-    "phpunit/phpunit": "^5.3"
+    "css-crush/css-crush": "*"
   },
   "require-dev": {
-    "phpunit/phpunit": "^5.3"
+    "phpunit/phpunit": "*"
   },
   "autoload": {
     "psr-4": {

src/Response.php

@@ -114,16 +114,6 @@ public function __construct($options, $input_file_list = false)
         // Create cache element
         $this->cache = new FilesystemCache($this->options['Cache']);
 
-        // Required option
-        if (empty($this->options['Remote_storage'])) {
-            throw new Exception('Remote file error', 'Missing remote file storage location');
-        } else if (!file_exists($this->options['Remote_storage'])) {
-            throw new Exception('Remote file error', 'Storage location not found');
-        }
-
-        // set remote file storage location
-        $this->remote_storage = $this->options['Remote_storage'];
-
         // Required option
         if (!empty($this->options['CacheNameSpace'])) {
             $this->cache->setNamespace('crecket_dependency_loader');

tests/test.php

@@ -1,8 +1,6 @@
 <?php
 require __DIR__ . '/../vendor/autoload.php';
 
-// TODO bower install: https://github.com/sebastianbergmann/phpunit-website
-
 class testMain extends PHPUnit_Framework_TestCase
 {
 
@@ -27,19 +25,19 @@ public function testSecret()
 
     public function testGetLinks()
     {
-        $this->assertEquals(\Crecket\DependencyManager\Loader::getCssLink(true), '?secret=ca48b5ff1fc8d475b138b336e49e0d2adc3bb11ef5546ac855045be2a17957a5&minify=true');
-        $this->assertEquals(\Crecket\DependencyManager\Loader::getCssLink(false), '?secret=ca48b5ff1fc8d475b138b336e49e0d2adc3bb11ef5546ac855045be2a17957a5');
-        $this->assertEquals(\Crecket\DependencyManager\Loader::getJsLink(true), '?secret=e195b9eaeaa09cab85600a86bfb7e88b34903088146706600191355b4191af0a&minify=true');
-        $this->assertEquals(\Crecket\DependencyManager\Loader::getJsLink(false), '?secret=e195b9eaeaa09cab85600a86bfb7e88b34903088146706600191355b4191af0a');
+        // TODO create better test case for link generation
+        $this->assertNotEmpty(\Crecket\DependencyManager\Loader::getCssLink(true));
+        $this->assertNotEmpty(\Crecket\DependencyManager\Loader::getCssLink(false));
+        $this->assertNotEmpty(\Crecket\DependencyManager\Loader::getJsLink(true));
+        $this->assertNotEmpty(\Crecket\DependencyManager\Loader::getJsLink(false));
     }
 
     public function testResponse()
     {
         define('ROOT', __DIR__);
 
         $options = array(
-            'Cache' => ROOT.'/cache',
-            'Remote_storage' => ROOT.'/bower_components/'
+            'Cache' => ROOT.'/cache'
         );
 
         $jsList = array(
@@ -61,7 +59,4 @@ public function testResponse()
         $CssResponse = new \Crecket\DependencyManager\Response($options, $cssList);
         $this->assertNotEmpty($CssResponse->getResult());
     }
-
-    // TODO Session testing
-
 }