diff options
Diffstat (limited to 'spec/frontend/performance_bar/services/performance_bar_service_spec.js')
-rw-r--r-- | spec/frontend/performance_bar/services/performance_bar_service_spec.js | 43 |
1 files changed, 21 insertions, 22 deletions
diff --git a/spec/frontend/performance_bar/services/performance_bar_service_spec.js b/spec/frontend/performance_bar/services/performance_bar_service_spec.js index cfec4b779e4..36bfd575c12 100644 --- a/spec/frontend/performance_bar/services/performance_bar_service_spec.js +++ b/spec/frontend/performance_bar/services/performance_bar_service_spec.js @@ -9,18 +9,12 @@ describe('PerformanceBarService', () => { it('returns false when the request URL is the peek URL', () => { expect( - fireCallback({ headers: { 'x-request-id': '123' }, url: '/peek' }, '/peek'), - ).toBeFalsy(); + fireCallback({ headers: { 'x-request-id': '123' }, config: { url: '/peek' } }, '/peek'), + ).toBe(false); }); it('returns false when there is no request ID', () => { - expect(fireCallback({ headers: {}, url: '/request' }, '/peek')).toBeFalsy(); - }); - - it('returns false when the request is an API request', () => { - expect( - fireCallback({ headers: { 'x-request-id': '123' }, url: '/api/' }, '/peek'), - ).toBeFalsy(); + expect(fireCallback({ headers: {}, config: { url: '/request' } }, '/peek')).toBe(false); }); it('returns false when the response is from the cache', () => { @@ -29,13 +23,22 @@ describe('PerformanceBarService', () => { { headers: { 'x-request-id': '123', 'x-gitlab-from-cache': 'true' }, url: '/request' }, '/peek', ), - ).toBeFalsy(); + ).toBe(false); + }); + + it('returns true when the request is an API request', () => { + expect( + fireCallback({ headers: { 'x-request-id': '123' }, config: { url: '/api/' } }, '/peek'), + ).toBe(true); }); - it('returns true when all conditions are met', () => { + it('returns true for all other requests', () => { expect( - fireCallback({ headers: { 'x-request-id': '123' }, url: '/request' }, '/peek'), - ).toBeTruthy(); + fireCallback( + { headers: { 'x-request-id': '123' }, config: { url: '/request' } }, + '/peek', + ), + ).toBe(true); }); }); @@ -45,7 +48,7 @@ describe('PerformanceBarService', () => { } it('gets the request ID from the headers', () => { - expect(requestId({ headers: { 'x-request-id': '123' } }, '/peek')).toEqual('123'); + expect(requestId({ headers: { 'x-request-id': '123' } }, '/peek')).toBe('123'); }); }); @@ -54,14 +57,10 @@ describe('PerformanceBarService', () => { return PerformanceBarService.callbackParams(response, peekUrl)[2]; } - it('gets the request URL from the response object', () => { - expect(requestUrl({ headers: {}, url: '/request' }, '/peek')).toEqual('/request'); - }); - - it('gets the request URL from response.config if present', () => { - expect( - requestUrl({ headers: {}, config: { url: '/config-url' }, url: '/request' }, '/peek'), - ).toEqual('/config-url'); + it('gets the request URL from response.config', () => { + expect(requestUrl({ headers: {}, config: { url: '/config-url' } }, '/peek')).toBe( + '/config-url', + ); }); }); }); |