Mark migrated users as verified (#4896)

app/controllers/api/v1/migrations/external_controller.rb

@@ -88,7 +88,7 @@ module Api
 
           user_hash[:password] = generate_secure_pwd if user_hash[:external_id].blank?
 
-          user = User.new(user_hash.merge(provider: 'greenlight', role:))
+          user = User.new(user_hash.merge(verified: true, provider: 'greenlight', role:))
 
           return render_error(status: :bad_request, errors: user&.errors&.to_a) unless user.save
 

spec/controllers/migrations/external_controller_spec.rb

@@ -235,6 +235,17 @@ RSpec.describe Api::V1::Migrations::ExternalController, type: :controller do
             expect(user.authenticate('Password1!')).to be_falsy
           end
         end
+
+        it 'sets user to verified' do
+          encrypted_params = encrypt_params({ user: valid_user_params }, expires_in: 10.seconds)
+
+          expect_any_instance_of(described_class).to receive(:generate_secure_pwd).and_call_original
+          expect { post :create_user, params: { v2: { encrypted_params: } } }.to change(User, :count).from(0).to(1)
+          expect(ActionMailer::MailDeliveryJob).not_to have_been_enqueued
+
+          user = User.take
+          expect(user.verified?).to be true
+        end
       end
 
       describe 'when decrypted params encapsulation is not conform' do