This build is red because of bindings-generation-tests, but dashboard says that it's green: <http://build.webkit.org/builders/Apple%20MountainLion%20Debug%20WK1%20%28Tests%29/builds/10527>. Long term direction is that we should unify most types of tests and have all issues reported by one script, but we are nowhere near that.
Created attachment 213204 [details] Patch
Comment on attachment 213204 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=213204&action=review > Tools/BuildSlaveSupport/build.webkit.org-config/public_html/dashboard/Scripts/BuildbotTesterQueueView.js:70 > + var bindingTestResults = iteration.bindingTestResults || {failureCount: 0}; Hmm, shouldn't this be {errorOccurred: false}?
Comment on attachment 213204 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=213204&action=review >> Tools/BuildSlaveSupport/build.webkit.org-config/public_html/dashboard/Scripts/BuildbotTesterQueueView.js:70 >> + var bindingTestResults = iteration.bindingTestResults || {failureCount: 0}; > > Hmm, shouldn't this be {errorOccurred: false}? No, this makes sure failureCount is 0 when adding totalFailures. Otherwise undefined causes a NaN total.
Comment on attachment 213204 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=213204&action=review >>> Tools/BuildSlaveSupport/build.webkit.org-config/public_html/dashboard/Scripts/BuildbotTesterQueueView.js:70 >>> + var bindingTestResults = iteration.bindingTestResults || {failureCount: 0}; >> >> Hmm, shouldn't this be {errorOccurred: false}? > > No, this makes sure failureCount is 0 when adding totalFailures. Otherwise undefined causes a NaN total. totalFailures uses errorOccurred: 88 var totalFailures = layoutTestResults.failureCount + javascriptTestResults.failureCount + pythonTestResults.failureCount + perlTestResults.failureCount + bindingTestResults.errorOccurred;
Comment on attachment 213204 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=213204&action=review >>>> Tools/BuildSlaveSupport/build.webkit.org-config/public_html/dashboard/Scripts/BuildbotTesterQueueView.js:70 >>>> + var bindingTestResults = iteration.bindingTestResults || {failureCount: 0}; >>> >>> Hmm, shouldn't this be {errorOccurred: false}? >> >> No, this makes sure failureCount is 0 when adding totalFailures. Otherwise undefined causes a NaN total. > > totalFailures uses errorOccurred: > > 88 var totalFailures = layoutTestResults.failureCount + javascriptTestResults.failureCount + pythonTestResults.failureCount + perlTestResults.failureCount + bindingTestResults.errorOccurred; Ah, yep! Fill fix. Good catch.
http://trac.webkit.org/changeset/157052