Merge remote-tracking branch 'upstream/pull/4548'

[rails.git] / test / factories / user.rb

diff --git a/test/factories/user.rb b/test/factories/user.rb
index d6ee1fa8c03249f9f4216cde09b890c921936bc0..5b92345c2dbcf0f356be857b7432018dec6abef5 100644 (file)
--- a/test/factories/user.rb
+++ b/test/factories/user.rb
@@ -2,13 +2,17 @@ FactoryBot.define do
   factory :user do
     sequence(:email) { |n| "user#{n}@example.com" }
     sequence(:display_name) { |n| "User #{n}" }
   factory :user do
     sequence(:email) { |n| "user#{n}@example.com" }
     sequence(:display_name) { |n| "User #{n}" }

-    pass_crypt Digest::MD5.hexdigest("test")
+    pass_crypt { PasswordHash.create("test").first }

 
     # These attributes are not the defaults, but in most tests we want
     # a 'normal' user who can log in without being redirected etc.
 
     # These attributes are not the defaults, but in most tests we want
     # a 'normal' user who can log in without being redirected etc.

-    status "active"
-    terms_seen true
-    data_public true
+    after(:build) do |user, _evaluator|
+      user.activate
+    end
+
+    terms_seen { true }
+    terms_agreed { Time.now.utc }
+    data_public { true }

 
     trait :with_home_location do
       home_lat { rand(-90.0...90.0) }
 
     trait :with_home_location do
       home_lat { rand(-90.0...90.0) }


@@ -16,23 +20,37 @@ FactoryBot.define do
     end
 
     trait :pending do
     end
 
     trait :pending do

-      status "pending"
+      after(:build) do |user, _evaluator|
+        user.deactivate
+      end

     end
 
     trait :active do
     end
 
     trait :active do

-      status "active"
+      # status { "active" }

     end
 
     trait :confirmed do
     end
 
     trait :confirmed do

-      status "confirmed"
+      after(:build) do |user, _evaluator|
+        user.confirm
+      end

     end
 
     trait :suspended do
     end
 
     trait :suspended do

-      status "suspended"
+      after(:build) do |user, _evaluator|
+        user.suspend
+      end

     end
 
     trait :deleted do
     end
 
     trait :deleted do

-      status "deleted"
+      after(:build) do |user, _evaluator|
+        user.soft_destroy
+      end
+    end
+
+    factory :importer_user do
+      after(:create) do |user, _evaluator|
+        create(:user_role, :role => "importer", :user => user)
+      end

     end
 
     factory :moderator_user do
     end
 
     factory :moderator_user do