Admin Users Controller Tests

- added test for verifying users
- changed tests for active status to verified status
- added unverified method tests including the SuperAdmin ones

Author: alihadimazeh

spec/controllers/admin/users_controller_spec.rb

@@ -28,8 +28,8 @@
   end
 
   describe '#verified_users' do
-    it 'returns the list of active users' do
-      users = create_list(:user, 3, status: 'active') + [user, user_with_manage_users_permission]
+    it 'returns the list of verified users' do
+      users = create_list(:user, 3, verified: true) + [user, user_with_manage_users_permission]
       get :verified
       expect(response).to have_http_status(:ok)
       response_user_ids = response.parsed_body['data'].pluck('id')
@@ -47,6 +47,24 @@
     end
   end
 
+  describe '#unverified_users' do
+    it 'returns the list of unverified users' do
+      users = create_list(:user, 3, verified: false)
+      get :unverified
+      expect(response).to have_http_status(:ok)
+      response_user_ids = response.parsed_body['data'].pluck('id')
+      expect(response_user_ids).to match_array(users.pluck(:id))
+    end
+
+    it 'excludes users with a different provider' do
+      greenlight_users = create_list(:user, 3, provider: 'greenlight', verified: false)
+      role_with_provider_test = create(:role, provider: 'test')
+      create(:user, provider: 'test', role: role_with_provider_test, verified: false)
+      get :unverified
+      expect(response.parsed_body['data'].pluck('id')).to match_array(greenlight_users.pluck(:id))
+    end
+  end
+
   describe '#pending' do
     it 'returns a list of pending users' do
       users = create_list(:user, 3, status: 'pending')
@@ -97,14 +115,22 @@
         sign_in_user(super_admin)
       end
 
-      it 'returns the list of active users' do
-        users = create_list(:user, 3, status: 'active') + [user, user_with_manage_users_permission]
+      it 'returns the list of verified users' do
+        users = create_list(:user, 3, verified: true) + [user, user_with_manage_users_permission]
         get :verified
         expect(response).to have_http_status(:ok)
         response_user_ids = response.parsed_body['data'].pluck('id')
         expect(response_user_ids).to match_array(users.pluck(:id))
       end
 
+      it 'returns the list of unverified users' do
+        users = create_list(:user, 3, verified: false)
+        get :unverified
+        expect(response).to have_http_status(:ok)
+        response_user_ids = response.parsed_body['data'].pluck('id')
+        expect(response_user_ids).to match_array(users.pluck(:id))
+      end
+
       it 'returns the list of pending users' do
         users = create_list(:user, 3, status: 'pending')
         get :pending
@@ -130,6 +156,14 @@
       expect(user.reload.status).to eq('banned')
     end
 
+    it 'verifies an unverified user' do
+      unverified_user = create(:user, verified: false)
+      post :update, params: { id: unverified_user.id, user: { verified: true } }
+      unverified_user.reload
+      expect(response).to have_http_status(:ok)
+      expect(unverified_user.verified).to be(true)
+    end
+
     context 'user without ManageUsers permission' do
       before do
         sign_in_user(user)