✅ Improve performance timings test coverage

elbywan · Aug 13, 2018 · c4db506 · c4db506
1 parent 429fd8d
commit c4db506
Showing 1 changed file with 14 additions and 6 deletions.
diff --git a/test/node/wretch.spec.ts b/test/node/wretch.spec.ts
@@ -350,25 +350,33 @@ describe("Wretch", function() {
  fetch: fetchPolyfill(1)
  })
  // Test empty perfs()
- wretch(`${_URL}/text`).get().perfs().res(_ => expect(_.ok).toBeTruthy()).then(_ =>
+ wretch(`${_URL}/text`).get().perfs().res(_ => expect(_.ok).toBeTruthy()).then(_ => {
  // Racing condition : observer triggered before response
- wretch(`${_URL}/bla`).get().perfs(_ => {
- expect(typeof _.startTime).toBe("number")
+ wretch(`${_URL}/bla`).get().perfs(timings => {
+ expect(timings.name).toBe(`${_URL}/bla`)
+ expect(typeof timings.startTime).toBe("number")
 
  // Racing condition : response triggered before observer
  wretch().polyfills({
  fetch: fetchPolyfill(1000)
  })
 
- wretch(`${_URL}/fakeurl`).get().perfs(_ => {
- expect(typeof _.startTime).toBe("number")
+ wretch(`${_URL}/fakeurl`).get().perfs(timings => {
+ expect(timings.name).toBe(`${_URL}/fakeurl`)
+ expect(typeof timings.startTime).toBe("number")
  wretch().polyfills({
  fetch: fetchPolyfill(1)
  })
  done()
  }).res().catch(() => "ignore")
  }).res().catch(_ => "ignore")
- )
+ })
+ // Test multiple requests concurrency and proper timings dispatching
+ for(let i = 0; i < 5; i++) {
+ wretch(`${_URL}/fake/${i}`).get().perfs(timings => {
+ expect(timings.name).toBe(`${_URL}/fake/${i}`)
+ }).res().catch(() => "ignore")
+ }
  })
 
  it("should abort a request", function(done) {