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
da5b5dc2
Commit
da5b5dc2
authored
Dec 06, 2018
by
Micaël Bergeron
Browse files
Options
Browse Files
Download
Email Patches
Plain Diff
Fix the spec failure
A manifest should be valid when there are no pseudonymized fields
parent
ccde2155
Changes
1
Show whitespace changes
Inline
Side-by-side
Showing
1 changed file
with
3 additions
and
3 deletions
+3
-3
ee/spec/lib/pseudonymizer/dumper_spec.rb
ee/spec/lib/pseudonymizer/dumper_spec.rb
+3
-3
No files found.
ee/spec/lib/pseudonymizer/dumper_spec.rb
View file @
da5b5dc2
...
...
@@ -76,7 +76,7 @@ describe Pseudonymizer::Dumper do
it
"all whitelisted attributes exist"
do
options
.
config
[
'tables'
].
each
do
|
table
,
table_def
|
whitelisted
=
table_def
[
'whitelist'
]
whitelisted
=
table_def
.
fetch
(
'whitelist'
,
[])
existing_columns
=
ActiveRecord
::
Base
.
connection
.
columns
(
table
.
to_sym
).
map
(
&
:name
)
diff
=
whitelisted
-
existing_columns
...
...
@@ -86,8 +86,8 @@ describe Pseudonymizer::Dumper do
it
"all pseudonymized attributes are whitelisted"
do
options
.
config
[
'tables'
].
each
do
|
table
,
table_def
|
whitelisted
=
table_def
[
'whitelist'
]
pseudonymized
=
table_def
[
'pseudo'
]
whitelisted
=
table_def
.
fetch
(
'whitelist'
,
[])
pseudonymized
=
table_def
.
fetch
(
'pseudo'
,
[])
diff
=
pseudonymized
-
whitelisted
expect
(
diff
).
to
be_empty
,
"
#{
table
}
should whitelist columns
#{
pseudonymized
.
inspect
}
: missing
#{
diff
.
inspect
}
"
...
...
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