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
3d1cf9e3
Commit
3d1cf9e3
authored
Aug 21, 2020
by
Mehmet Emin INAC
Browse files
Options
Browse Files
Download
Email Patches
Plain Diff
Implement method to get `missing_security_scan_types` for an MR
parent
1f176abf
Changes
2
Show whitespace changes
Inline
Side-by-side
Showing
2 changed files
with
76 additions
and
0 deletions
+76
-0
ee/app/models/ee/merge_request.rb
ee/app/models/ee/merge_request.rb
+6
-0
ee/spec/models/merge_request_spec.rb
ee/spec/models/merge_request_spec.rb
+70
-0
No files found.
ee/app/models/ee/merge_request.rb
View file @
3d1cf9e3
...
...
@@ -257,6 +257,12 @@ module EE
end
end
def
missing_security_scan_types
return
[]
unless
actual_head_pipeline
&&
base_pipeline
base_pipeline
.
security_scans
.
pluck
(
:scan_type
)
-
actual_head_pipeline
.
security_scans
.
pluck
(
:scan_type
)
end
private
def
has_approved_license_check?
...
...
ee/spec/models/merge_request_spec.rb
View file @
3d1cf9e3
...
...
@@ -880,4 +880,74 @@ RSpec.describe MergeRequest do
expect
(
described_class
.
sort_by_attribute
(
'review_time_desc'
)).
to
match
([
mr3
,
mr4
,
mr2
,
mr1
,
mr5
])
end
end
describe
'#missing_security_scan_types'
do
let_it_be
(
:project
)
{
create
(
:project
,
:repository
)
}
let_it_be
(
:merge_request
)
{
create
(
:ee_merge_request
,
source_project:
project
)
}
subject
{
merge_request
.
missing_security_scan_types
}
context
'when there is no head pipeline'
do
context
'when there is no base pipeline'
do
it
{
is_expected
.
to
be_empty
}
end
context
'when there is a base pipeline'
do
let_it_be
(
:base_pipeline
)
do
create
(
:ee_ci_pipeline
,
project:
project
,
ref:
merge_request
.
target_branch
,
sha:
merge_request
.
diff_base_sha
)
end
context
'when there is no security scan for the base pipeline'
do
it
{
is_expected
.
to
be_empty
}
end
context
'when there are security scans for the base_pipeline'
do
before
do
build
=
create
(
:ci_build
,
:success
,
pipeline:
base_pipeline
,
project:
project
)
create
(
:security_scan
,
build:
build
)
end
it
{
is_expected
.
to
be_empty
}
end
end
end
context
'when there is a head pipeline'
do
before
do
create
(
:ee_ci_pipeline
,
project:
project
,
sha:
merge_request
.
diff_head_sha
)
merge_request
.
update_head_pipeline
end
context
'when there is no base pipeline'
do
it
{
is_expected
.
to
be_empty
}
end
context
'when there is a base pipeline'
do
let_it_be
(
:base_pipeline
)
do
create
(
:ee_ci_pipeline
,
project:
project
,
ref:
merge_request
.
target_branch
,
sha:
merge_request
.
diff_base_sha
)
end
context
'when there are missing security scans for the head pipeline'
do
let
(
:missing_scan_type
)
{
'sast'
}
before
do
build
=
create
(
:ci_build
,
:success
,
pipeline:
base_pipeline
,
project:
project
)
create
(
:security_scan
,
build:
build
,
scan_type:
missing_scan_type
)
end
it
{
is_expected
.
to
eq
([
missing_scan_type
])
}
end
context
'when there is no missing security scan for the head pipeline'
do
it
{
is_expected
.
to
be_empty
}
end
end
end
end
end
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