Commit efd5e533 authored by Filipa Lacerda's avatar Filipa Lacerda

Prevents rendering empty badge when pipeline request fails

parent 1b9c318c
<script> <script>
export default { export default {
name: 'PipelineNavigationTabs', name: 'PipelineNavigationTabs',
props: { props: {
scope: { scope: {
...@@ -18,6 +18,12 @@ export default { ...@@ -18,6 +18,12 @@ export default {
mounted() { mounted() {
$(document).trigger('init.scrolling-tabs'); $(document).trigger('init.scrolling-tabs');
}, },
methods: {
shouldRenderBadge(count) {
// 0 is valid in a badge, but evaluates to false, we need to check for undefined
return count !== undefined;
},
},
}; };
</script> </script>
<template> <template>
...@@ -27,7 +33,9 @@ export default { ...@@ -27,7 +33,9 @@ export default {
:class="{ active: scope === 'all'}"> :class="{ active: scope === 'all'}">
<a :href="paths.allPath"> <a :href="paths.allPath">
All All
<span class="badge js-totalbuilds-count"> <span
v-if="shouldRenderBadge(count.all)"
class="badge js-totalbuilds-count">
{{count.all}} {{count.all}}
</span> </span>
</a> </a>
...@@ -37,7 +45,9 @@ export default { ...@@ -37,7 +45,9 @@ export default {
:class="{ active: scope === 'pending'}"> :class="{ active: scope === 'pending'}">
<a :href="paths.pendingPath"> <a :href="paths.pendingPath">
Pending Pending
<span class="badge"> <span
v-if="shouldRenderBadge(count.pending)"
class="badge">
{{count.pending}} {{count.pending}}
</span> </span>
</a> </a>
...@@ -47,7 +57,9 @@ export default { ...@@ -47,7 +57,9 @@ export default {
:class="{ active: scope === 'running'}"> :class="{ active: scope === 'running'}">
<a :href="paths.runningPath"> <a :href="paths.runningPath">
Running Running
<span class="badge"> <span
v-if="shouldRenderBadge(count.running)"
class="badge">
{{count.running}} {{count.running}}
</span> </span>
</a> </a>
...@@ -57,7 +69,9 @@ export default { ...@@ -57,7 +69,9 @@ export default {
:class="{ active: scope === 'finished'}"> :class="{ active: scope === 'finished'}">
<a :href="paths.finishedPath"> <a :href="paths.finishedPath">
Finished Finished
<span class="badge"> <span
v-if="shouldRenderBadge(count.finished)"
class="badge">
{{count.finished}} {{count.finished}}
</span> </span>
</a> </a>
......
---
title: Prevents rendering empty badges when request fails
merge_request:
author:
type: fixed
export default (Component, props = {}) => new Component({
propsData: props,
}).$mount();
import Vue from 'vue';
import navigationTabs from '~/pipelines/components/navigation_tabs.vue';
import mountComponent from '../helpers/vue_mount_component_helper';
describe('navigation tabs pipeline component', () => {
let vm;
let Component;
let data;
beforeEach(() => {
data = {
scope: 'all',
count: {
all: 16,
running: 1,
pending: 10,
finished: 0,
},
paths: {
allPath: '/gitlab-org/gitlab-ce/pipelines',
pendingPath: '/gitlab-org/gitlab-ce/pipelines?scope=pending',
finishedPath: '/gitlab-org/gitlab-ce/pipelines?scope=finished',
runningPath: '/gitlab-org/gitlab-ce/pipelines?scope=running',
branchesPath: '/gitlab-org/gitlab-ce/pipelines?scope=branches',
tagsPath: '/gitlab-org/gitlab-ce/pipelines?scope=tags',
},
};
Component = Vue.extend(navigationTabs);
});
afterEach(() => {
vm.$destroy();
});
it('should render tabs with correct paths', () => {
vm = mountComponent(Component, data);
// All
const allTab = vm.$el.querySelector('.js-pipelines-tab-all a');
expect(allTab.textContent.trim()).toContain('All');
expect(allTab.getAttribute('href')).toEqual(data.paths.allPath);
// Pending
const pendingTab = vm.$el.querySelector('.js-pipelines-tab-pending a');
expect(pendingTab.textContent.trim()).toContain('Pending');
expect(pendingTab.getAttribute('href')).toEqual(data.paths.pendingPath);
// Running
const runningTab = vm.$el.querySelector('.js-pipelines-tab-running a');
expect(runningTab.textContent.trim()).toContain('Running');
expect(runningTab.getAttribute('href')).toEqual(data.paths.runningPath);
// Finished
const finishedTab = vm.$el.querySelector('.js-pipelines-tab-finished a');
expect(finishedTab.textContent.trim()).toContain('Finished');
expect(finishedTab.getAttribute('href')).toEqual(data.paths.finishedPath);
// Branches
const branchesTab = vm.$el.querySelector('.js-pipelines-tab-branches a');
expect(branchesTab.textContent.trim()).toContain('Branches');
// Tags
const tagsTab = vm.$el.querySelector('.js-pipelines-tab-tags a');
expect(tagsTab.textContent.trim()).toContain('Tags');
});
describe('scope', () => {
it('should render scope provided as active tab', () => {
vm = mountComponent(Component, data);
expect(vm.$el.querySelector('.js-pipelines-tab-all').className).toContain('active');
});
});
describe('badges', () => {
it('should render provided number', () => {
vm = mountComponent(Component, data);
// All
expect(
vm.$el.querySelector('.js-totalbuilds-count').textContent.trim(),
).toContain(data.count.all);
// Pending
expect(
vm.$el.querySelector('.js-pipelines-tab-pending .badge').textContent.trim(),
).toContain(data.count.pending);
// Running
expect(
vm.$el.querySelector('.js-pipelines-tab-running .badge').textContent.trim(),
).toContain(data.count.running);
// Finished
expect(
vm.$el.querySelector('.js-pipelines-tab-finished .badge').textContent.trim(),
).toContain(data.count.finished);
});
it('should not render badge when number is undefined', () => {
vm = mountComponent(Component, {
scope: 'all',
paths: {},
count: {},
});
// All
expect(
vm.$el.querySelector('.js-totalbuilds-count'),
).toEqual(null);
// Pending
expect(
vm.$el.querySelector('.js-pipelines-tab-pending .badge'),
).toEqual(null);
// Running
expect(
vm.$el.querySelector('.js-pipelines-tab-running .badge'),
).toEqual(null);
// Finished
expect(
vm.$el.querySelector('.js-pipelines-tab-finished .badge'),
).toEqual(null);
});
});
});
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