mirror of
https://github.com/matthiasmullie/minify.git
synced 2025-02-24 17:03:18 +01:00
I originally executed this on content where string had not yet been replaced. Because this could cause false positives and match content inside strings, I switched it to be executed after strings had been extracted. However, this means those array key texts were also replaced, by something that always looks like a valid variable. I've now moved that check to match the extracted value, which we'll be replacing.
403 lines
10 KiB
PHP
403 lines
10 KiB
PHP
<?php
|
|
|
|
use MatthiasMullie\Minify;
|
|
|
|
/**
|
|
* JS minifier test case.
|
|
*/
|
|
class JSTest extends PHPUnit_Framework_TestCase
|
|
{
|
|
/**
|
|
* @var Minify\JS
|
|
*/
|
|
private $minifier;
|
|
|
|
/**
|
|
* Prepares the environment before running a test.
|
|
*/
|
|
protected function setUp()
|
|
{
|
|
parent::setUp();
|
|
$this->minifier = new Minify\JS();
|
|
}
|
|
|
|
/**
|
|
* Cleans up the environment after running a test.
|
|
*/
|
|
protected function tearDown()
|
|
{
|
|
$this->minifier = null;
|
|
parent::tearDown();
|
|
}
|
|
|
|
/**
|
|
* Test JS minifier rules, provided by dataProvider
|
|
*
|
|
* @test
|
|
* @dataProvider dataProvider
|
|
*/
|
|
public function minify($input, $expected)
|
|
{
|
|
$this->minifier->add($input);
|
|
$result = $this->minifier->minify();
|
|
|
|
$this->assertEquals($expected, $result);
|
|
}
|
|
|
|
/**
|
|
* @return array [input, expected result]
|
|
*/
|
|
public function dataProvider()
|
|
{
|
|
$tests = array();
|
|
|
|
// escaped quotes should not terminate string
|
|
$tests[] = array(
|
|
'alert("Escaped quote which is same as string quotes: \"; should not match")',
|
|
'alert("Escaped quote which is same as string quotes: \"; should not match")',
|
|
);
|
|
|
|
// regex delimiters need to be treated as strings
|
|
// (two forward slashes could look like a comment)
|
|
$tests[] = array(
|
|
'/abc\/def\//.test("abc")',
|
|
'/abc\/def\//.test("abc")',
|
|
);
|
|
$tests[] = array(
|
|
'var a = /abc\/def\//.test("abc")',
|
|
'var a=/abc\/def\//.test("abc")',
|
|
);
|
|
|
|
// don't confuse multiple slashes for regexes
|
|
$tests[] = array(
|
|
'a = b / c; d = e / f',
|
|
'a=b/c;d=e/f',
|
|
);
|
|
|
|
// mixture of quotes starting in comment/regex, to make sure strings are
|
|
// matched correctly, not inside comment/regex.
|
|
$tests[] = array(
|
|
'/abc"def/.test("abc")',
|
|
'/abc"def/.test("abc")',
|
|
);
|
|
$tests[] = array(
|
|
'/* Bogus " */var test="test";',
|
|
'var test="test"',
|
|
);
|
|
|
|
// replace comments
|
|
$tests[] = array(
|
|
'/* This is a JS comment */',
|
|
'',
|
|
);
|
|
|
|
// make sure no ; is added in places it shouldn't
|
|
$tests[] = array(
|
|
'if(true){}else{}',
|
|
'if(true){}else{}',
|
|
);
|
|
$tests[] = array(
|
|
'do{i++}while(i<1)',
|
|
'do{i++}while(i<1)',
|
|
);
|
|
|
|
$tests[] = array(
|
|
'if(true)statement;else statement',
|
|
'if(true)statement;else statement',
|
|
);
|
|
|
|
$tests[] = array(
|
|
'for (i = 0; (i < 10); i++) statement',
|
|
'for(i=0;(i<10);i++)statement',
|
|
);
|
|
$tests[] = array(
|
|
'-1
|
|
+2',
|
|
'-1+2',
|
|
);
|
|
$tests[] = array(
|
|
'-1+
|
|
2',
|
|
'-1+2',
|
|
);
|
|
$tests[] = array(
|
|
'alert("this is a test");',
|
|
'alert("this is a test")',
|
|
);
|
|
|
|
// test where newline should be preserved (for ASI) or semicolon added
|
|
$tests[] = array(
|
|
'function(){console.log("this is a test");}',
|
|
'function(){console.log("this is a test")}',
|
|
);
|
|
$tests[] = array(
|
|
'alert("this is a test")
|
|
alert("this is another test")',
|
|
'alert("this is a test")
|
|
alert("this is another test")',
|
|
);
|
|
$tests[] = array(
|
|
'a=b+c
|
|
d=e+f',
|
|
'a=b+c
|
|
d=e+f',
|
|
);
|
|
$tests[] = array(
|
|
'a++
|
|
|
|
++b',
|
|
'a++
|
|
++b',
|
|
);
|
|
$tests[] = array(
|
|
'!a
|
|
!b',
|
|
'!a
|
|
!b',
|
|
);
|
|
$tests[] = array(
|
|
// don't confuse with 'if'
|
|
'digestif
|
|
(true)
|
|
statement',
|
|
'digestif(true)
|
|
statement',
|
|
);
|
|
$tests[] = array(
|
|
'if
|
|
(
|
|
(
|
|
true
|
|
)
|
|
&&
|
|
(
|
|
true
|
|
)
|
|
)
|
|
statement',
|
|
'if((true)&&(true))
|
|
statement',
|
|
);
|
|
$tests[] = array(
|
|
'if
|
|
(
|
|
true
|
|
)
|
|
{
|
|
}
|
|
else
|
|
{
|
|
}',
|
|
'if(true){}
|
|
else{}',
|
|
);
|
|
$tests[] = array(
|
|
'do
|
|
{
|
|
i++
|
|
}
|
|
while
|
|
(
|
|
i<1
|
|
)',
|
|
'do{i++}
|
|
while(i<1)',
|
|
);
|
|
$tests[] = array(
|
|
'if ( true )
|
|
statement
|
|
else
|
|
statement',
|
|
'if(true)
|
|
statement
|
|
else statement',
|
|
);
|
|
|
|
// test if whitespace around keywords is properly collapsed
|
|
$tests[] = array(
|
|
'var
|
|
variable
|
|
=
|
|
"value";',
|
|
'var variable="value"',
|
|
);
|
|
$tests[] = array(
|
|
'var variable = {
|
|
test:
|
|
{
|
|
}
|
|
}',
|
|
'var variable={test:{}}',
|
|
);
|
|
$tests[] = array(
|
|
'if ( true ) {
|
|
} else {
|
|
}',
|
|
'if(true){}else{}',
|
|
);
|
|
$tests[] = array(
|
|
'53 instanceof String',
|
|
'53 instanceof String'
|
|
);
|
|
|
|
// remove whitespace around operators
|
|
$tests[] = array(
|
|
'a = 1 + 2',
|
|
'a=1+2',
|
|
);
|
|
$tests[] = array(
|
|
'object . property',
|
|
'object.property',
|
|
);
|
|
$tests[] = array(
|
|
'object
|
|
.property',
|
|
'object.property',
|
|
);
|
|
$tests[] = array(
|
|
'alert ( "this is a test" );',
|
|
'alert("this is a test")',
|
|
);
|
|
|
|
// mix of ++ and +: three consecutive +es will be interpreted as ++ +
|
|
$tests[] = array(
|
|
'a++ +b',
|
|
'a++ +b',
|
|
);
|
|
$tests[] = array(
|
|
'a+ ++b',
|
|
'a+ ++b', // +++ would actually be allowed as well
|
|
);
|
|
|
|
// SyntaxError: identifier starts immediately after numeric literal
|
|
$tests[] = array(
|
|
'42 .toString()',
|
|
'42 .toString()',
|
|
);
|
|
|
|
// add comment in between whitespace that needs to be stripped
|
|
$tests[] = array(
|
|
'object
|
|
// haha, some comment, just to make things harder!
|
|
.property',
|
|
'object.property',
|
|
);
|
|
|
|
// random bits of code that tripped errors during development
|
|
$tests[] = array(
|
|
'
|
|
// check if it isn\'t a text-element
|
|
if(currentElement.attr(\'type\') != \'text\')
|
|
{
|
|
// remove the current one
|
|
currentElement.remove();
|
|
}
|
|
|
|
// already a text element
|
|
else newElement = currentElement;
|
|
',
|
|
'if(currentElement.attr(\'type\')!=\'text\'){currentElement.remove()}
|
|
else newElement=currentElement',
|
|
);
|
|
$tests[] = array(
|
|
'var jsBackend =
|
|
{
|
|
debug: false,
|
|
current: {}
|
|
}',
|
|
'var jsBackend={debug:false,current:{}}',
|
|
);
|
|
$tests[] = array(
|
|
'var utils =
|
|
{
|
|
debug: false
|
|
}
|
|
utils.array =
|
|
{
|
|
}',
|
|
'var utils={debug:false}
|
|
utils.array={}',
|
|
);
|
|
$tests[] = array(
|
|
'rescape = /\'|\\\\/g,
|
|
|
|
// blablabla here was some more code but the point was that somewhere
|
|
// down below, there would be a closing quote which would cause the
|
|
// regex (confused for escaped closing tag) not to be recognized,
|
|
// taking the opening single quote & looking for a string.
|
|
// So here\'s <-- the closing quote
|
|
runescape = \'blabla\'',
|
|
'rescape=/\'|\\\\/g,runescape=\'blabla\'',
|
|
);
|
|
$tests[] = array(
|
|
'var rsingleTag = (/^<(\w+)\s*\/?>(?:<\/\1>|)$/)',
|
|
'var rsingleTag=(/^<(\w+)\s*\/?>(?:<\/\1>|)$/)',
|
|
);
|
|
|
|
// replace associative array key references by property notation
|
|
$tests[] = array(
|
|
'array["key"][\'key2\']',
|
|
'array.key.key2',
|
|
);
|
|
$tests[] = array(
|
|
'array[ "key" ][ \'key2\' ]',
|
|
'array.key.key2',
|
|
);
|
|
|
|
// https://github.com/matthiasmullie/minify/issues/10
|
|
$tests[] = array(
|
|
'// first mutation patch
|
|
// second mutation patch
|
|
// third mutation patch
|
|
// fourth mutation patch',
|
|
'',
|
|
);
|
|
$tests[] = array(
|
|
'/////////////////////////
|
|
// first mutation patch
|
|
// second mutation patch
|
|
// third mutation patch
|
|
// fourth mutation patch
|
|
/////////////////////////',
|
|
'',
|
|
);
|
|
|
|
// https://github.com/matthiasmullie/minify/issues/14
|
|
$tests[] = array(
|
|
'function foo (a, b)
|
|
{
|
|
return a / b;
|
|
}
|
|
function foo (a, b)
|
|
{
|
|
return a / b;
|
|
}',
|
|
'function foo(a,b){return a/b}
|
|
function foo(a,b){return a/b}',
|
|
);
|
|
|
|
// https://github.com/matthiasmullie/minify/issues/15
|
|
$tests[] = array(
|
|
'if ( !data.success )
|
|
deferred.reject(); else
|
|
deferred.resolve(data);',
|
|
'if(!data.success)
|
|
deferred.reject();else deferred.resolve(data)',
|
|
);
|
|
$tests[] = array(
|
|
"if ( typeof jQuery === 'undefined' )
|
|
throw new Error('.editManager.js: jQuery is required and must be loaded first');",
|
|
"if(typeof jQuery==='undefined')
|
|
throw new Error('.editManager.js: jQuery is required and must be loaded first')",
|
|
);
|
|
|
|
// https://github.com/matthiasmullie/minify/issues/27
|
|
$tests[] = array(
|
|
'$.expr[":"]',
|
|
'$.expr[":"]',
|
|
);
|
|
|
|
return $tests;
|
|
}
|
|
}
|