Commit c0257f10 authored by Filipa Lacerda's avatar Filipa Lacerda Committed by Kamil Trzciński

Removes loading spinner and props

It no longer makes a request to the API
there is no need to render a loading spinner
parent 062ef121
...@@ -50,8 +50,7 @@ export default { ...@@ -50,8 +50,7 @@ export default {
class="js-linked-pipeline-content linked-pipeline-content" class="js-linked-pipeline-content linked-pipeline-content"
@click="onClickLinkedPipeline" @click="onClickLinkedPipeline"
> >
<gl-loading-icon v-if="pipeline.isLoading" class="js-linked-pipeline-loading d-inline" /> <ci-status :status="pipelineStatus" class="js-linked-pipeline-status" />
<ci-status v-else :status="pipelineStatus" class="js-linked-pipeline-status" />
<span class="str-truncated align-bottom"> {{ projectName }} &#8226; #{{ pipeline.id }} </span> <span class="str-truncated align-bottom"> {{ projectName }} &#8226; #{{ pipeline.id }} </span>
</gl-button> </gl-button>
......
...@@ -37,7 +37,7 @@ export default { ...@@ -37,7 +37,7 @@ export default {
:key="pipeline.id" :key="pipeline.id"
:class="{ :class="{
'flat-connector-before': index === 0 && graphPosition === 'right', 'flat-connector-before': index === 0 && graphPosition === 'right',
active: pipeline.isExpanded || pipeline.isLoading, active: pipeline.isExpanded,
'left-connector': pipeline.isExpanded && graphPosition === 'left', 'left-connector': pipeline.isExpanded && graphPosition === 'left',
}" }"
:pipeline-id="pipeline.id" :pipeline-id="pipeline.id"
......
...@@ -13,31 +13,12 @@ describe('Linked pipeline', () => { ...@@ -13,31 +13,12 @@ describe('Linked pipeline', () => {
vm.$destroy(); vm.$destroy();
}); });
describe('while is loading', () => { describe('rendered output', () => {
const props = { const props = {
pipelineId: mockPipeline.id, pipelineId: mockPipeline.id,
pipelinePath: mockPipeline.path, pipelinePath: mockPipeline.path,
pipelineStatus: mockPipeline.details.status, pipelineStatus: mockPipeline.details.status,
projectName: mockPipeline.project.name, projectName: mockPipeline.project.name,
isLoading: true,
};
beforeEach(() => {
vm = mountComponent(Component, props);
});
it('renders loading icon', () => {
expect(vm.$el.querySelector('.js-linked-pipeline-loading')).not.toBeNull();
});
});
describe('when it is not loading', () => {
const props = {
pipelineId: mockPipeline.id,
pipelinePath: mockPipeline.path,
pipelineStatus: mockPipeline.details.status,
projectName: mockPipeline.project.name,
isLoading: false,
}; };
beforeEach(() => { beforeEach(() => {
......
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