Commit 10d8b3a0 authored by Filipa Lacerda's avatar Filipa Lacerda

Update tests with json endpoint

Cleans merge request css
parent 312928f5
...@@ -156,10 +156,6 @@ ...@@ -156,10 +156,6 @@
.dropdown-menu { .dropdown-menu {
z-index: 300; z-index: 300;
} }
.ci-action-icon-wrapper {
line-height: 16px;
}
} }
.mini-pipeline-graph-dropdown-toggle { .mini-pipeline-graph-dropdown-toggle {
......
...@@ -307,10 +307,10 @@ ActiveRecord::Schema.define(version: 20180418053107) do ...@@ -307,10 +307,10 @@ ActiveRecord::Schema.define(version: 20180418053107) do
t.integer "auto_canceled_by_id" t.integer "auto_canceled_by_id"
t.boolean "retried" t.boolean "retried"
t.integer "stage_id" t.integer "stage_id"
t.integer "artifacts_file_store"
t.integer "artifacts_metadata_store"
t.boolean "protected" t.boolean "protected"
t.integer "failure_reason" t.integer "failure_reason"
t.integer "artifacts_file_store"
t.integer "artifacts_metadata_store"
end end
add_index "ci_builds", ["artifacts_expire_at"], name: "index_ci_builds_on_artifacts_expire_at", where: "(artifacts_file <> ''::text)", using: :btree add_index "ci_builds", ["artifacts_expire_at"], name: "index_ci_builds_on_artifacts_expire_at", where: "(artifacts_file <> ''::text)", using: :btree
...@@ -357,13 +357,13 @@ ActiveRecord::Schema.define(version: 20180418053107) do ...@@ -357,13 +357,13 @@ ActiveRecord::Schema.define(version: 20180418053107) do
t.integer "project_id", null: false t.integer "project_id", null: false
t.integer "job_id", null: false t.integer "job_id", null: false
t.integer "file_type", null: false t.integer "file_type", null: false
t.integer "file_store"
t.integer "size", limit: 8 t.integer "size", limit: 8
t.datetime_with_timezone "created_at", null: false t.datetime_with_timezone "created_at", null: false
t.datetime_with_timezone "updated_at", null: false t.datetime_with_timezone "updated_at", null: false
t.datetime_with_timezone "expire_at" t.datetime_with_timezone "expire_at"
t.string "file" t.string "file"
t.binary "file_sha256" t.binary "file_sha256"
t.integer "file_store"
end end
add_index "ci_job_artifacts", ["expire_at", "job_id"], name: "index_ci_job_artifacts_on_expire_at_and_job_id", using: :btree add_index "ci_job_artifacts", ["expire_at", "job_id"], name: "index_ci_job_artifacts_on_expire_at_and_job_id", using: :btree
......
This diff is collapsed.
...@@ -4,6 +4,7 @@ import axios from '~/lib/utils/axios_utils'; ...@@ -4,6 +4,7 @@ import axios from '~/lib/utils/axios_utils';
import stage from '~/pipelines/components/stage.vue'; import stage from '~/pipelines/components/stage.vue';
import eventHub from '~/pipelines/event_hub'; import eventHub from '~/pipelines/event_hub';
import mountComponent from 'spec/helpers/vue_mount_component_helper'; import mountComponent from 'spec/helpers/vue_mount_component_helper';
import { stageReply } from './mock_data';
describe('Pipelines stage component', () => { describe('Pipelines stage component', () => {
let StageComponent; let StageComponent;
...@@ -41,7 +42,7 @@ describe('Pipelines stage component', () => { ...@@ -41,7 +42,7 @@ describe('Pipelines stage component', () => {
describe('with successfull request', () => { describe('with successfull request', () => {
beforeEach(() => { beforeEach(() => {
mock.onGet('path.json').reply(200, { html: 'foo' }); mock.onGet('path.json').reply(200, stageReply);
}); });
it('should render the received data and emit `clickedDropdown` event', done => { it('should render the received data and emit `clickedDropdown` event', done => {
...@@ -51,7 +52,7 @@ describe('Pipelines stage component', () => { ...@@ -51,7 +52,7 @@ describe('Pipelines stage component', () => {
setTimeout(() => { setTimeout(() => {
expect( expect(
component.$el.querySelector('.js-builds-dropdown-container ul').textContent.trim(), component.$el.querySelector('.js-builds-dropdown-container ul').textContent.trim(),
).toEqual('foo'); ).toContain(stageReply.latest_statuses[0].name);
expect(eventHub.$emit).toHaveBeenCalledWith('clickedDropdown'); expect(eventHub.$emit).toHaveBeenCalledWith('clickedDropdown');
done(); done();
}, 0); }, 0);
...@@ -74,7 +75,9 @@ describe('Pipelines stage component', () => { ...@@ -74,7 +75,9 @@ describe('Pipelines stage component', () => {
describe('update endpoint correctly', () => { describe('update endpoint correctly', () => {
beforeEach(() => { beforeEach(() => {
mock.onGet('bar.json').reply(200, { html: 'this is the updated content' }); const copyStage = Object.assign({}, stageReply);
copyStage.latest_statuses[0].name = 'this is the updated content';
mock.onGet('bar.json').reply(200, copyStage);
}); });
it('should update the stage to request the new endpoint provided', done => { it('should update the stage to request the new endpoint provided', done => {
...@@ -93,7 +96,7 @@ describe('Pipelines stage component', () => { ...@@ -93,7 +96,7 @@ describe('Pipelines stage component', () => {
setTimeout(() => { setTimeout(() => {
expect( expect(
component.$el.querySelector('.js-builds-dropdown-container ul').textContent.trim(), component.$el.querySelector('.js-builds-dropdown-container ul').textContent.trim(),
).toEqual('this is the updated content'); ).toContain('this is the updated content');
done(); done();
}); });
}); });
......
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