REGRESSION: code-review-tests.html fails with error "Received an error at line 214"

https://bugs.webkit.org/show_bug.cgi?id=152103

Reviewed by Darin Adler.

Fixes an issue where opening the file code-review-test.html in Safari shows the error message
"FAIL: Received an error at line 214" and in the console there is a JavaScript TypeError:
    undefined is not an object (evaluating '$('.overallComments textarea').val().trim')

Notice that test testReaddDiscardedCommentWithPreviousComment() is the only test that appends
the toolbar, which inserts the overall comments textarea. The test testSaveCommentsWithMissingLineIds()
assumes that the own properties of the window object are enumerated in the same order as they
were defined in the file code-review-test.html such that the test testReaddDiscardedCommentWithPreviousComment()
is executed before test testSaveCommentsWithMissingLineIds(). The behavior of JavaScriptCore with
respect to the enumeration order of properties on the window object have changed since the
test testSaveCommentsWithMissingLineIds() was written such that the order of the own properties
on the window object do not reflect the order in which they were defined in the file. We should
append the toolbar before running the tests and explicitly execute the tests in chosen order so
as to be able to reason of the test results regardless of the enumeration ordering of the
properties in the DOM window object.

Also, removed duplicate 'd' in the name of function "testReaddDiscardedCommentWithPreviousComment"
such that it reads "testReadDiscardedCommentWithPreviousComment".

* code-review-test.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@193889 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/Websites/bugs.webkit.org/ChangeLog b/Websites/bugs.webkit.org/ChangeLog
index 45afaad36960c87a19e61412750bc8847b511a6c..f67c1594c7077630e799a0aed2dd72b4487cc6e7 100644 (file)
--- a/Websites/bugs.webkit.org/ChangeLog
+++ b/Websites/bugs.webkit.org/ChangeLog
@@ -1,3 +1,31 @@
+2015-12-09  Daniel Bates  <dabates@apple.com>
+
+        REGRESSION: code-review-tests.html fails with error "Received an error at line 214"
+        https://bugs.webkit.org/show_bug.cgi?id=152103
+
+        Reviewed by Darin Adler.
+
+        Fixes an issue where opening the file code-review-test.html in Safari shows the error message
+        "FAIL: Received an error at line 214" and in the console there is a JavaScript TypeError:
+            undefined is not an object (evaluating '$('.overallComments textarea').val().trim')
+
+        Notice that test testReaddDiscardedCommentWithPreviousComment() is the only test that appends
+        the toolbar, which inserts the overall comments textarea. The test testSaveCommentsWithMissingLineIds()
+        assumes that the own properties of the window object are enumerated in the same order as they
+        were defined in the file code-review-test.html such that the test testReaddDiscardedCommentWithPreviousComment()
+        is executed before test testSaveCommentsWithMissingLineIds(). The behavior of JavaScriptCore with
+        respect to the enumeration order of properties on the window object have changed since the
+        test testSaveCommentsWithMissingLineIds() was written such that the order of the own properties
+        on the window object do not reflect the order in which they were defined in the file. We should
+        append the toolbar before running the tests and explicitly execute the tests in chosen order so
+        as to be able to reason of the test results regardless of the enumeration ordering of the
+        properties in the DOM window object.
+
+        Also, removed duplicate 'd' in the name of function "testReaddDiscardedCommentWithPreviousComment"
+        such that it reads "testReadDiscardedCommentWithPreviousComment".
+
+        * code-review-test.html:
+
 2015-11-17  Lucas Forschler  <lforschler@apple.com>
 
         bugs.webkit.org is running with changes that are not checked in.

diff --git a/Websites/bugs.webkit.org/code-review-test.html b/Websites/bugs.webkit.org/code-review-test.html
index d2469ba60a7d1323ab685d56c5bf70337f376a45..b59fa4c43d3f3895983aa49fcabea1e59b673e01 100644 (file)
--- a/Websites/bugs.webkit.org/code-review-test.html
+++ b/Websites/bugs.webkit.org/code-review-test.html
@@ -218,7 +218,7 @@ function syncSlideUp(type, handler) {
   handler.call(this);
 }
 
-function testReaddDiscardedCommentWithPreviousComment() {
+function testReadDiscardedCommentWithPreviousComment() {
   document.getElementById('diff-content').innerHTML =
       '<div class="FileDiff">' +
         '<h1><a href="http://trac.webkit.org/browser/trunk/Source/WebCore/ChangeLog">Source/WebCore/ChangeLog</a></h1>' +
@@ -247,7 +247,6 @@ function testReaddDiscardedCommentWithPreviousComment() {
 
   eraseDraftComments();
   crawlDiff();
-  appendToolbar();
 
   var line = document.getElementById('line0');
   var author = "ojan@chromium.org";
@@ -456,9 +455,17 @@ function testSaveCommentsWithMissingLineIds() {
 }
 
 
-for (var property in window) {
-  if (property.indexOf('test') == 0) {
-    window[property]();
-  }
-}
+var tests_to_run = [
+    window.testTracLinks,
+    window.testDraftCommentSaver,
+    window.testReadDiscardedCommentWithPreviousComment,
+    window.testSideBySideDiffWithPreviousCommentsOnSharedLine,
+    window.testSanitizeFragmentForCopy,
+    window.testIsChangeLog,
+    window.testSaveCommentsWithMissingLineIds,
+];
+
+appendToolbar();
+for (var i = 0; i < tests_to_run.length; ++i)
+    tests_to_run[i]();
 </script>