@@ -1,6 +1,6 @@
 FactoryGirl.define do
   factory :user do
-    client_slug { ENV['CLIENT_SLUG_DEFAULT'] || 'ncr' }
+    client_slug { ENV['CLIENT_SLUG_DEFAULT'] || nil }
     sequence(:email_address) {|n| "user#{n}@example.com" }
     sequence(:first_name) {|n| "FirstName#{n}" }
     sequence(:last_name) {|n| "LastName#{n}" }
FactoryGirl.define do
  factory :user do
    client_slug { ENV['CLIENT_SLUG_DEFAULT'] || 'ncr' }
    sequence(:email_address) {|n| "user#{n}@example.com" }
    sequence(:first_name) {|n| "FirstName#{n}" }
    sequence(:last_name) {|n| "LastName#{n}" }

    trait :active do
      active true
    end

    trait :inactive do
      active false
    end

    trait :admin do
      after(:create) do |user|
        user.roles << Role.find_or_create_by(name: 'admin')
      end
    end

    trait :with_delegate do
      after(:create) do |user|
        delegate = create(:user, client_slug: user.client_slug)
        user.add_delegate(delegate)
      end
    end
  end
end

maybe we should just set to `nil` rather than using env var? I haven't ever set it, so not sure what the use case for env var is.