fix: change request enabled check should ignore disabled envs (#7869)

src/lib/features/change-request-access-service/sql-change-request-access-read-model.test.ts

@@ -0,0 +1,49 @@
+import dbInit, { type ITestDb } from '../../../test/e2e/helpers/database-init';
+import getLogger from '../../../test/fixtures/no-logger';
+import { createChangeRequestAccessReadModel } from './createChangeRequestAccessReadModel';
+import { createTestConfig } from '../../../test/config/test-config';
+import type { IChangeRequestAccessReadModel } from './change-request-access-read-model';
+
+let db: ITestDb;
+
+let readModel: IChangeRequestAccessReadModel;
+
+beforeAll(async () => {
+    db = await dbInit('change_request_access_read_model', getLogger);
+
+    const config = createTestConfig({
+        getLogger,
+    });
+
+    readModel = createChangeRequestAccessReadModel(db.rawDatabase, config);
+});
+
+afterAll(async () => {
+    await db.destroy();
+});
+
+test(`Should indicate change request enabled status`, async () => {
+    // no change requests
+    const defaultStatus =
+        await readModel.isChangeRequestsEnabledForProject('default');
+    expect(defaultStatus).toBe(false);
+
+    // change request enabled in enabled environment
+    await db.rawDatabase('change_request_settings').insert({
+        project: 'default',
+        environment: 'default',
+        required_approvals: 1,
+    });
+    const enabledStatus =
+        await readModel.isChangeRequestsEnabledForProject('default');
+    expect(enabledStatus).toBe(true);
+
+    // change request enabled in disabled environment
+    await db.stores.projectStore.deleteEnvironmentForProject(
+        'default',
+        'default',
+    );
+    const disabledStatus =
+        await readModel.isChangeRequestsEnabledForProject('default');
+    expect(disabledStatus).toBe(false);
+});

src/lib/features/change-request-access-service/sql-change-request-access-read-model.ts

@@ -70,14 +70,20 @@ export class ChangeRequestAccessReadModel
     public async isChangeRequestsEnabledForProject(
         project: string,
     ): Promise<boolean> {
-        const result = await this.db.raw(
+        const result = await this.db('change_request_settings')
-            `SELECT EXISTS(SELECT 1
+            .join('project_environments', function () {
-                           FROM change_request_settings
+                return this.on(
-                           WHERE project = ?
+                    'change_request_settings.project',
-                           ) AS present`,
+                    'project_environments.project_id',
-            [project],
+                ).andOn(
+                    'change_request_settings.environment',
+                    'project_environments.environment_name',
                 );
-        const { present } = result.rows[0];
+            })
-        return present;
+            .where('change_request_settings.project', project)
+            .select('change_request_settings.project')
+            .first();
+
+        return Boolean(result);
     }
 }

src/lib/features/change-request-segment-usage-service/change-request-segment-usage-read-model.test.ts

@@ -18,7 +18,7 @@ const FLAG_NAME = 'crarm-test-flag';
 let readModel: IChangeRequestSegmentUsageReadModel;
 
 beforeAll(async () => {
-    db = await dbInit('change_request_access_read_model_serial', getLogger);
+    db = await dbInit('change_request_segment_usage_read_model', getLogger);
 
     user = await db.stores.userStore.insert({
         username: 'cr-creator',