aboutsummaryrefslogtreecommitdiffstats
path: root/tests/phpunit/includes/libs/JavaScriptMinifierTest.php
Commit message (Collapse)AuthorAgeFilesLines
* Followup r103865 and r103915Brion Vibber2011-11-221-4/+7
| | | | | | | | | | | | | | r103915 added a parse error for 'more than 2 decimal points' in a number; this is the wrong place to check for that. Should only check whether there's more digits or identifier chars -- identifier chars would be illegal. Added test cases for the exponent missing fails, tweaked it to be more consistent (only need to check for one e; if we have more we can lump them in with 'not digits' :) Also cleaned up no-longer-needed suppress/restore warnings around JS parser invocation Notes: http://mediawiki.org/wiki/Special:Code/MediaWiki/103978
* Final fixup to r103910 and follow-ups.Platonides2011-11-221-1/+1
| | | | | | | Reverts r103931 test change. Notes: http://mediawiki.org/wiki/Special:Code/MediaWiki/103940
* Two decimal points may be valid, as 5..toString() == (5.).toString()Platonides2011-11-221-0/+7
| | | | | | | Added some tests Notes: http://mediawiki.org/wiki/Special:Code/MediaWiki/103937
* Follow-up r103915: We need to increment $end before the strcspn.Platonides2011-11-221-1/+1
| | | | | | | Adjusting the tests, since it is now adding the newline after the number. Notes: http://mediawiki.org/wiki/Special:Code/MediaWiki/103931
* Add PHPUnit tests for the minification failure case in bug 32548.Brion Vibber2011-11-211-0/+36
| | | | | | | | | This will trigger 2 test failures, where an exponent in a JS numeric literal gets split over line breaks at the '-' or '+', causing a parse error in the resulting output. A number with the same string length but without using + or - in the exponent passes through fine, indicating that it's the -/+ that's getting misinterpreted. Notes: http://mediawiki.org/wiki/Special:Code/MediaWiki/103846
* * (bug 31187) Fix for user JavaScript validation to allow identifiers with ↵Brion Vibber2011-09-271-0/+6
| | | | | | | | | | | | | | | | | | | | valid Unicode letter characters Followup r91591, r93020: patch to jsminplus to support Unicode chars and char escapes in identifiers Fast-path check keeps runtime about the same on most scripts (eg jquery.js parsing was abround 4100ms both before and after on my test machine) Slow-path code kicks in if plain ASCII word chars don't extend all the way to the next whitespace or punctuation char. Using PCRE's Unicode properties magic to ensure that we're catching everything, following ECMA-262 edition 5.1 spec. Note that identifiers using escapes don't get normalized to their UTF-8 form; this might be a nice thing to do as it saves a couple bytes, but currently there's no change made to output. Added QUnit tests to verify that unicode letter & escapes work in identifiers in all supported browsers (ok back to IE 6, yay) Notes: http://mediawiki.org/wiki/Special:Code/MediaWiki/98281
* Followup r83885: add JSMin+ 1.3 to use its parser to verify output of ↵Brion Vibber2011-07-061-8/+21
| | | | | | | | | | | JavaScriptMinifierTest unit test cases. Had to change some of the test cases because they were not valid JavaScript programs -- one of the quoting tests was incorrectly quoted in the PHP sources, and several tests around return, continue, and break keywords failed due to not using them in the contexts that they require. http://crisp.tweakblogs.net/blog/1856/jsmin+-version-13.html JSMin+ under MPL 1.1/ GPL 2.0 / LGPL 2.1 license. Notes: http://mediawiki.org/wiki/Special:Code/MediaWiki/91591
* Followup r83891: don't insert a newline before ++ or -- . Patch by Paul ↵Roan Kattouw2011-03-141-0/+3
| | | | | | | Copperman Notes: http://mediawiki.org/wiki/Special:Code/MediaWiki/83934
* Per Nikerabbit, add some minifier tests for historic failure cases that I ↵Roan Kattouw2011-03-141-1/+14
| | | | | | | had lying around Notes: http://mediawiki.org/wiki/Special:Code/MediaWiki/83894
* (bug 27528) Incorporate Paul Copperman's minifierRoan Kattouw2011-03-141-0/+70
Notes: http://mediawiki.org/wiki/Special:Code/MediaWiki/83885