Commit aca0d610 authored by Phil Hughes's avatar Phil Hughes

fixed mutations spec

parent cdc92d94
...@@ -47,7 +47,7 @@ export default { ...@@ -47,7 +47,7 @@ export default {
Pipeline Pipeline
</strong> </strong>
<a <a
:href="latestPipeline.details.status.details_path" :href="latestPipeline.path"
target="_blank" target="_blank"
> >
#{{ latestPipeline.id }} #{{ latestPipeline.id }}
......
...@@ -12,7 +12,14 @@ export default { ...@@ -12,7 +12,14 @@ export default {
state.isLoadingPipeline = false; state.isLoadingPipeline = false;
if (pipeline) { if (pipeline) {
state.latestPipeline = pipeline; state.latestPipeline = {
id: pipeline.id,
path: pipeline.path,
commit: pipeline.commit,
details: {
status: pipeline.details.status,
},
};
state.stages = pipeline.details.stages.map((stage, i) => { state.stages = pipeline.details.stages.map((stage, i) => {
const foundStage = state.stages.find(s => s.id === i); const foundStage = state.stages.find(s => s.id === i);
return { return {
......
...@@ -89,14 +89,16 @@ export const fullPipelinesResponse = { ...@@ -89,14 +89,16 @@ export const fullPipelinesResponse = {
pipelines: [ pipelines: [
{ {
id: '51', id: '51',
path: 'test',
commit: { commit: {
id: 'xxxxxxxxxxxxxxxxxxxx', id: '123',
}, },
details: { details: {
status: { status: {
icon: 'status_failed', icon: 'status_failed',
text: 'failed', text: 'failed',
}, },
stages: [...stages],
}, },
}, },
{ {
...@@ -109,6 +111,7 @@ export const fullPipelinesResponse = { ...@@ -109,6 +111,7 @@ export const fullPipelinesResponse = {
icon: 'status_passed', icon: 'status_passed',
text: 'passed', text: 'passed',
}, },
stages: [...stages],
}, },
}, },
], ],
......
import mutations from '~/ide/stores/modules/pipelines/mutations'; import mutations from '~/ide/stores/modules/pipelines/mutations';
import state from '~/ide/stores/modules/pipelines/state'; import state from '~/ide/stores/modules/pipelines/state';
import * as types from '~/ide/stores/modules/pipelines/mutation_types'; import * as types from '~/ide/stores/modules/pipelines/mutation_types';
import { pipelines, stages } from '../../../mock_data'; import { fullPipelinesResponse, stages } from '../../../mock_data';
describe('IDE pipelines mutations', () => { describe('IDE pipelines mutations', () => {
let mockedState; let mockedState;
...@@ -28,17 +28,25 @@ describe('IDE pipelines mutations', () => { ...@@ -28,17 +28,25 @@ describe('IDE pipelines mutations', () => {
describe(types.RECEIVE_LASTEST_PIPELINE_SUCCESS, () => { describe(types.RECEIVE_LASTEST_PIPELINE_SUCCESS, () => {
it('sets loading to false on success', () => { it('sets loading to false on success', () => {
mutations[types.RECEIVE_LASTEST_PIPELINE_SUCCESS](mockedState, pipelines[0]); mutations[types.RECEIVE_LASTEST_PIPELINE_SUCCESS](
mockedState,
fullPipelinesResponse.data.pipelines[0],
);
expect(mockedState.isLoadingPipeline).toBe(false); expect(mockedState.isLoadingPipeline).toBe(false);
}); });
it('sets latestPipeline', () => { it('sets latestPipeline', () => {
mutations[types.RECEIVE_LASTEST_PIPELINE_SUCCESS](mockedState, pipelines[0]); mutations[types.RECEIVE_LASTEST_PIPELINE_SUCCESS](
mockedState,
fullPipelinesResponse.data.pipelines[0],
);
expect(mockedState.latestPipeline).toEqual({ expect(mockedState.latestPipeline).toEqual({
id: pipelines[0].id, id: '51',
status: pipelines[0].status, path: 'test',
commit: { id: '123' },
details: { status: jasmine.any(Object) },
}); });
}); });
...@@ -47,33 +55,12 @@ describe('IDE pipelines mutations', () => { ...@@ -47,33 +55,12 @@ describe('IDE pipelines mutations', () => {
expect(mockedState.latestPipeline).toEqual(null); expect(mockedState.latestPipeline).toEqual(null);
}); });
});
describe(types.REQUEST_STAGES, () => {
it('sets stages loading to true', () => {
mutations[types.REQUEST_STAGES](mockedState);
expect(mockedState.isLoadingJobs).toBe(true);
});
});
describe(types.RECEIVE_STAGES_ERROR, () => {
it('sets jobs loading to false', () => {
mutations[types.RECEIVE_STAGES_ERROR](mockedState);
expect(mockedState.isLoadingJobs).toBe(false);
});
});
describe(types.RECEIVE_STAGES_SUCCESS, () => {
it('sets jobs loading to false on success', () => {
mutations[types.RECEIVE_STAGES_SUCCESS](mockedState, stages);
expect(mockedState.isLoadingJobs).toBe(false);
});
it('sets stages', () => { it('sets stages', () => {
mutations[types.RECEIVE_STAGES_SUCCESS](mockedState, stages); mutations[types.RECEIVE_LASTEST_PIPELINE_SUCCESS](
mockedState,
fullPipelinesResponse.data.pipelines[0],
);
expect(mockedState.stages.length).toBe(2); expect(mockedState.stages.length).toBe(2);
expect(mockedState.stages).toEqual([ expect(mockedState.stages).toEqual([
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment