Commit 95167417 authored by James Lopez's avatar James Lopez

Fix spec failures

parent aa2952ec
...@@ -9,7 +9,7 @@ module EE ...@@ -9,7 +9,7 @@ module EE
expose :items_per_page, as: :itemsPerPage expose :items_per_page, as: :itemsPerPage
expose :start_index, as: :startIndex expose :start_index, as: :startIndex
expose :resources, as: :Resources, using: ::EE::Gitlab::Scim::User #, if: ->(identity, _) { identity.present? } expose :resources, as: :Resources, using: ::EE::Gitlab::Scim::User
private private
......
...@@ -19,13 +19,13 @@ describe EE::Gitlab::Scim::ParamsParser do ...@@ -19,13 +19,13 @@ describe EE::Gitlab::Scim::ParamsParser do
it 'returns the correct operation attributes' do it 'returns the correct operation attributes' do
operations = [{ "op": "Replace", "path": "active", "value": "False" }] operations = [{ "op": "Replace", "path": "active", "value": "False" }]
expect(described_class.new(operations: operations).to_hash).to eq(active: false) expect(described_class.new(Operations: operations).to_hash).to eq(active: false)
end end
it 'returns an empty hash for the wrong operations' do it 'returns an empty hash for the wrong operations' do
operations = [{ "op": "Replace", "path": "test", "value": "False" }] operations = [{ "op": "Replace", "path": "test", "value": "False" }]
expect(described_class.new(operations: operations).to_hash).to eq({}) expect(described_class.new(Operations: operations).to_hash).to eq({})
end end
end end
...@@ -33,13 +33,13 @@ describe EE::Gitlab::Scim::ParamsParser do ...@@ -33,13 +33,13 @@ describe EE::Gitlab::Scim::ParamsParser do
it 'returns true when deprovisioning' do it 'returns true when deprovisioning' do
operations = [{ "op": "Replace", "path": "active", "value": "False" }] operations = [{ "op": "Replace", "path": "active", "value": "False" }]
expect(described_class.new(operations: operations).deprovision_user?).to be true expect(described_class.new(Operations: operations).deprovision_user?).to be true
end end
it 'returns false when not deprovisioning' do it 'returns false when not deprovisioning' do
operations = [{ "op": "Replace", "path": "active", "value": "True" }] operations = [{ "op": "Replace", "path": "active", "value": "True" }]
expect(described_class.new(operations: operations).deprovision_user?).to be false expect(described_class.new(Operations: operations).deprovision_user?).to be false
end end
end end
end end
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