diff options
Diffstat (limited to 'spec/frontend/releases/stores')
-rw-r--r-- | spec/frontend/releases/stores/modules/detail/actions_spec.js | 12 | ||||
-rw-r--r-- | spec/frontend/releases/stores/modules/detail/mutations_spec.js | 31 |
2 files changed, 23 insertions, 20 deletions
diff --git a/spec/frontend/releases/stores/modules/detail/actions_spec.js b/spec/frontend/releases/stores/modules/detail/actions_spec.js index 4a1790adb09..854f06821be 100644 --- a/spec/frontend/releases/stores/modules/detail/actions_spec.js +++ b/spec/frontend/releases/stores/modules/detail/actions_spec.js @@ -130,6 +130,15 @@ describe('Release detail actions', () => { }); }); + describe('updateReleaseMilestones', () => { + it(`commits ${types.UPDATE_RELEASE_MILESTONES} with the updated release milestones`, () => { + const newReleaseMilestones = ['v0.0', 'v0.1']; + return testAction(actions.updateReleaseMilestones, newReleaseMilestones, state, [ + { type: types.UPDATE_RELEASE_MILESTONES, payload: newReleaseMilestones }, + ]); + }); + }); + describe('requestUpdateRelease', () => { it(`commits ${types.REQUEST_UPDATE_RELEASE}`, () => testAction(actions.requestUpdateRelease, undefined, state, [ @@ -143,7 +152,7 @@ describe('Release detail actions', () => { { type: types.RECEIVE_UPDATE_RELEASE_SUCCESS }, ])); - describe('when the releaseShowPage feature flag is enabled', () => { + it('redirects to the releases page if releaseShowPage feature flag is enabled', () => { const rootState = { featureFlags: { releaseShowPage: true } }; const updatedState = merge({}, state, { releasesPagePath: 'path/to/releases/page', @@ -248,6 +257,7 @@ describe('Release detail actions', () => { { name: state.release.name, description: state.release.description, + milestones: state.release.milestones.map(milestone => milestone.title), }, ], ]); diff --git a/spec/frontend/releases/stores/modules/detail/mutations_spec.js b/spec/frontend/releases/stores/modules/detail/mutations_spec.js index cb5a1880b0c..f3f7ca797b4 100644 --- a/spec/frontend/releases/stores/modules/detail/mutations_spec.js +++ b/spec/frontend/releases/stores/modules/detail/mutations_spec.js @@ -1,10 +1,3 @@ -/* eslint-disable jest/valid-describe */ -/* - * ESLint disable directive ↑ can be removed once - * https://github.com/jest-community/eslint-plugin-jest/issues/203 - * is resolved - */ - import createState from '~/releases/stores/modules/detail/state'; import mutations from '~/releases/stores/modules/detail/mutations'; import * as types from '~/releases/stores/modules/detail/mutation_types'; @@ -27,7 +20,7 @@ describe('Release detail mutations', () => { release = convertObjectPropsToCamelCase(originalRelease); }); - describe(types.REQUEST_RELEASE, () => { + describe(`${types.REQUEST_RELEASE}`, () => { it('set state.isFetchingRelease to true', () => { mutations[types.REQUEST_RELEASE](state); @@ -35,7 +28,7 @@ describe('Release detail mutations', () => { }); }); - describe(types.RECEIVE_RELEASE_SUCCESS, () => { + describe(`${types.RECEIVE_RELEASE_SUCCESS}`, () => { it('handles a successful response from the server', () => { mutations[types.RECEIVE_RELEASE_SUCCESS](state, release); @@ -49,7 +42,7 @@ describe('Release detail mutations', () => { }); }); - describe(types.RECEIVE_RELEASE_ERROR, () => { + describe(`${types.RECEIVE_RELEASE_ERROR}`, () => { it('handles an unsuccessful response from the server', () => { const error = { message: 'An error occurred!' }; mutations[types.RECEIVE_RELEASE_ERROR](state, error); @@ -62,7 +55,7 @@ describe('Release detail mutations', () => { }); }); - describe(types.UPDATE_RELEASE_TITLE, () => { + describe(`${types.UPDATE_RELEASE_TITLE}`, () => { it("updates the release's title", () => { state.release = release; const newTitle = 'The new release title'; @@ -72,7 +65,7 @@ describe('Release detail mutations', () => { }); }); - describe(types.UPDATE_RELEASE_NOTES, () => { + describe(`${types.UPDATE_RELEASE_NOTES}`, () => { it("updates the release's notes", () => { state.release = release; const newNotes = 'The new release notes'; @@ -82,7 +75,7 @@ describe('Release detail mutations', () => { }); }); - describe(types.REQUEST_UPDATE_RELEASE, () => { + describe(`${types.REQUEST_UPDATE_RELEASE}`, () => { it('set state.isUpdatingRelease to true', () => { mutations[types.REQUEST_UPDATE_RELEASE](state); @@ -90,7 +83,7 @@ describe('Release detail mutations', () => { }); }); - describe(types.RECEIVE_UPDATE_RELEASE_SUCCESS, () => { + describe(`${types.RECEIVE_UPDATE_RELEASE_SUCCESS}`, () => { it('handles a successful response from the server', () => { mutations[types.RECEIVE_UPDATE_RELEASE_SUCCESS](state, release); @@ -100,7 +93,7 @@ describe('Release detail mutations', () => { }); }); - describe(types.RECEIVE_UPDATE_RELEASE_ERROR, () => { + describe(`${types.RECEIVE_UPDATE_RELEASE_ERROR}`, () => { it('handles an unsuccessful response from the server', () => { const error = { message: 'An error occurred!' }; mutations[types.RECEIVE_UPDATE_RELEASE_ERROR](state, error); @@ -111,7 +104,7 @@ describe('Release detail mutations', () => { }); }); - describe(types.ADD_EMPTY_ASSET_LINK, () => { + describe(`${types.ADD_EMPTY_ASSET_LINK}`, () => { it('adds a new, empty link object to the release', () => { state.release = release; @@ -130,7 +123,7 @@ describe('Release detail mutations', () => { }); }); - describe(types.UPDATE_ASSET_LINK_URL, () => { + describe(`${types.UPDATE_ASSET_LINK_URL}`, () => { it('updates an asset link with a new URL', () => { state.release = release; @@ -145,7 +138,7 @@ describe('Release detail mutations', () => { }); }); - describe(types.UPDATE_ASSET_LINK_NAME, () => { + describe(`${types.UPDATE_ASSET_LINK_NAME}`, () => { it('updates an asset link with a new name', () => { state.release = release; @@ -160,7 +153,7 @@ describe('Release detail mutations', () => { }); }); - describe(types.REMOVE_ASSET_LINK, () => { + describe(`${types.REMOVE_ASSET_LINK}`, () => { it('removes an asset link from the release', () => { state.release = release; |