Title: [92396] trunk/Tools
Revision
92396
Author
[email protected]
Date
2011-08-04 12:19:23 -0700 (Thu, 04 Aug 2011)

Log Message

checkout_unittest.js should have better coverage of checkout.js
https://bugs.webkit.org/show_bug.cgi?id=65671

Reviewed by Dimitri Glazkov.

* BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/checkout.js:
    - Remove unused functions.
* BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/checkout_unittests.js:
    - Test remaining public functions in the checkout module.

Modified Paths

Diff

Modified: trunk/Tools/BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/checkout.js (92395 => 92396)


--- trunk/Tools/BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/checkout.js	2011-08-04 19:11:13 UTC (rev 92395)
+++ trunk/Tools/BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/checkout.js	2011-08-04 19:19:23 UTC (rev 92396)
@@ -29,30 +29,11 @@
 
 var kWebKitTrunk = 'http://svn.webkit.org/repository/webkit/trunk/';
 
-function subversionURLAtRevision(subversionURL, revision)
-{
-    return subversionURL + '?r=' + revision;
-}
-
 checkout.subversionURLForTest = function(testName)
 {
     return kWebKitTrunk + 'LayoutTests/' + testName;
 }
 
-checkout.existsAtRevision = function(subversionURL, revision, callback)
-{
-    net.ajax({
-        method: 'HEAD',
-        url: subversionURLAtRevision(subversionURL, revision), 
-        success: function() {
-            callback(true);
-        },
-        error: function() {
-            callback(false);
-        }
-    });
-};
-
 checkout.updateExpectations = function(failureInfoList, callback)
 {
     net.post('/updateexpectations', JSON.stringify(failureInfoList), function() {

Modified: trunk/Tools/BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/checkout_unittests.js (92395 => 92396)


--- trunk/Tools/BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/checkout_unittests.js	2011-08-04 19:11:13 UTC (rev 92395)
+++ trunk/Tools/BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/checkout_unittests.js	2011-08-04 19:19:23 UTC (rev 92396)
@@ -31,4 +31,77 @@
     equals(checkout.subversionURLForTest("path/to/test.html"), "http://svn.webkit.org/repository/webkit/trunk/LayoutTests/path/to/test.html");
 });
 
+test("updateExpectations", 4, function() {
+    var simulator = new NetworkSimulator();
+    simulator.post = function(url, data, callback)
+    {
+        equals(url, '/updateexpectations');
+        equals(data, '[{"builderName":"WebKit Linux","testName":"another/test.svg","failureTypeList":["IMAGE"]}]');
+        simulator.scheduleCallback(callback);
+    };
+
+    simulator.runTest(function() {
+        checkout.updateExpectations([{
+            'builderName': 'WebKit Linux',
+            'testName': 'another/test.svg',
+            'failureTypeList': ['IMAGE'],
+        }], function() {
+            ok(true);
+        });
+    });
+});
+
+test("optimizeBaselines", 3, function() {
+    var simulator = new NetworkSimulator();
+    simulator.post = function(url, callback)
+    {
+        equals(url, '/optimizebaselines?test=another%2Ftest.svg');
+        simulator.scheduleCallback(callback);
+    };
+
+    simulator.runTest(function() {
+        checkout.optimizeBaselines('another/test.svg', function() {
+            ok(true);
+        });
+    });
+});
+
+test("rebaseline", 3, function() {
+    var simulator = new NetworkSimulator();
+
+    var requestedURLs = [];
+    simulator.post = function(url, callback)
+    {
+        requestedURLs.push(url);
+        simulator.scheduleCallback(callback);
+    };
+
+    simulator.runTest(function() {
+        checkout.rebaseline([{
+            'builderName': 'WebKit Linux',
+            'testName': 'another/test.svg',
+            'failureTypeList': ['IMAGE'],
+        }, {
+            'builderName': 'WebKit Mac10.6',
+            'testName': 'another/test.svg',
+            'failureTypeList': ['IMAGE+TEXT'],
+        }, {
+            'builderName': 'Webkit Vista',
+            'testName': 'fast/test.html',
+            'failureTypeList': ['TEXT'],
+        }], function() {
+            ok(true);
+        });
+    });
+
+    deepEqual(requestedURLs, [
+        "/rebaseline?builder=WebKit+Linux&test=another%2Ftest.svg&extension=png",
+        "/rebaseline?builder=WebKit+Mac10.6&test=another%2Ftest.svg&extension=txt",
+        "/rebaseline?builder=WebKit+Mac10.6&test=another%2Ftest.svg&extension=png",
+        "/rebaseline?builder=Webkit+Vista&test=fast%2Ftest.html&extension=txt",
+        "/optimizebaselines?test=another%2Ftest.svg",
+        "/optimizebaselines?test=fast%2Ftest.html"
+    ]);
+});
+
 })();

Modified: trunk/Tools/ChangeLog (92395 => 92396)


--- trunk/Tools/ChangeLog	2011-08-04 19:11:13 UTC (rev 92395)
+++ trunk/Tools/ChangeLog	2011-08-04 19:19:23 UTC (rev 92396)
@@ -1,3 +1,15 @@
+2011-08-04  Adam Barth  <[email protected]>
+
+        checkout_unittest.js should have better coverage of checkout.js
+        https://bugs.webkit.org/show_bug.cgi?id=65671
+
+        Reviewed by Dimitri Glazkov.
+
+        * BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/checkout.js:
+            - Remove unused functions.
+        * BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/checkout_unittests.js:
+            - Test remaining public functions in the checkout module.
+
 2011-08-04  Dmitry Lomov  <[email protected]>
 
         https://bugs.webkit.org/show_bug.cgi?id=65706
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to