Skip to content

Commit

Permalink
Only provide default values for required parameters
Browse files Browse the repository at this point in the history
The test suite already provide optional parameter values for each test
that rely on them, so we do not need to pass them to all tests.

Moreover, some on these changed default values prevent the test suite
from failing because of a regression introduced when refactoring maps.

Remove the default values to trigger the test suite breakage and allow
checking that a fix is working as expected.
  • Loading branch information
smortex committed Sep 10, 2024
1 parent 12d6b62 commit 1deae36
Showing 1 changed file with 0 additions and 6 deletions.
6 changes: 0 additions & 6 deletions spec/defines/resource_map_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -15,12 +15,6 @@
let :default_params do
{
string: '$uri',
default: 'pool_a',
mappings: {
'foo' => 'pool_b',
'bar' => 'pool_c',
'baz' => 'pool_d'
}
}
end

Expand Down

0 comments on commit 1deae36

Please sign in to comment.