Skip to main content

Browse the archive

Revision 410fe8465f4c5a51d25d95c62a53b569d980c1cf authored by Robert Ma on 19 March 2018, 15:34:37 UTC, committed by Robert Ma on 19 March 2018, 15:34:37 UTC 
Exit check_stability with an error when no tests run
 
Similar to #10030 (which was for `wpt run`), this commit makes `wpt
check-stability` return a non-zero exit code when no tests run.

Also fix some code smells:

* `run` was redefined by `do_delayed_imports` to a different function.
  Rename the other `run` method in this module to `setup_and_run`.
* `retcode` was unnecessarily defined in the top scope. Remove it to
  avoid redefinition.
* `global logger` and the call to `get_parser` were extraneous in `run`
  (now `setup_and_run`) and hence are removed.
1 parent 9b3bb05
Raw File
event-readystate-sync-open.htm 
<!DOCTYPE html>
<html>
  <head>
    <title>XMLHttpRequest: open() call fires sync readystate event</title>
    <script src="/resources/testharness.js"></script>
    <script src="/resources/testharnessreport.js"></script>
    <link rel="help" href="https://xhr.spec.whatwg.org/#the-open()-method" data-tested-assertations="following::ol[1]/li[13]/ol[1]/li[2]" />

  </head>
  <body>
    <div id="log"></div>
    <script>
      test(function() {
        var client = new XMLHttpRequest()
        var eventsFired = []
        client.onreadystatechange = function(){
          eventsFired.push(client.readyState)
        }
        client.open('GET', "...", false)
        assert_array_equals(eventsFired, [1])
      }, document.title + ' (sync)')
      test(function() {
        var client = new XMLHttpRequest()
        var eventsFired = []
        client.onreadystatechange = function(){
          eventsFired.push(client.readyState)
        }
        client.open('GET', "...", true)
        assert_array_equals(eventsFired, [1])
      }, document.title + ' (async)')
    </script>
  </body>
</html>
back to top