Reduce differences in spec/support/matchers/access_matchers.rb

Signed-off-by: Rémy Coutable <remy@rymai.me>

spec/support/matchers/access_matchers.rb

@@ -7,29 +7,28 @@ module AccessMatchers
   extend RSpec::Matchers::DSL
   include Warden::Test::Helpers
 
-  def emulate_user(user, membership = nil)
+  def emulate_user(user_type_or_trait, membership = nil)
-    case user
+    case user_type_or_trait
-    when :user
+    when :user, :admin
-      login_as(create(:user))
+      login_as(create(user_type_or_trait))
+    when :external, :auditor
+      login_as(create(:user, user_type_or_trait))
     when :visitor
       logout
-    when :admin
-      login_as(create(:admin))
-    when :external
-      login_as(create(:user, external: true))
     when User
-      login_as(user)
+      login_as(user_type_or_trait)
     when *Gitlab::Access.sym_options_with_owner.keys
-      raise ArgumentError, "cannot emulate #{user} without membership parent" unless membership
+      raise ArgumentError, "cannot emulate #{user_type_or_trait} without membership parent" unless membership
 
-      role = user
+      role = user_type_or_trait
-
+      user =
-      if role == :owner && membership.owner
+        if role == :owner && membership.owner
-        user = membership.owner
+          membership.owner
-      else
+        else
-        user = create(:user)
+          create(:user).tap do |new_user|
-        membership.public_send(:"add_#{role}", user)
+            membership.public_send(:"add_#{role}", new_user)
-      end
+          end
+        end
 
       login_as(user)
     else