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
0
Merge Requests
0
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
Jérome Perrin
gitlab-ce
Commits
28a4e9d8
Commit
28a4e9d8
authored
Apr 06, 2017
by
Kushal Pandya
Committed by
Jacob Schatz
Apr 06, 2017
Browse files
Options
Browse Files
Download
Email Patches
Plain Diff
Show CI status as Favicon on Pipelines, Job and MR pages
parent
32db15b2
Changes
21
Show whitespace changes
Inline
Side-by-side
Showing
21 changed files
with
109 additions
and
3 deletions
+109
-3
app/assets/images/ci_favicons/icon_status_canceled.ico
app/assets/images/ci_favicons/icon_status_canceled.ico
+0
-0
app/assets/images/ci_favicons/icon_status_created.ico
app/assets/images/ci_favicons/icon_status_created.ico
+0
-0
app/assets/images/ci_favicons/icon_status_failed.ico
app/assets/images/ci_favicons/icon_status_failed.ico
+0
-0
app/assets/images/ci_favicons/icon_status_manual.ico
app/assets/images/ci_favicons/icon_status_manual.ico
+0
-0
app/assets/images/ci_favicons/icon_status_not_found.ico
app/assets/images/ci_favicons/icon_status_not_found.ico
+0
-0
app/assets/images/ci_favicons/icon_status_pending.ico
app/assets/images/ci_favicons/icon_status_pending.ico
+0
-0
app/assets/images/ci_favicons/icon_status_running.ico
app/assets/images/ci_favicons/icon_status_running.ico
+0
-0
app/assets/images/ci_favicons/icon_status_skipped.ico
app/assets/images/ci_favicons/icon_status_skipped.ico
+0
-0
app/assets/images/ci_favicons/icon_status_success.ico
app/assets/images/ci_favicons/icon_status_success.ico
+0
-0
app/assets/images/ci_favicons/icon_status_warning.ico
app/assets/images/ci_favicons/icon_status_warning.ico
+0
-0
app/assets/javascripts/build.js
app/assets/javascripts/build.js
+1
-0
app/assets/javascripts/dispatcher.js
app/assets/javascripts/dispatcher.js
+2
-0
app/assets/javascripts/lib/utils/common_utils.js
app/assets/javascripts/lib/utils/common_utils.js
+31
-0
app/assets/javascripts/merge_request_widget.js
app/assets/javascripts/merge_request_widget.js
+5
-2
app/assets/javascripts/pipelines.js
app/assets/javascripts/pipelines.js
+4
-0
app/views/layouts/_head.html.haml
app/views/layouts/_head.html.haml
+1
-1
app/views/projects/merge_requests/widget/_show.html.haml
app/views/projects/merge_requests/widget/_show.html.haml
+1
-0
changelogs/unreleased/12818-ci-status-as-favicon.yml
changelogs/unreleased/12818-ci-status-as-favicon.yml
+4
-0
spec/javascripts/build_spec.js
spec/javascripts/build_spec.js
+2
-0
spec/javascripts/lib/utils/common_utils_spec.js
spec/javascripts/lib/utils/common_utils_spec.js
+51
-0
spec/javascripts/merge_request_widget_spec.js
spec/javascripts/merge_request_widget_spec.js
+7
-0
No files found.
app/assets/images/ci_favicons/icon_status_canceled.ico
0 → 100755
View file @
28a4e9d8
5.3 KB
app/assets/images/ci_favicons/icon_status_created.ico
0 → 100755
View file @
28a4e9d8
5.3 KB
app/assets/images/ci_favicons/icon_status_failed.ico
0 → 100755
View file @
28a4e9d8
5.3 KB
app/assets/images/ci_favicons/icon_status_manual.ico
0 → 100755
View file @
28a4e9d8
5.3 KB
app/assets/images/ci_favicons/icon_status_not_found.ico
0 → 100755
View file @
28a4e9d8
5.3 KB
app/assets/images/ci_favicons/icon_status_pending.ico
0 → 100755
View file @
28a4e9d8
5.3 KB
app/assets/images/ci_favicons/icon_status_running.ico
0 → 100755
View file @
28a4e9d8
5.3 KB
app/assets/images/ci_favicons/icon_status_skipped.ico
0 → 100755
View file @
28a4e9d8
5.3 KB
app/assets/images/ci_favicons/icon_status_success.ico
0 → 100755
View file @
28a4e9d8
5.3 KB
app/assets/images/ci_favicons/icon_status_warning.ico
0 → 100755
View file @
28a4e9d8
5.3 KB
app/assets/javascripts/build.js
View file @
28a4e9d8
...
...
@@ -88,6 +88,7 @@ window.Build = (function() {
dataType
:
'
json
'
,
success
:
function
(
buildData
)
{
$
(
'
.js-build-output
'
).
html
(
buildData
.
trace_html
);
gl
.
utils
.
setCiStatusFavicon
(
`
${
this
.
pageUrl
}
/status.json`
);
if
(
window
.
location
.
hash
===
DOWN_BUILD_TRACE
)
{
$
(
"
html,body
"
).
scrollTop
(
this
.
$buildTrace
.
height
());
}
...
...
app/assets/javascripts/dispatcher.js
View file @
28a4e9d8
...
...
@@ -226,9 +226,11 @@ const ShortcutsBlob = require('./shortcuts_blob');
case
'
projects:pipelines:builds
'
:
case
'
projects:pipelines:show
'
:
const
{
controllerAction
}
=
document
.
querySelector
(
'
.js-pipeline-container
'
).
dataset
;
const
pipelineStatusUrl
=
`
${
document
.
querySelector
(
'
.js-pipeline-tab-link a
'
).
getAttribute
(
'
href
'
)}
/status.json`
;
new
gl
.
Pipelines
({
initTabs
:
true
,
pipelineStatusUrl
,
tabsOptions
:
{
action
:
controllerAction
,
defaultAction
:
'
pipelines
'
,
...
...
app/assets/javascripts/lib/utils/common_utils.js
View file @
28a4e9d8
...
...
@@ -2,6 +2,8 @@
(
function
()
{
(
function
(
w
)
{
var
base
;
const
faviconEl
=
document
.
getElementById
(
'
favicon
'
);
const
originalFavicon
=
faviconEl
?
faviconEl
.
getAttribute
(
'
href
'
)
:
null
;
w
.
gl
||
(
w
.
gl
=
{});
(
base
=
w
.
gl
).
utils
||
(
base
.
utils
=
{});
w
.
gl
.
utils
.
isInGroupsPage
=
function
()
{
...
...
@@ -361,5 +363,34 @@
fn
(
next
,
stop
);
});
};
w
.
gl
.
utils
.
setFavicon
=
(
iconName
)
=>
{
if
(
faviconEl
&&
iconName
)
{
faviconEl
.
setAttribute
(
'
href
'
,
`/assets/
${
iconName
}
.ico`
);
}
};
w
.
gl
.
utils
.
resetFavicon
=
()
=>
{
if
(
faviconEl
)
{
faviconEl
.
setAttribute
(
'
href
'
,
originalFavicon
);
}
};
w
.
gl
.
utils
.
setCiStatusFavicon
=
(
pageUrl
)
=>
{
$
.
ajax
({
url
:
pageUrl
,
dataType
:
'
json
'
,
success
:
function
(
data
)
{
if
(
data
&&
data
.
icon
)
{
gl
.
utils
.
setFavicon
(
`ci_favicons/
${
data
.
icon
}
`
);
}
else
{
gl
.
utils
.
resetFavicon
();
}
},
error
:
function
()
{
gl
.
utils
.
resetFavicon
();
}
});
};
})(
window
);
}).
call
(
window
);
app/assets/javascripts/merge_request_widget.js
View file @
28a4e9d8
...
...
@@ -41,8 +41,10 @@ import MiniPipelineGraph from './mini_pipeline_graph_dropdown';
// check_enable - Boolean, whether to check automerge status
// merge_check_url - String, URL to use to check automerge status
// ci_status_url - String, URL to use to check CI status
// pipeline_status_url - String, URL to use to get CI status for Favicon
//
this
.
opts
=
opts
;
this
.
opts
.
pipeline_status_url
=
`
${
this
.
opts
.
pipeline_status_url
}
.json`
;
this
.
$widgetBody
=
$
(
'
.mr-widget-body
'
);
$
(
'
#modal_merge_info
'
).
modal
({
show
:
false
...
...
@@ -159,6 +161,7 @@ import MiniPipelineGraph from './mini_pipeline_graph_dropdown';
_this
.
status
=
data
.
status
;
_this
.
hasCi
=
data
.
has_ci
;
_this
.
updateMergeButton
(
_this
.
status
,
_this
.
hasCi
);
gl
.
utils
.
setCiStatusFavicon
(
_this
.
opts
.
pipeline_status_url
);
if
(
data
.
environments
&&
data
.
environments
.
length
)
_this
.
renderEnvironments
(
data
.
environments
);
if
(
data
.
status
!==
_this
.
opts
.
ci_status
||
data
.
sha
!==
_this
.
opts
.
ci_sha
||
...
...
app/assets/javascripts/pipelines.js
View file @
28a4e9d8
...
...
@@ -9,6 +9,10 @@ require('./lib/utils/bootstrap_linked_tabs');
new
global
.
LinkedTabs
(
options
.
tabsOptions
);
}
if
(
options
.
pipelineStatusUrl
)
{
gl
.
utils
.
setCiStatusFavicon
(
options
.
pipelineStatusUrl
);
}
this
.
addMarginToBuildColumns
();
}
...
...
app/views/layouts/_head.html.haml
View file @
28a4e9d8
...
...
@@ -23,7 +23,7 @@
%title
=
page_title
(
site_name
)
%meta
{
name:
"description"
,
content:
page_description
}
=
favicon_link_tag
favicon
=
favicon_link_tag
favicon
,
id:
'favicon'
=
stylesheet_link_tag
"application"
,
media:
"all"
=
stylesheet_link_tag
"print"
,
media:
"print"
...
...
app/views/projects/merge_requests/widget/_show.html.haml
View file @
28a4e9d8
...
...
@@ -12,6 +12,7 @@
merge_check_url
:
"
#{
merge_check_namespace_project_merge_request_path
(
@project
.
namespace
,
@project
,
@merge_request
)
}
"
,
check_enable
:
#{
@merge_request
.
unchecked?
?
"true"
:
"false"
}
,
ci_status_url
:
"
#{
ci_status_namespace_project_merge_request_path
(
@project
.
namespace
,
@project
,
@merge_request
)
}
"
,
pipeline_status_url
:
"
#{
pipeline_status_namespace_project_merge_request_path
(
@project
.
namespace
,
@project
,
@merge_request
)
}
"
,
ci_environments_status_url
:
"
#{
ci_environments_status_namespace_project_merge_request_path
(
@project
.
namespace
,
@project
,
@merge_request
)
}
"
,
gitlab_icon
:
"
#{
asset_path
'gitlab_logo.png'
}
"
,
ci_status
:
"
#{
@merge_request
.
head_pipeline
?
@merge_request
.
head_pipeline
.
status
:
''
}
"
,
...
...
changelogs/unreleased/12818-ci-status-as-favicon.yml
0 → 100644
View file @
28a4e9d8
---
title
:
Show CI status as Favicon on Pipelines, Job and MR pages
merge_request
:
10144
author
:
spec/javascripts/build_spec.js
View file @
28a4e9d8
...
...
@@ -75,6 +75,7 @@ describe('Build', () => {
expect
(
url
).
toBe
(
`
${
BUILD_URL
}
.json`
);
expect
(
dataType
).
toBe
(
'
json
'
);
expect
(
success
).
toEqual
(
jasmine
.
any
(
Function
));
spyOn
(
gl
.
utils
,
'
setCiStatusFavicon
'
).
and
.
callFake
(()
=>
{});
success
.
call
(
context
,
{
trace_html
:
'
<span>Example</span>
'
,
status
:
'
running
'
});
...
...
@@ -83,6 +84,7 @@ describe('Build', () => {
it
(
'
removes the spinner
'
,
()
=>
{
const
[{
success
,
context
}]
=
$
.
ajax
.
calls
.
argsFor
(
0
);
spyOn
(
gl
.
utils
,
'
setCiStatusFavicon
'
).
and
.
callFake
(()
=>
{});
success
.
call
(
context
,
{
trace_html
:
'
<span>Example</span>
'
,
status
:
'
success
'
});
expect
(
$
(
'
.js-build-refresh
'
).
length
).
toBe
(
0
);
...
...
spec/javascripts/lib/utils/common_utils_spec.js
View file @
28a4e9d8
...
...
@@ -310,5 +310,56 @@ require('~/lib/utils/common_utils');
});
},
10000
);
});
describe
(
'
gl.utils.setFavicon
'
,
()
=>
{
it
(
'
should set page favicon to provided favicon
'
,
()
=>
{
const
faviconName
=
'
custom_favicon
'
;
const
fakeLink
=
{
setAttribute
()
{},
};
spyOn
(
window
.
document
,
'
getElementById
'
).
and
.
callFake
(()
=>
fakeLink
);
spyOn
(
fakeLink
,
'
setAttribute
'
).
and
.
callFake
((
attr
,
val
)
=>
{
expect
(
attr
).
toEqual
(
'
href
'
);
expect
(
val
.
indexOf
(
'
/assets/custom_favicon.ico
'
)
>
-
1
).
toBe
(
true
);
});
gl
.
utils
.
setFavicon
(
faviconName
);
});
});
describe
(
'
gl.utils.resetFavicon
'
,
()
=>
{
it
(
'
should reset page favicon to tanuki
'
,
()
=>
{
const
fakeLink
=
{
setAttribute
()
{},
};
spyOn
(
window
.
document
,
'
getElementById
'
).
and
.
callFake
(()
=>
fakeLink
);
spyOn
(
fakeLink
,
'
setAttribute
'
).
and
.
callFake
((
attr
,
val
)
=>
{
expect
(
attr
).
toEqual
(
'
href
'
);
expect
(
val
).
toMatch
(
/favicon/
);
});
gl
.
utils
.
resetFavicon
();
});
});
describe
(
'
gl.utils.setCiStatusFavicon
'
,
()
=>
{
it
(
'
should set page favicon to CI status favicon based on provided status
'
,
()
=>
{
const
BUILD_URL
=
`
${
gl
.
TEST_HOST
}
/frontend-fixtures/builds-project/builds/1/status.json`
;
const
FAVICON_PATH
=
'
ci_favicons/
'
;
const
FAVICON
=
'
icon_status_success
'
;
const
spySetFavicon
=
spyOn
(
gl
.
utils
,
'
setFavicon
'
).
and
.
stub
();
const
spyResetFavicon
=
spyOn
(
gl
.
utils
,
'
resetFavicon
'
).
and
.
stub
();
spyOn
(
$
,
'
ajax
'
).
and
.
callFake
(
function
(
options
)
{
options
.
success
({
icon
:
FAVICON
});
expect
(
spySetFavicon
).
toHaveBeenCalledWith
(
FAVICON_PATH
+
FAVICON
);
options
.
success
();
expect
(
spyResetFavicon
).
toHaveBeenCalled
();
options
.
error
();
expect
(
spyResetFavicon
).
toHaveBeenCalled
();
});
gl
.
utils
.
setCiStatusFavicon
(
BUILD_URL
);
});
});
});
})();
spec/javascripts/merge_request_widget_spec.js
View file @
28a4e9d8
...
...
@@ -142,18 +142,21 @@ require('~/lib/utils/datetime_utility');
it
(
'
should call showCIStatus even if a notification should not be displayed
'
,
function
()
{
var
spy
;
spy
=
spyOn
(
this
[
"
class
"
],
'
showCIStatus
'
).
and
.
stub
();
spyOn
(
gl
.
utils
,
'
setCiStatusFavicon
'
).
and
.
callFake
(()
=>
{});
this
[
"
class
"
].
getCIStatus
(
false
);
return
expect
(
spy
).
toHaveBeenCalledWith
(
this
.
ciStatusData
.
status
);
});
it
(
'
should call showCIStatus when a notification should be displayed
'
,
function
()
{
var
spy
;
spy
=
spyOn
(
this
[
"
class
"
],
'
showCIStatus
'
).
and
.
stub
();
spyOn
(
gl
.
utils
,
'
setCiStatusFavicon
'
).
and
.
callFake
(()
=>
{});
this
[
"
class
"
].
getCIStatus
(
true
);
return
expect
(
spy
).
toHaveBeenCalledWith
(
this
.
ciStatusData
.
status
);
});
it
(
'
should call showCICoverage when the coverage rate is set
'
,
function
()
{
var
spy
;
spy
=
spyOn
(
this
[
"
class
"
],
'
showCICoverage
'
).
and
.
stub
();
spyOn
(
gl
.
utils
,
'
setCiStatusFavicon
'
).
and
.
callFake
(()
=>
{});
this
[
"
class
"
].
getCIStatus
(
false
);
return
expect
(
spy
).
toHaveBeenCalledWith
(
this
.
ciStatusData
.
coverage
);
});
...
...
@@ -161,12 +164,14 @@ require('~/lib/utils/datetime_utility');
var
spy
;
this
.
ciStatusData
.
coverage
=
null
;
spy
=
spyOn
(
this
[
"
class
"
],
'
showCICoverage
'
).
and
.
stub
();
spyOn
(
gl
.
utils
,
'
setCiStatusFavicon
'
).
and
.
callFake
(()
=>
{});
this
[
"
class
"
].
getCIStatus
(
false
);
return
expect
(
spy
).
not
.
toHaveBeenCalled
();
});
it
(
'
should not display a notification on the first check after the widget has been created
'
,
function
()
{
var
spy
;
spy
=
spyOn
(
window
,
'
notify
'
);
spyOn
(
gl
.
utils
,
'
setCiStatusFavicon
'
).
and
.
callFake
(()
=>
{});
this
[
"
class
"
]
=
new
window
.
gl
.
MergeRequestWidget
(
this
.
opts
);
this
[
"
class
"
].
getCIStatus
(
true
);
return
expect
(
spy
).
not
.
toHaveBeenCalled
();
...
...
@@ -174,6 +179,7 @@ require('~/lib/utils/datetime_utility');
it
(
'
should update the pipeline URL when the pipeline changes
'
,
function
()
{
var
spy
;
spy
=
spyOn
(
this
[
"
class
"
],
'
updatePipelineUrls
'
).
and
.
stub
();
spyOn
(
gl
.
utils
,
'
setCiStatusFavicon
'
).
and
.
callFake
(()
=>
{});
this
[
"
class
"
].
getCIStatus
(
false
);
this
.
ciStatusData
.
pipeline
+=
1
;
this
[
"
class
"
].
getCIStatus
(
false
);
...
...
@@ -182,6 +188,7 @@ require('~/lib/utils/datetime_utility');
it
(
'
should update the commit URL when the sha changes
'
,
function
()
{
var
spy
;
spy
=
spyOn
(
this
[
"
class
"
],
'
updateCommitUrls
'
).
and
.
stub
();
spyOn
(
gl
.
utils
,
'
setCiStatusFavicon
'
).
and
.
callFake
(()
=>
{});
this
[
"
class
"
].
getCIStatus
(
false
);
this
.
ciStatusData
.
sha
=
"
9b50b99a
"
;
this
[
"
class
"
].
getCIStatus
(
false
);
...
...
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