Skip to content

Commit

Permalink
chore: Upgrade jest to 23.6.0 (#11409)
Browse files Browse the repository at this point in the history
The failed tests that had to be fixed relate to jestjs/jest#6792
  • Loading branch information
evanpurkhiser committed Jan 8, 2019
1 parent 3e16607 commit 2f79b6f
Show file tree
Hide file tree
Showing 6 changed files with 134 additions and 125 deletions.
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -111,7 +111,7 @@
"enzyme-to-json": "3.3.1",
"eslint": "4.4.1",
"eslint-config-sentry-app": "1.4.3",
"jest": "23.4.0",
"jest": "23.6.0",
"jest-junit": "^3.4.1",
"mockdate": "2.0.2",
"prettier": "1.7.4",
Expand Down
2 changes: 1 addition & 1 deletion tests/js/spec/components/modals/sudoModal.spec.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -153,6 +153,6 @@ describe('Sudo Modal', function() {

// Should have Modal + input
expect(wrapper.find('ModalDialog input')).toHaveLength(0);
expect(wrapper.find('Button').prop('href')).toMatch('/auth/login/?next=blank');
expect(wrapper.find('Button').prop('href')).toMatch('/auth/login/?next=%2F');
});
});
4 changes: 2 additions & 2 deletions tests/js/spec/views/__snapshots__/installWizard.spec.jsx.snap
Original file line number Diff line number Diff line change
Expand Up @@ -115,7 +115,7 @@ exports[`InstallWizard renders 1`] = `
allowEmpty={false}
component={[Function]}
default="root@localhost"
defaultValue="sentry@"
defaultValue="sentry@localhost"
disabled={false}
disabledReason={null}
help="Email address to be used in From for all outbound email."
Expand Down Expand Up @@ -146,7 +146,7 @@ exports[`InstallWizard renders 1`] = `
onChange={[Function]}
required={true}
type="email"
value="sentry@"
value="sentry@localhost"
/>
<p
className="help-block"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ exports[`AdminSettings render() renders 1`] = `
<TextField
allowEmpty={true}
default=""
defaultValue="about://"
defaultValue="http://localhost"
disabled={true}
disabledReason="This setting is defined in config.yml and may not be changed via the web UI."
help="The root web address which is used to communicate with the Sentry backend."
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ describe('AddIntegration', function() {

const newIntegration = {
source: null,
origin: 'null',
origin: 'http://localhost',
data: {
success: true,
data: Object.assign({}, integration, {
Expand Down

0 comments on commit 2f79b6f

Please sign in to comment.