New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: ping SpecRef in ordering-safe way #1499
Conversation
tests/spec/core/biblio-spec.js
Outdated
const bibRefsURL = new URL("https://specref.herokuapp.com/bibrefs"); | ||
const specRefPing = fetch(bibRefsURL, { method: "HEAD" }); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let's move this to beforeAll
, and await
it there...
specRefOk = (await specRefPing).ok;
tests/spec/core/biblio-spec.js
Outdated
@@ -85,7 +85,7 @@ describe("W3C — Bibliographic References", () => { | |||
let ref = doc.querySelector("#bib-TestRef1 + dd"); | |||
expect(ref).toBeTruthy(); | |||
// This prevents Jasmine from taking down the whole test suite if SpecRef is down. | |||
if (!isSpecRefAvailable) { | |||
if (!(await specRefPing).ok) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ooops... here needs updating too.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Small bug on line 88.
tests/spec/core/biblio-spec.js
Outdated
@@ -85,7 +84,7 @@ describe("W3C — Bibliographic References", () => { | |||
let ref = doc.querySelector("#bib-TestRef1 + dd"); | |||
expect(ref).toBeTruthy(); | |||
// This prevents Jasmine from taking down the whole test suite if SpecRef is down. | |||
if (!(await specRefPing).ok) { | |||
if (!specRefOk.ok) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, this should just be if(!specRefOk){ ... }
as specRefOk
is just a boolean and won't have a .ok
member, right?
Sorry for intermediate messes 🤣 |
So, I guess this one closes #828 |
* develop: v19.2.0 docs(examples): use github config option (#1500) fix(tools/release): use build:components instead (#1501) fix: ping SpecRef in ordering-safe way (#1499) chore: update dependencies (#1498) fix: prevent unordered tests from conflicting (#1495) refactor: remove WebIDL array support (#1496) Feat: link to pull requests in headers Skip detectBrowsers when explicitly passing browsers (#1490) refactor: use node.js api rather than unix commands (#1488) refactor: use puppeteer instead of nightmare (#1483) chore(package): update jasmine-core to version 3.0.0 (#1481) workaround: use sudo required on travis (#1478)
No description provided.