Skip to content

Commit

Permalink
Merge pull request #13039 from isimluk/remove-consumption-admin
Browse files Browse the repository at this point in the history
Remove default consumption admin user
  • Loading branch information
gtanzillo authored Dec 8, 2016
2 parents a495372 + db5b999 commit ab987fc
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 6 deletions.
4 changes: 0 additions & 4 deletions db/fixtures/users.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,3 @@
:userid: admin
:password: smartvm
:group: EvmGroup-super_administrator
- :name: Consumption Administrator
:userid: consumption_admin
:password: smartvm
:group: EvmGroup-consumption_administrator
2 changes: 1 addition & 1 deletion spec/models/user_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -364,7 +364,7 @@
end

context ".seed" do
include_examples(".seed called multiple times", 2)
include_examples(".seed called multiple times", 1)

include_examples("seeding users with", [])

Expand Down
2 changes: 1 addition & 1 deletion spec/support/examples_group/shared_examples_for_seeding.rb
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
end

shared_examples_for "seeding users with" do |klasses|
let(:users) { {"admin" => "super_administrator", "consumption_admin" => "consumption_administrator"} }
let(:users) { {'admin' => 'super_administrator' } }

it "seeds users #{klasses.present? ? 'with' : ''} #{klasses.collect(&:to_s).join(', ')}" do
klasses.push(User)
Expand Down

0 comments on commit ab987fc

Please sign in to comment.