Merge branch '50904-use-vuex-store-job' into 'master'

Uses new Vuex store in job log page

See merge request gitlab-org/gitlab-ce!21961

3 files changed, 7 insertions, 65 deletions

diff --git a/spec/javascripts/jobs/components/job_details_mediator_spec.js b/spec/javascripts/jobs/components/job_details_mediator_spec.js
deleted file mode 100644
index 3e2fb7bfbbb..00000000000
--- a/spec/javascripts/jobs/components/job_details_mediator_spec.js
+++ /dev/null
@@ -1,39 +0,0 @@
-import MockAdapter from 'axios-mock-adapter';
-import axios from '~/lib/utils/axios_utils';
-import JobMediator from '~/jobs/job_details_mediator';
-import job from '../mock_data';
-
-describe('JobMediator', () => {
-  let mediator;
-  let mock;
-
-  beforeEach(() => {
-    mediator = new JobMediator({ endpoint: 'jobs/40291672.json' });
-    mock = new MockAdapter(axios);
-  });
-
-  afterEach(() => {
-    mock.restore();
-  });
-
-  it('should set defaults', () => {
-    expect(mediator.store).toBeDefined();
-    expect(mediator.service).toBeDefined();
-    expect(mediator.options).toEqual({ endpoint: 'jobs/40291672.json' });
-    expect(mediator.state.isLoading).toEqual(false);
-  });
-
-  describe('request and store data', () => {
-    beforeEach(() => {
-      mock.onGet().reply(200, job, {});
-    });
-
-    it('should store received data', (done) => {
-      mediator.fetchJob();
-      setTimeout(() => {
-        expect(mediator.store.state.job).toEqual(job);
-        done();
-      }, 0);
-    });
-  });
-});
diff --git a/spec/javascripts/jobs/components/job_store_spec.js b/spec/javascripts/jobs/components/job_store_spec.js
deleted file mode 100644
index 0dad5111b32..00000000000
--- a/spec/javascripts/jobs/components/job_store_spec.js
+++ /dev/null
@@ -1,26 +0,0 @@
-import JobStore from '~/jobs/stores/job_store';
-import job from '../mock_data';
-
-describe('Job Store', () => {
-  let store;
-
-  beforeEach(() => {
-    store = new JobStore();
-  });
-
-  it('should set defaults', () => {
-    expect(store.state.job).toEqual({});
-  });
-
-  describe('storeJob', () => {
-    it('should store empty object if none is provided', () => {
-      store.storeJob();
-      expect(store.state.job).toEqual({});
-    });
-
-    it('should store provided argument', () => {
-      store.storeJob(job);
-      expect(store.state.job).toEqual(job);
-    });
-  });
-});
diff --git a/spec/javascripts/jobs/store/mutations_spec.js b/spec/javascripts/jobs/store/mutations_spec.js
index 6900b2e5602..9ba543d32a8 100644
--- a/spec/javascripts/jobs/store/mutations_spec.js
+++ b/spec/javascripts/jobs/store/mutations_spec.js
@@ -12,6 +12,13 @@ describe('Jobs Store Mutations', () => {
     stateCopy = state();
   });
 
+  describe('SET_JOB_ENDPOINT', () => {
+    it('should set jobEndpoint', () => {
+      mutations[types.SET_JOB_ENDPOINT](stateCopy, 'job/21312321.json');
+      expect(stateCopy.jobEndpoint).toEqual('job/21312321.json');
+    });
+  });
+
   describe('REQUEST_STATUS_FAVICON', () => {
     it('should set fetchingStatusFavicon to true', () => {
       mutations[types.REQUEST_STATUS_FAVICON](stateCopy);