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
access-control-preflight-headers-sync.htm 
<!DOCTYPE html>
<html>
  <head>
    <title>Test that sync CORS requests with custom headers are not sent with OPTIONS preflight</title>
    <script src="/resources/testharness.js"></script>
    <script src="/resources/testharnessreport.js"></script>
    <script src="/common/get-host-info.sub.js"></script>
    <script src="/common/utils.js"></script>
  </head>
  <body>
    <script type="text/javascript">
    test(function() {
      let xhr = new XMLHttpRequest;
      const uuid = token();

      xhr.open("GET", get_host_info().HTTP_REMOTE_ORIGIN +
          "/xhr/resources/reset-token.py?token=" + uuid, false);
      xhr.send();

      xhr = new XMLHttpRequest;
      xhr.open("GET", get_host_info().HTTP_REMOTE_ORIGIN +
          "/xhr/resources/no-custom-header-on-preflight.py?token=" + uuid, false);
      xhr.setRequestHeader("x-test", "foobar");
      xhr.send();
      assert_equals(xhr.responseText, "PASS");
    }, "Preflighted sync request with custom header");
    </script>
  </body>
</html>
back to top