Skip to content

Commit

Permalink
Merge branch 'fix/test-run-id' of github.com:kubeshop/tracetest into …
Browse files Browse the repository at this point in the history
…fix/test-run-id
  • Loading branch information
mathnogueira committed Sep 11, 2023
2 parents d9d2e96 + 4c24ccf commit c18ca89
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ const defaultProps = {
onClearSelectorSuggestions: jest.fn(),
onClickPrevSelector: jest.fn(),
testId: 'testId',
runId: 'runId',
runId: 1,
isValid: false,
onIsValid: jest.fn(),
};
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ import TestSpecs from '../TestSpecs.provider';
describe('TestDefinitionProvider', () => {
it('should render with the proper values', () => {
const {getByText} = render(
<TestSpecs testId="testId" runId="runId">
<TestSpecs testId="testId" runId={1}>
<div>
<p>Hello</p>
</div>
Expand Down
4 changes: 2 additions & 2 deletions web/src/redux/actions/__tests_/TestSpecs.actions.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ describe('TestDefinitionActions', () => {
expect(setRequest.url).toEqual('http://localhost/api/tests/testId');
expect(setRequest.method).toEqual(HTTP_METHOD.PUT);

expect(reRunRequest.url).toEqual('http://localhost/api/tests/testId/run/runId/rerun');
expect(reRunRequest.url).toEqual('http://localhost/api/tests/testId/run/1/rerun');
expect(reRunRequest.method).toEqual(HTTP_METHOD.POST);

expect(fetchMock.mock.calls.length).toBe(2);
Expand All @@ -59,7 +59,7 @@ describe('TestDefinitionActions', () => {

const dryRunRequest = fetchMock.mock.calls[0][0] as Request;

expect(dryRunRequest.url).toEqual('http://localhost/api/tests/testId/run/runId/dry-run');
expect(dryRunRequest.url).toEqual('http://localhost/api/tests/testId/run/1/dry-run');
expect(dryRunRequest.method).toEqual(HTTP_METHOD.PUT);

expect(fetchMock.mock.calls.length).toBe(1);
Expand Down

0 comments on commit c18ca89

Please sign in to comment.