<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>171044</bug_id>
          
          <creation_ts>2017-04-19 23:36:12 -0700</creation_ts>
          <short_desc>Test262 bot does not go red with failures</short_desc>
          <delta_ts>2017-04-20 18:50:18 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>Tools / Tests</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Joseph Pecoraro">joepeck</reporter>
          <assigned_to name="Joseph Pecoraro">joepeck</assigned_to>
          <cc>aakash_jain</cc>
    
    <cc>ap</cc>
    
    <cc>buildbot</cc>
    
    <cc>joepeck</cc>
    
    <cc>lforschler</cc>
    
    <cc>saam</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1299040</commentid>
    <comment_count>0</comment_count>
    <who name="Joseph Pecoraro">joepeck</who>
    <bug_when>2017-04-19 23:36:12 -0700</bug_when>
    <thetext>Summary:
Test262 bot does not go red with failures

Case:
https://build.webkit.org/builders/Apple%20El%20Capitan%20Release%20Test262%20%28Tests%29/builds/265

Output:
...
Running test262.yaml/test262/test/annexB/language/eval-code/indirect/global-block-decl-eval-global-exsting-fn-update.js.default
Running test262.yaml/test262/test/annexB/language/eval-code/indirect/global-block-decl-eval-global-exsting-global-init.js.default
test262.yaml/test262/test/annexB/language/eval-code/direct/var-env-lower-lex-catch-non-strict.js.default: Exception: SyntaxError: Can&apos;t create duplicate variable in eval: &apos;err&apos;
test262.yaml/test262/test/annexB/language/eval-code/direct/var-env-lower-lex-catch-non-strict.js.default: at var-env-lower-lex-catch-non-strict.js:28
test262.yaml/test262/test/annexB/language/eval-code/direct/var-env-lower-lex-catch-non-strict.js.default: eval@[native code]
test262.yaml/test262/test/annexB/language/eval-code/direct/var-env-lower-lex-catch-non-strict.js.default: global code@var-env-lower-lex-catch-non-strict.js:28:7
Running test262.yaml/test262/test/annexB/language/eval-code/indirect/global-block-decl-eval-global-exsting-global-update.js.default
Running test262.yaml/test262/test/annexB/language/eval-code/indirect/global-block-decl-eval-global-exsting-var-no-init.js.default
Running test262.yaml/test262/test/annexB/language/eval-code/indirect/global-block-decl-eval-global-exsting-var-update.js.default
test262.yaml/test262/test/annexB/language/eval-code/direct/var-env-lower-lex-catch-non-strict.js.default: ERROR: Unexpected exit code: 3
Running test262.yaml/test262/test/annexB/language/eval-code/indirect/global-block-decl-eval-global-init.js.default
FAIL: test262.yaml/test262/test/annexB/language/eval-code/direct/var-env-lower-lex-catch-non-strict.js.default
Running test262.yaml/test262/test/annexB/language/eval-code/indirect/global-block-decl-eval-global-no-skip-try.js.default
...

Notes:
• It appears that when run-jsc-stress-tests output is piped it produces different output:
  - Running: &lt;test&gt;
  - Running: &lt;test&gt;
  - FAIL: &lt;test&gt;

So we should change the Test bot to check for this output instead of the output when it is not piped!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1299041</commentid>
    <comment_count>1</comment_count>
    <who name="Joseph Pecoraro">joepeck</who>
    <bug_when>2017-04-19 23:43:22 -0700</bug_when>
    <thetext>Also note that from where run-jsc-stress-tests is run, there is a results directory which produces a &quot;failures&quot; file with all of the tests that failed. Presumably that could be used to check for failures instead of parsing the entire content (loading into memory is probably a lot of memory because it contains a string for each of the thousands of tests).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1299047</commentid>
    <comment_count>2</comment_count>
      <attachid>307566</attachid>
    <who name="Joseph Pecoraro">joepeck</who>
    <bug_when>2017-04-20 00:00:14 -0700</bug_when>
    <thetext>Created attachment 307566
[PATCH] Proposed Fix

Again, I don&apos;t have the means to test this on bots, but I&apos;m hopeful!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1299048</commentid>
    <comment_count>3</comment_count>
    <who name="Build Bot">buildbot</who>
    <bug_when>2017-04-20 00:01:16 -0700</bug_when>
    <thetext>Attachment 307566 did not pass style-queue:


ERROR: Tools/BuildSlaveSupport/build.webkit.org-config/mastercfg_unittest.py:155:  [RunTest262TestsTest.test_no_regressions_output] Undefined variable &apos;SUCCESS&apos;  [pylint/E0602] [5]
ERROR: Tools/BuildSlaveSupport/build.webkit.org-config/mastercfg_unittest.py:165:  [RunTest262TestsTest.test_failure_output] Undefined variable &apos;FAILURE&apos;  [pylint/E0602] [5]
ERROR: Tools/BuildSlaveSupport/build.webkit.org-config/mastercfg_unittest.py:177:  [RunTest262TestsTest.test_failures_output] Undefined variable &apos;FAILURE&apos;  [pylint/E0602] [5]
Total errors found: 3 in 3 files


If any of these errors are false positives, please file a bug against check-webkit-style.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1299316</commentid>
    <comment_count>4</comment_count>
      <attachid>307566</attachid>
    <who name="Aakash Jain">aakash_jain</who>
    <bug_when>2017-04-20 14:59:37 -0700</bug_when>
    <thetext>Comment on attachment 307566
[PATCH] Proposed Fix

View in context: https://bugs.webkit.org/attachment.cgi?id=307566&amp;action=review

looks good to me. minor comments inline.

&gt; Tools/BuildSlaveSupport/build.webkit.org-config/mastercfg_unittest.py:155
&gt; +        self.assertResults(SUCCESS, [&quot;test262-test&quot;], 0, &quot;&quot;&quot;Using the following jsc path: /Volumes/Data/slave/elcapitan-release-tests-test262/build/WebKitBuild/Release/jsc

do we need to have the complete path having &apos;elcapitan-release-tests&apos; for unit-tests? previous path seems fine to me.

&gt; Tools/BuildSlaveSupport/build.webkit.org-config/mastercfg_unittest.py:165
&gt; +        self.assertResults(FAILURE, [&quot;1 Test262 test failed&quot;], 0, &quot;&quot;&quot;Using the following jsc path: /Volumes/Data/slave/elcapitan-release-tests-test262/build/WebKitBuild/Release/jsc

Ditto.

&gt; Tools/BuildSlaveSupport/build.webkit.org-config/mastercfg_unittest.py:171
&gt; +Running test262.yaml/test262/test/annexB/built-ins/Date/prototype/getYear/length.js.default-strict

Do we need to have 4 lines for passing test-cases? probably just 1 would be enough, since we don&apos;t use them anyways.

&gt; Tools/BuildSlaveSupport/build.webkit.org-config/mastercfg_unittest.py:177
&gt; +        self.assertResults(FAILURE, [&quot;2 Test262 tests failed&quot;], 0, &quot;&quot;&quot;Using the following jsc path: /Volumes/Data/slave/elcapitan-release-tests-test262/build/WebKitBuild/Release/jsc

Ditto.

&gt; Tools/BuildSlaveSupport/build.webkit.org-config/mastercfg_unittest.py:182
&gt; +Running test262.yaml/test262/test/annexB/built-ins/Date/prototype/getYear/length.js.default

Ditto. Do we need to have 3 lines for passing test-cases? probably just 1 line would be enough.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1299318</commentid>
    <comment_count>5</comment_count>
    <who name="Joseph Pecoraro">joepeck</who>
    <bug_when>2017-04-20 15:01:09 -0700</bug_when>
    <thetext>(In reply to Aakash Jain from comment #4)
&gt; Comment on attachment 307566 [details]
&gt; [PATCH] Proposed Fix
&gt; 
&gt; View in context:
&gt; https://bugs.webkit.org/attachment.cgi?id=307566&amp;action=review
&gt; 
&gt; looks good to me. minor comments inline.
&gt; 
&gt; &gt; Tools/BuildSlaveSupport/build.webkit.org-config/mastercfg_unittest.py:155
&gt; &gt; +        self.assertResults(SUCCESS, [&quot;test262-test&quot;], 0, &quot;&quot;&quot;Using the following jsc path: /Volumes/Data/slave/elcapitan-release-tests-test262/build/WebKitBuild/Release/jsc
&gt; 
&gt; do we need to have the complete path having &apos;elcapitan-release-tests&apos; for
&gt; unit-tests? previous path seems fine to me.
&gt; 
&gt; &gt; Tools/BuildSlaveSupport/build.webkit.org-config/mastercfg_unittest.py:165
&gt; &gt; +        self.assertResults(FAILURE, [&quot;1 Test262 test failed&quot;], 0, &quot;&quot;&quot;Using the following jsc path: /Volumes/Data/slave/elcapitan-release-tests-test262/build/WebKitBuild/Release/jsc
&gt; 
&gt; Ditto.
&gt; 
&gt; &gt; Tools/BuildSlaveSupport/build.webkit.org-config/mastercfg_unittest.py:171
&gt; &gt; +Running test262.yaml/test262/test/annexB/built-ins/Date/prototype/getYear/length.js.default-strict
&gt; 
&gt; Do we need to have 4 lines for passing test-cases? probably just 1 would be
&gt; enough, since we don&apos;t use them anyways.
&gt; 
&gt; &gt; Tools/BuildSlaveSupport/build.webkit.org-config/mastercfg_unittest.py:177
&gt; &gt; +        self.assertResults(FAILURE, [&quot;2 Test262 tests failed&quot;], 0, &quot;&quot;&quot;Using the following jsc path: /Volumes/Data/slave/elcapitan-release-tests-test262/build/WebKitBuild/Release/jsc
&gt; 
&gt; Ditto.
&gt; 
&gt; &gt; Tools/BuildSlaveSupport/build.webkit.org-config/mastercfg_unittest.py:182
&gt; &gt; +Running test262.yaml/test262/test/annexB/built-ins/Date/prototype/getYear/length.js.default
&gt; 
&gt; Ditto. Do we need to have 3 lines for passing test-cases? probably just 1
&gt; line would be enough.


The intent was to make the output as realistic as possible, and here I just pulled real output from a bot and modified it. But yes, I can simplify it.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1299440</commentid>
    <comment_count>6</comment_count>
    <who name="Joseph Pecoraro">joepeck</who>
    <bug_when>2017-04-20 18:50:18 -0700</bug_when>
    <thetext>&lt;https://trac.webkit.org/changeset/215599/webkit&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>307566</attachid>
            <date>2017-04-20 00:00:14 -0700</date>
            <delta_ts>2017-04-20 18:47:26 -0700</delta_ts>
            <desc>[PATCH] Proposed Fix</desc>
            <filename>proposed-fix-1.patch</filename>
            <type>text/plain</type>
            <size>5650</size>
            <attacher name="Joseph Pecoraro">joepeck</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1Rvb2xzL0J1aWxkU2xhdmVTdXBwb3J0L2J1aWxkLndlYmtpdC5vcmctY29u
ZmlnL21hc3Rlci5jZmcgYi9Ub29scy9CdWlsZFNsYXZlU3VwcG9ydC9idWlsZC53ZWJraXQub3Jn
LWNvbmZpZy9tYXN0ZXIuY2ZnCmluZGV4IDdlMWU4NWFmZTQ1Li40Y2UxZTY3ZjQ4YyAxMDA2NDQK
LS0tIGEvVG9vbHMvQnVpbGRTbGF2ZVN1cHBvcnQvYnVpbGQud2Via2l0Lm9yZy1jb25maWcvbWFz
dGVyLmNmZworKysgYi9Ub29scy9CdWlsZFNsYXZlU3VwcG9ydC9idWlsZC53ZWJraXQub3JnLWNv
bmZpZy9tYXN0ZXIuY2ZnCkBAIC0zMzUsOSArMzM1LDkgQEAgY2xhc3MgUnVuVGVzdDI2MlRlc3Rz
KFRlc3RXaXRoRmFpbHVyZUNvdW50KToKIAogICAgIGRlZiBjb3VudEZhaWx1cmVzKHNlbGYsIGNt
ZCk6CiAgICAgICAgIGxvZ1RleHQgPSBjbWQubG9nc1snc3RkaW8nXS5nZXRUZXh0KCkKLSAgICAg
ICAgbWF0Y2hlcyA9IHJlLmZpbmRhbGwocidcKGZhaWxlZCAoXGQrKVwpJywgbG9nVGV4dCkKKyAg
ICAgICAgbWF0Y2hlcyA9IHJlLmZpbmRhbGwocideRkFJTDonLCBsb2dUZXh0LCBmbGFncz1yZS5N
VUxUSUxJTkUpCiAgICAgICAgIGlmIG1hdGNoZXM6Ci0gICAgICAgICAgICByZXR1cm4gaW50KG1h
dGNoZXNbLTFdKQorICAgICAgICAgICAgcmV0dXJuIGxlbihtYXRjaGVzKQogICAgICAgICByZXR1
cm4gMAogCiAKZGlmZiAtLWdpdCBhL1Rvb2xzL0J1aWxkU2xhdmVTdXBwb3J0L2J1aWxkLndlYmtp
dC5vcmctY29uZmlnL21hc3RlcmNmZ191bml0dGVzdC5weSBiL1Rvb2xzL0J1aWxkU2xhdmVTdXBw
b3J0L2J1aWxkLndlYmtpdC5vcmctY29uZmlnL21hc3RlcmNmZ191bml0dGVzdC5weQppbmRleCA2
NmUzNmMzMWE3My4uZjIxNDhiNjZjOTYgMTAwNzU1Ci0tLSBhL1Rvb2xzL0J1aWxkU2xhdmVTdXBw
b3J0L2J1aWxkLndlYmtpdC5vcmctY29uZmlnL21hc3RlcmNmZ191bml0dGVzdC5weQorKysgYi9U
b29scy9CdWlsZFNsYXZlU3VwcG9ydC9idWlsZC53ZWJraXQub3JnLWNvbmZpZy9tYXN0ZXJjZmdf
dW5pdHRlc3QucHkKQEAgLTE1MiwyMCArMTUyLDQwIEBAIGNsYXNzIFJ1blRlc3QyNjJUZXN0c1Rl
c3QodW5pdHRlc3QuVGVzdENhc2UpOgogICAgICAgICBzZWxmLmFzc2VydEVxdWFsKGFjdHVhbF90
ZXh0LCBleHBlY3RlZF90ZXh0KQogCiAgICAgZGVmIHRlc3Rfbm9fcmVncmVzc2lvbnNfb3V0cHV0
KHNlbGYpOgotICAgICAgICBzZWxmLmFzc2VydFJlc3VsdHMoU1VDQ0VTUywgWyJ0ZXN0MjYyLXRl
c3QiXSwgMCwgIiIiVXNpbmcgdGhlIGZvbGxvd2luZyBqc2MgcGF0aDogL1dlYktpdEJ1aWxkL1Jl
bGVhc2UvanNjCi0xNjgvMTY4ICAgICAgICAgIiIiKQorICAgICAgICBzZWxmLmFzc2VydFJlc3Vs
dHMoU1VDQ0VTUywgWyJ0ZXN0MjYyLXRlc3QiXSwgMCwgIiIiVXNpbmcgdGhlIGZvbGxvd2luZyBq
c2MgcGF0aDogL1ZvbHVtZXMvRGF0YS9zbGF2ZS9lbGNhcGl0YW4tcmVsZWFzZS10ZXN0cy10ZXN0
MjYyL2J1aWxkL1dlYktpdEJ1aWxkL1JlbGVhc2UvanNjCisKKworUnVubmluZyB0ZXN0MjYyLnlh
bWwvdGVzdDI2Mi90ZXN0L2FubmV4Qi9idWlsdC1pbnMvRGF0ZS9wcm90b3R5cGUvZ2V0WWVhci9C
LjIuNC5qcy5kZWZhdWx0CitSdW5uaW5nIHRlc3QyNjIueWFtbC90ZXN0MjYyL3Rlc3QvYW5uZXhC
L2J1aWx0LWlucy9EYXRlL3Byb3RvdHlwZS9nZXRZZWFyL0IuMi40LmpzLmRlZmF1bHQtc3RyaWN0
CitSdW5uaW5nIHRlc3QyNjIueWFtbC90ZXN0MjYyL3Rlc3QvYW5uZXhCL2J1aWx0LWlucy9EYXRl
L3Byb3RvdHlwZS9nZXRZZWFyL2xlbmd0aC5qcy5kZWZhdWx0CitSdW5uaW5nIHRlc3QyNjIueWFt
bC90ZXN0MjYyL3Rlc3QvYW5uZXhCL2J1aWx0LWlucy9EYXRlL3Byb3RvdHlwZS9nZXRZZWFyL2xl
bmd0aC5qcy5kZWZhdWx0LXN0cmljdAorIiIiKQogCiAgICAgZGVmIHRlc3RfZmFpbHVyZV9vdXRw
dXQoc2VsZik6Ci0gICAgICAgIHNlbGYuYXNzZXJ0UmVzdWx0cyhGQUlMVVJFLCBbIjEgVGVzdDI2
MiB0ZXN0IGZhaWxlZCJdLCAxLCAiIiJVc2luZyB0aGUgZm9sbG93aW5nIGpzYyBwYXRoOiAvV2Vi
S2l0QnVpbGQvUmVsZWFzZS9qc2MKLXRlc3QyNjIueWFtbC90ZXN0MjYyL3Rlc3QvYnVpbHQtaW5z
L0FycmF5L2Zyb20vaXRlci1zZXQtZWxlbS1wcm9wLWVyci5qcy5kZWZhdWx0OiBFUlJPUjogVW5l
eHBlY3RlZCBleGl0IGNvZGU6IDAKLXRlc3QyNjIueWFtbC90ZXN0MjYyL3Rlc3QvYnVpbHQtaW5z
L0FycmF5L2Zyb20vaXRlci1zZXQtZWxlbS1wcm9wLWVyci5qcy5kZWZhdWx0LXN0cmljdDogRVJS
T1I6IFVuZXhwZWN0ZWQgZXhpdCBjb2RlOiAwCi0uLi4KLTQzNzY4LzQzNzY4IChmYWlsZWQgMSkg
ICAgICAgICAiIiIpCisgICAgICAgIHNlbGYuYXNzZXJ0UmVzdWx0cyhGQUlMVVJFLCBbIjEgVGVz
dDI2MiB0ZXN0IGZhaWxlZCJdLCAwLCAiIiJVc2luZyB0aGUgZm9sbG93aW5nIGpzYyBwYXRoOiAv
Vm9sdW1lcy9EYXRhL3NsYXZlL2VsY2FwaXRhbi1yZWxlYXNlLXRlc3RzLXRlc3QyNjIvYnVpbGQv
V2ViS2l0QnVpbGQvUmVsZWFzZS9qc2MKKworCitSdW5uaW5nIHRlc3QyNjIueWFtbC90ZXN0MjYy
L3Rlc3QvYW5uZXhCL2J1aWx0LWlucy9EYXRlL3Byb3RvdHlwZS9nZXRZZWFyL0IuMi40LmpzLmRl
ZmF1bHQKK1J1bm5pbmcgdGVzdDI2Mi55YW1sL3Rlc3QyNjIvdGVzdC9hbm5leEIvYnVpbHQtaW5z
L0RhdGUvcHJvdG90eXBlL2dldFllYXIvQi4yLjQuanMuZGVmYXVsdC1zdHJpY3QKK1J1bm5pbmcg
dGVzdDI2Mi55YW1sL3Rlc3QyNjIvdGVzdC9hbm5leEIvYnVpbHQtaW5zL0RhdGUvcHJvdG90eXBl
L2dldFllYXIvbGVuZ3RoLmpzLmRlZmF1bHQKK1J1bm5pbmcgdGVzdDI2Mi55YW1sL3Rlc3QyNjIv
dGVzdC9hbm5leEIvYnVpbHQtaW5zL0RhdGUvcHJvdG90eXBlL2dldFllYXIvbGVuZ3RoLmpzLmRl
ZmF1bHQtc3RyaWN0Cit0ZXN0MjYyLnlhbWwvdGVzdDI2Mi90ZXN0L2FubmV4Qi9idWlsdC1pbnMv
RGF0ZS9wcm90b3R5cGUvZ2V0WWVhci9sZW5ndGguanMuZGVmYXVsdC1zdHJpY3Q6IEVSUk9SOiBV
bmV4cGVjdGVkIGV4aXQgY29kZTogMAorRkFJTDogdGVzdDI2Mi55YW1sL3Rlc3QyNjIvdGVzdC9h
bm5leEIvYnVpbHQtaW5zL0RhdGUvcHJvdG90eXBlL2dldFllYXIvbGVuZ3RoLmpzLmRlZmF1bHQt
c3RyaWN0CisiIiIpCiAKICAgICBkZWYgdGVzdF9mYWlsdXJlc19vdXRwdXQoc2VsZik6Ci0gICAg
ICAgIHNlbGYuYXNzZXJ0UmVzdWx0cyhGQUlMVVJFLCBbIjc1IFRlc3QyNjIgdGVzdHMgZmFpbGVk
Il0sIDc1LCAiIiJVc2luZyB0aGUgZm9sbG93aW5nIGpzYyBwYXRoOiAvV2ViS2l0QnVpbGQvUmVs
ZWFzZS9qc2MKLS4uLgotNDM3NjgvNDM3NjggKGZhaWxlZCA3NSkgICAgICAgICAiIiIpCisgICAg
ICAgIHNlbGYuYXNzZXJ0UmVzdWx0cyhGQUlMVVJFLCBbIjIgVGVzdDI2MiB0ZXN0cyBmYWlsZWQi
XSwgMCwgIiIiVXNpbmcgdGhlIGZvbGxvd2luZyBqc2MgcGF0aDogL1ZvbHVtZXMvRGF0YS9zbGF2
ZS9lbGNhcGl0YW4tcmVsZWFzZS10ZXN0cy10ZXN0MjYyL2J1aWxkL1dlYktpdEJ1aWxkL1JlbGVh
c2UvanNjCisKKworUnVubmluZyB0ZXN0MjYyLnlhbWwvdGVzdDI2Mi90ZXN0L2FubmV4Qi9idWls
dC1pbnMvRGF0ZS9wcm90b3R5cGUvZ2V0WWVhci9CLjIuNC5qcy5kZWZhdWx0CitSdW5uaW5nIHRl
c3QyNjIueWFtbC90ZXN0MjYyL3Rlc3QvYW5uZXhCL2J1aWx0LWlucy9EYXRlL3Byb3RvdHlwZS9n
ZXRZZWFyL0IuMi40LmpzLmRlZmF1bHQtc3RyaWN0CitSdW5uaW5nIHRlc3QyNjIueWFtbC90ZXN0
MjYyL3Rlc3QvYW5uZXhCL2J1aWx0LWlucy9EYXRlL3Byb3RvdHlwZS9nZXRZZWFyL2xlbmd0aC5q
cy5kZWZhdWx0Cit0ZXN0MjYyLnlhbWwvdGVzdDI2Mi90ZXN0L2FubmV4Qi9idWlsdC1pbnMvRGF0
ZS9wcm90b3R5cGUvZ2V0WWVhci9sZW5ndGguanMuZGVmYXVsdDogRVJST1I6IFVuZXhwZWN0ZWQg
ZXhpdCBjb2RlOiAwCitGQUlMOiB0ZXN0MjYyLnlhbWwvdGVzdDI2Mi90ZXN0L2FubmV4Qi9idWls
dC1pbnMvRGF0ZS9wcm90b3R5cGUvZ2V0WWVhci9sZW5ndGguanMuZGVmYXVsdAorUnVubmluZyB0
ZXN0MjYyLnlhbWwvdGVzdDI2Mi90ZXN0L2FubmV4Qi9idWlsdC1pbnMvRGF0ZS9wcm90b3R5cGUv
Z2V0WWVhci9sZW5ndGguanMuZGVmYXVsdC1zdHJpY3QKK3Rlc3QyNjIueWFtbC90ZXN0MjYyL3Rl
c3QvYW5uZXhCL2J1aWx0LWlucy9EYXRlL3Byb3RvdHlwZS9nZXRZZWFyL2xlbmd0aC5qcy5kZWZh
dWx0LXN0cmljdDogRVJST1I6IFVuZXhwZWN0ZWQgZXhpdCBjb2RlOiAwCitGQUlMOiB0ZXN0MjYy
LnlhbWwvdGVzdDI2Mi90ZXN0L2FubmV4Qi9idWlsdC1pbnMvRGF0ZS9wcm90b3R5cGUvZ2V0WWVh
ci9sZW5ndGguanMuZGVmYXVsdC1zdHJpY3QKKyIiIikKIAogCiBjbGFzcyBSdW5MTElOVENMb29w
VGVzdHNUZXN0KHVuaXR0ZXN0LlRlc3RDYXNlKToKZGlmZiAtLWdpdCBhL1Rvb2xzL0NoYW5nZUxv
ZyBiL1Rvb2xzL0NoYW5nZUxvZwppbmRleCAxZmVkM2QyMTRiOS4uODk2ZGFkODk0YWMgMTAwNjQ0
Ci0tLSBhL1Rvb2xzL0NoYW5nZUxvZworKysgYi9Ub29scy9DaGFuZ2VMb2cKQEAgLTEsMyArMSwy
NSBAQAorMjAxNy0wNC0xOSAgSm9zZXBoIFBlY29yYXJvICA8cGVjb3Jhcm9AYXBwbGUuY29tPgor
CisgICAgICAgIFRlc3QyNjIgYm90IGRvZXMgbm90IGdvIHJlZCB3aXRoIGZhaWx1cmVzCisgICAg
ICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xNzEwNDQKKworICAg
ICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBUaGUgcGlwZWQgb3V0
cHV0IG9mIHJ1bi1qc2Mtc3RyZXNzLXRlc3RzIGlzIGRpZmZlcmVudCBmcm9tCisgICAgICAgIGl0
cyBub24tcGlwZWQgb3V0cHV0LiBTd2l0Y2ggdG8gcGFyc2luZyB0aGUgY29ycmVjdCBvdXRwdXQu
CisKKyAgICAgICAgKiBCdWlsZFNsYXZlU3VwcG9ydC9idWlsZC53ZWJraXQub3JnLWNvbmZpZy9t
YXN0ZXIuY2ZnOgorICAgICAgICAoUnVuVGVzdDI2MlRlc3RzLmNvdW50RmFpbHVyZXMpOgorICAg
ICAgICBKdXN0IGNvdW50IHRoZSBudW1iZXIgb2YgbGluZXMgdGhhdCBzdGFydCB3aXRoICJGQUlM
OiIuCisKKyAgICAgICAgKiBCdWlsZFNsYXZlU3VwcG9ydC9idWlsZC53ZWJraXQub3JnLWNvbmZp
Zy9tYXN0ZXJjZmdfdW5pdHRlc3QucHk6CisgICAgICAgIChSdW5UZXN0MjYyVGVzdHNUZXN0LnRl
c3Rfbm9fcmVncmVzc2lvbnNfb3V0cHV0KToKKyAgICAgICAgKHRlc3RfZmFpbHVyZV9vdXRwdXQp
OgorICAgICAgICAodGVzdF9mYWlsdXJlc19vdXRwdXQpOgorICAgICAgICBVcGRhdGVkIHRlc3Rz
IHdpdGggbmV3IHNhbXBsZSBvdXRwdXQgcHJvZHVjZWQgcGFydGlhbGx5IGZyb20gYm90cyBhbmQK
KyAgICAgICAgbG9jYWxseS4gSW4gY2FzZXMgb2YgZmFpbHVyZSB0aGUgcmV0dXJuIGNvZGUgb2Yg
cnVuLWpzYy1zdHJlc3MtdGVzdHMKKyAgICAgICAgaXMgc3RpbGwgMCBzbyB1cGRhdGUgdGhlIGV4
cGVjdGVkIHJldHVybiBjb2RlIGluIHRoZSB0ZXN0cy4KKwogMjAxNy0wNC0xOCAgV2Vuc29uIEhz
aWVoICA8d2Vuc29uX2hzaWVoQGFwcGxlLmNvbT4KIAogICAgICAgICBbV0syXSBBZGQgaW5mcmFz
dHJ1Y3R1cmUgYW5kIHVuaXQgdGVzdHMgZm9yIGZpbGUgdXBsb2FkcyB1c2luZyBkYXRhIGludGVy
YWN0aW9uCg==
</data>
<flag name="review"
          id="328837"
          type_id="1"
          status="+"
          setter="aakash_jain"
    />
          </attachment>
      

    </bug>

</bugzilla>