Commit cf91ff1c authored by Rémy Coutable's avatar Rémy Coutable

Merge branch 'ce-to-ee-2018-06-11' into 'master'

CE upstream - 2018-06-11 09:22 UTC

Closes gitlab-qa#269

See merge request gitlab-org/gitlab-ee!6073
parents 534620a7 8646f517
...@@ -95,7 +95,7 @@ export default class ClusterStore { ...@@ -95,7 +95,7 @@ export default class ClusterStore {
this.state.applications.jupyter.hostname = this.state.applications.jupyter.hostname =
serverAppEntry.hostname || serverAppEntry.hostname ||
(this.state.applications.ingress.externalIp (this.state.applications.ingress.externalIp
? `jupyter.${this.state.applications.ingress.externalIp}.xip.io` ? `jupyter.${this.state.applications.ingress.externalIp}.nip.io`
: ''); : '');
} }
}); });
......
<script> <script>
import { parseSeconds, stringifyTime } from '../../../lib/utils/pretty_time'; import { parseSeconds, stringifyTime } from '../../../lib/utils/pretty_time';
import tooltip from '../../../vue_shared/directives/tooltip';
export default { export default {
name: 'TimeTrackingComparisonPane', name: 'TimeTrackingComparisonPane',
directives: {
tooltip,
},
props: { props: {
timeSpent: { timeSpent: {
type: Number, type: Number,
...@@ -51,17 +55,12 @@ export default { ...@@ -51,17 +55,12 @@ export default {
<div class="time-tracking-comparison-pane"> <div class="time-tracking-comparison-pane">
<div <div
class="compare-meter" class="compare-meter"
data-toggle="tooltip"
data-placement="top"
role="timeRemainingDisplay"
:aria-valuenow="timeRemainingTooltip"
:title="timeRemainingTooltip" :title="timeRemainingTooltip"
:data-original-title="timeRemainingTooltip" v-tooltip
:class="timeRemainingStatusClass" :class="timeRemainingStatusClass"
> >
<div <div
class="meter-container" class="meter-container"
role="timeSpentPercent"
:aria-valuenow="timeRemainingPercent" :aria-valuenow="timeRemainingPercent"
> >
<div <div
......
...@@ -6,6 +6,7 @@ module QA::Page ...@@ -6,6 +6,7 @@ module QA::Page
view 'app/views/shared/builds/_build_output.html.haml' do view 'app/views/shared/builds/_build_output.html.haml' do
element :build_output, '.js-build-output' element :build_output, '.js-build-output'
element :loading_animation, '.js-build-refresh'
end end
view 'app/assets/javascripts/vue_shared/components/ci_badge_link.vue' do view 'app/assets/javascripts/vue_shared/components/ci_badge_link.vue' do
...@@ -20,6 +21,10 @@ module QA::Page ...@@ -20,6 +21,10 @@ module QA::Page
find('.ci-status').text == PASSED_STATUS find('.ci-status').text == PASSED_STATUS
end end
def trace_loading?
has_css?('.js-build-refresh')
end
# Reminder: You may wish to wait for a particular job status before checking output # Reminder: You may wish to wait for a particular job status before checking output
def output def output
find('.js-build-output').text find('.js-build-output').text
......
...@@ -92,7 +92,9 @@ module QA ...@@ -92,7 +92,9 @@ module QA
Page::Project::Pipeline::Show.act { go_to_first_job } Page::Project::Pipeline::Show.act { go_to_first_job }
Page::Project::Job::Show.perform do |job| Page::Project::Job::Show.perform do |job|
job.wait(reload: false) { job.completed? } job.wait(reload: false) do
job.completed? && !job.trace_loading?
end
expect(job.passed?).to be_truthy, "Job status did not become \"passed\"." expect(job.passed?).to be_truthy, "Job status did not become \"passed\"."
expect(job.output).to include(sha1sum) expect(job.output).to include(sha1sum)
......
...@@ -110,7 +110,7 @@ describe('Clusters Store', () => { ...@@ -110,7 +110,7 @@ describe('Clusters Store', () => {
expect( expect(
store.state.applications.jupyter.hostname, store.state.applications.jupyter.hostname,
).toEqual(`jupyter.${store.state.applications.ingress.externalIp}.xip.io`); ).toEqual(`jupyter.${store.state.applications.ingress.externalIp}.nip.io`);
}); });
}); });
}); });
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