Skip to content
Projects
Groups
Snippets
Help
Loading...
Help
Support
Keyboard shortcuts
?
Submit feedback
Contribute to GitLab
Sign in / Register
Toggle navigation
G
gitlab-ce
Project overview
Project overview
Details
Activity
Releases
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Issues
0
Issues
0
List
Boards
Labels
Milestones
Merge Requests
1
Merge Requests
1
Analytics
Analytics
Repository
Value Stream
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Create a new issue
Commits
Issue Boards
Open sidebar
nexedi
gitlab-ce
Commits
67209d3a
Commit
67209d3a
authored
Apr 09, 2018
by
Filipa Lacerda
Committed by
Phil Hughes
Apr 09, 2018
Browse files
Options
Browse Files
Download
Email Patches
Plain Diff
Resolve "Cannot read property 'commit_path' of null"
parent
d391a218
Changes
3
Hide whitespace changes
Inline
Side-by-side
Showing
3 changed files
with
76 additions
and
20 deletions
+76
-20
app/assets/javascripts/vue_merge_request_widget/components/mr_widget_pipeline.vue
...ue_merge_request_widget/components/mr_widget_pipeline.vue
+30
-20
changelogs/unreleased/44296-commit-path.yml
changelogs/unreleased/44296-commit-path.yml
+6
-0
spec/javascripts/vue_mr_widget/components/mr_widget_pipeline_spec.js
...ripts/vue_mr_widget/components/mr_widget_pipeline_spec.js
+40
-0
No files found.
app/assets/javascripts/vue_merge_request_widget/components/mr_widget_pipeline.vue
View file @
67209d3a
<
script
>
/* eslint-disable vue/require-default-prop */
import
p
ipelineStage
from
'
~/pipelines/components/stage.vue
'
;
import
c
iIcon
from
'
~/vue_shared/components/ci_icon.vue
'
;
import
i
con
from
'
~/vue_shared/components/icon.vue
'
;
import
l
inkedPipelinesMiniList
from
'
ee/vue_shared/components/linked_pipelines_mini_list.vue
'
;
import
P
ipelineStage
from
'
~/pipelines/components/stage.vue
'
;
import
C
iIcon
from
'
~/vue_shared/components/ci_icon.vue
'
;
import
I
con
from
'
~/vue_shared/components/icon.vue
'
;
import
L
inkedPipelinesMiniList
from
'
ee/vue_shared/components/linked_pipelines_mini_list.vue
'
;
export
default
{
name
:
'
MRWidgetPipeline
'
,
components
:
{
p
ipelineStage
,
c
iIcon
,
i
con
,
l
inkedPipelinesMiniList
,
P
ipelineStage
,
C
iIcon
,
I
con
,
L
inkedPipelinesMiniList
,
},
props
:
{
pipeline
:
{
...
...
@@ -37,15 +37,20 @@
return
this
.
hasCi
&&
!
this
.
ciStatus
;
},
status
()
{
return
this
.
pipeline
.
details
&&
this
.
pipeline
.
details
.
status
?
this
.
pipeline
.
details
.
status
:
{};
return
this
.
pipeline
.
details
&&
this
.
pipeline
.
details
.
status
?
this
.
pipeline
.
details
.
status
:
{};
},
hasStages
()
{
return
this
.
pipeline
.
details
&&
return
(
this
.
pipeline
.
details
&&
this
.
pipeline
.
details
.
stages
&&
this
.
pipeline
.
details
.
stages
.
length
;
this
.
pipeline
.
details
.
stages
.
length
);
},
hasCommitInfo
()
{
return
this
.
pipeline
.
commit
&&
Object
.
keys
(
this
.
pipeline
.
commit
).
length
>
0
;
},
/* We typically set defaults ([]) in the store or prop declarations, but because triggered
* and triggeredBy are appended to `pipeline`, we can't set defaults in the store, and we
* need to check their length here to prevent initializing linked-pipeline-mini-lists
...
...
@@ -63,7 +68,8 @@
<
template
>
<div
v-if=
"hasPipeline || hasCIError"
class=
"mr-widget-heading"
>
class=
"mr-widget-heading"
>
<div
class=
"ci-widget media"
>
<template
v-if=
"hasCIError"
>
<div
class=
"ci-status-icon ci-status-icon-failed ci-error js-ci-error append-right-10"
>
...
...
@@ -90,13 +96,17 @@
#
{{
pipeline
.
id
}}
</a>
{{
pipeline
.
details
.
status
.
label
}}
for
{{
pipeline
.
details
.
status
.
label
}}
<a
:href=
"pipeline.commit.commit_path"
class=
"commit-sha js-commit-link"
>
{{
pipeline
.
commit
.
short_id
}}
</a>
.
<template
v-if=
"hasCommitInfo"
>
for
<a
:href=
"pipeline.commit.commit_path"
class=
"commit-sha js-commit-link"
>
{{
pipeline
.
commit
.
short_id
}}
</a>
.
</
template
>
<span
class=
"mr-widget-pipeline-graph"
>
<span
class=
"stage-cell"
>
...
...
changelogs/unreleased/44296-commit-path.yml
0 → 100644
View file @
67209d3a
---
title
:
Verifiy if pipeline has commit idetails and render information in MR widget
when branch is deleted
merge_request
:
author
:
type
:
fixed
spec/javascripts/vue_mr_widget/components/mr_widget_pipeline_spec.js
View file @
67209d3a
...
...
@@ -114,6 +114,46 @@ describe('MRWidgetPipeline', () => {
});
});
describe
(
'
without commit path
'
,
()
=>
{
beforeEach
(()
=>
{
const
mockCopy
=
Object
.
assign
({},
mockData
);
delete
mockCopy
.
pipeline
.
commit
;
vm
=
mountComponent
(
Component
,
{
pipeline
:
mockCopy
.
pipeline
,
hasCi
:
true
,
ciStatus
:
'
success
'
,
});
});
it
(
'
should render pipeline ID
'
,
()
=>
{
expect
(
vm
.
$el
.
querySelector
(
'
.pipeline-id
'
).
textContent
.
trim
(),
).
toEqual
(
`#
${
mockData
.
pipeline
.
id
}
`
);
});
it
(
'
should render pipeline status
'
,
()
=>
{
expect
(
vm
.
$el
.
querySelector
(
'
.media-body
'
).
textContent
.
trim
(),
).
toContain
(
mockData
.
pipeline
.
details
.
status
.
label
);
expect
(
vm
.
$el
.
querySelector
(
'
.js-commit-link
'
),
).
toBeNull
();
});
it
(
'
should render pipeline graph
'
,
()
=>
{
expect
(
vm
.
$el
.
querySelector
(
'
.mr-widget-pipeline-graph
'
)).
toBeDefined
();
expect
(
vm
.
$el
.
querySelectorAll
(
'
.stage-container
'
).
length
).
toEqual
(
mockData
.
pipeline
.
details
.
stages
.
length
);
});
it
(
'
should render coverage information
'
,
()
=>
{
expect
(
vm
.
$el
.
querySelector
(
'
.media-body
'
).
textContent
,
).
toContain
(
`Coverage
${
mockData
.
pipeline
.
coverage
}
`
);
});
});
describe
(
'
without coverage
'
,
()
=>
{
it
(
'
should not render a coverage
'
,
()
=>
{
const
mockCopy
=
Object
.
assign
({},
mockData
);
...
...
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment