fix: rename getInActiveUsers to getInactiveUsers for consistency

Author: Romarionijim

infra/api/entities/users-api/Users.ts

@@ -45,14 +45,14 @@ export class Users extends ApiClient {
         }
     }
 
-    async getInActiveUsers() {
+    async getInactiveUsers() {
         let res = await this.getUserStatus('inactive')
         return res
     }
 
     async deleteInactiveUsers() {
         let response: APIResponse | undefined
-        let inActiveUsers = await this.getInActiveUsers()
+        let inActiveUsers = await this.getInactiveUsers()
         for (let user of inActiveUsers) {
             response = await this.delete(`${this.usersEndpoint}/${user.id}`, { isAuthorizationRequired: true })
         }

tests/api_tests/users/UsersApiTests.spec.ts

@@ -44,7 +44,7 @@ test.describe('Api tests for GoRestApi endpoints', async () => {
         await test.step('make a request to delete all users that have an inactive status', async () => {
             let response = await usersApi.deleteInactiveUsers()
             expect(response?.status()).toBe(StatusCode.UNAUTHORIZED)
-            let actualInactiveUsers = await usersApi.getInActiveUsers()
+            let actualInactiveUsers = await usersApi.getInactiveUsers()
             let expectedInactiveUsersLength = actualInactiveUsers.length
             expect(expectedInactiveUsersLength).toBe(0)
         })