Skip to content
This repository has been archived by the owner on Nov 28, 2022. It is now read-only.

Commit

Permalink
Update skipped unwritten tests to deal with jest regression
Browse files Browse the repository at this point in the history
  • Loading branch information
Dom Harrington committed Aug 2, 2018
1 parent 41f40da commit 84deae8
Show file tree
Hide file tree
Showing 6 changed files with 11 additions and 11 deletions.
2 changes: 1 addition & 1 deletion packages/api-explorer/__tests__/CodeSample.test.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ const props = {

describe('tabs', () => {
// TODO this doesnt work in readme
test('should display tabs if there are examples in the oas file');
test.skip('should display tabs if there are examples in the oas file', () => {});

test('should display tabs if SAMPLES_ENABLED is true', () => {
const languages = ['node', 'curl'];
Expand Down
4 changes: 2 additions & 2 deletions packages/api-explorer/__tests__/SecurityInput.test.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -49,8 +49,8 @@ describe('oauth2', () => {
expect(securityInput.find('input').prop('value')).toBe(apiKey);
});

test('should display markdown description');
test('should work for multiple oauths and allow selection');
test.skip('should display markdown description', () => {});
test.skip('should work for multiple oauths and allow selection', () => {});

test('should send auth apiKey into onChange()', () => {
const onChange = jest.fn();
Expand Down
6 changes: 3 additions & 3 deletions packages/api-explorer/__tests__/lib/Oas.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -224,12 +224,12 @@ describe('operation.prepareSecurity()', () => {
expect(operation.prepareSecurity().Header.length).toBe(1);
});

test('should set a `key` property');
test.skip('should set a `key` property', () => {});

// TODO We dont currently support cookies?
test('apiKey/cookie: should return with a type of Cookie');
test.skip('apiKey/cookie: should return with a type of Cookie', () => {});

test('should throw if attempting to use a non-existent scheme');
test.skip('should throw if attempting to use a non-existent scheme', () => {});

test('should return empty object if no security', () => {
const operation = new Oas(multipleSecurities).operation('/no-auth', 'post');
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -350,4 +350,4 @@ test('it should pull out schemas from `components/requestBodies`', () => {
]);
});

test('it should make things required correctly');
test.skip('it should make things required correctly', () => {});
6 changes: 3 additions & 3 deletions packages/api-explorer/__tests__/lib/show-code.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,9 @@ const extensions = require('@readme/oas-extensions');

const showCode = require('../../src/lib/show-code');

test('should return true if there are examples');
test('should return true if there are results');
test.skip('should return true if there are examples', () => {});
test.skip('should return true if there are results', () => {});
test('should return true if it has try it now', () => {
expect(showCode({ [extensions.EXPLORER_ENABLED]: true }, {})).toBe(true);
});
test('should return false otherwise');
test.skip('should return false otherwise', () => {});
2 changes: 1 addition & 1 deletion packages/api-explorer/__tests__/sanity-check-dist.test.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -7,4 +7,4 @@
// expect(() => shallow(<ApiExplorer />)).not.toThrow(/element.type is not a function/);
// });

test('This test is taking too long on CI right now');
test.skip('This test is taking too long on CI right now', () => {});

0 comments on commit 84deae8

Please sign in to comment.