fix(api,api-client): Add environmentSlug in multiple places across the variable module

apps/api/src/variable/service/variable.service.ts

@@ -121,6 +121,12 @@ export class VariableService {
         versions: {
           select: {
             environmentId: true,
+            environment: {
+              select: {
+                id: true,
+                slug: true
+              }
+            },
             value: true
           }
         }
@@ -245,6 +251,12 @@ export class VariableService {
             select: {
               id: true,
               environmentId: true,
+              environment: {
+                select: {
+                  id: true,
+                  slug: true
+                }
+              },
               value: true,
               version: true
             }
@@ -506,6 +518,16 @@ export class VariableService {
           where: {
             environmentId
           },
+          select: {
+            value: true,
+            environmentId: true,
+            environment: {
+              select: {
+                id: true,
+                slug: true
+              }
+            }
+          },
           orderBy: {
             version: 'desc'
           },
@@ -568,11 +590,21 @@ export class VariableService {
             id: true,
             name: true
           }
+        },
+        versions: {
+          select: {
+            environmentId: true,
+            environment: {
+              select: {
+                id: true,
+                slug: true
+              }
+            }
+          }
         }
       },
       skip: page * limit,
       take: limitMaxItemsPerPage(limit),
-
       orderBy: {
         [sort]: order
       }
@@ -586,6 +618,7 @@ export class VariableService {
           environment: {
             name: Environment['name']
             id: Environment['id']
+            slug: Environment['slug']
           }
           value: VariableVersion['value']
           version: VariableVersion['version']
@@ -619,6 +652,9 @@ export class VariableService {
           },
           orderBy: {
             version: 'desc'
+          },
+          include: {
+            environment: true
           }
         })
 
@@ -628,7 +664,8 @@ export class VariableService {
           variablesWithEnvironmentalValues.get(variable.id).values.push({
             environment: {
               id: latestVersion.environmentId,
-              name: envIds.get(latestVersion.environmentId)
+              name: envIds.get(latestVersion.environmentId),
+              slug: latestVersion.environment.slug
             },
             value: latestVersion.value,
             version: latestVersion.version
@@ -640,7 +677,8 @@ export class VariableService {
               {
                 environment: {
                   id: latestVersion.environmentId,
-                  name: envIds.get(latestVersion.environmentId)
+                  name: envIds.get(latestVersion.environmentId),
+                  slug: latestVersion.environment.slug
                 },
                 value: latestVersion.value,
                 version: latestVersion.version

apps/api/src/variable/variable.e2e.spec.ts

@@ -204,6 +204,8 @@ describe('Variable Controller Tests', () => {
       expect(body.projectId).toBe(project1.id)
       expect(body.versions.length).toBe(1)
       expect(body.versions[0].value).toBe('Variable 3 value')
+      // expect(body.versions[0].environment.id).toBe(environment2.id)
+      expect(body.versions[0].environment.slug).toBe(environment2.slug)
 
       const variable = await prisma.variable.findUnique({
         where: {
@@ -382,10 +384,15 @@ describe('Variable Controller Tests', () => {
       const variableVersion = await prisma.variableVersion.findMany({
         where: {
           variableId: variable1.id
+        },
+        include: {
+          environment: true
         }
       })
 
       expect(variableVersion.length).toBe(1)
+      expect(variableVersion[0].environment.id).toBe(environment1.id)
+      expect(variableVersion[0].environment.slug).toBe(environment1.slug)
     })
 
     it('should create a new version if the value is updated', async () => {
@@ -596,6 +603,7 @@ describe('Variable Controller Tests', () => {
       expect(values.length).toBe(1)
       expect(values[0].value).toBe('Variable 1 value')
       expect(values[0].environment.id).toBe(environment1.id)
+      expect(values[0].environment.slug).toBe(environment1.slug)
       expect(variable.id).toBe(variable1.id)
       expect(variable.name).toBe('Variable 1')
 

packages/api-client/src/types/variable.types.d.ts

@@ -16,6 +16,10 @@ export interface Variable {
     {
       value: string
       environmentId: string
+      environment: {
+        id: string
+        slug: string
+      }
     }
   ]
 }
@@ -50,6 +54,10 @@ export interface UpdateVariableResponse {
     {
       value: string
       environmentId: string
+      environment: {
+        id: string
+        slug: string
+      }
     }
   ]
 }
@@ -87,6 +95,7 @@ export interface GetAllVariablesOfProjectResponse
         environment: {
           id: string
           name: string
+          slug: string
         }
         value: string
         version: number

packages/api-client/tests/variable.spec.ts

@@ -110,6 +110,8 @@ describe('Get Variable Tests', () => {
     expect(variable.data.versions.length).toBe(1)
     expect(variable.data.versions[0].value).toBe('Variable 2 value')
     expect(variable.data.versions[0].environmentId).toBe(environment.id)
+    expect(variable.data.versions[0].environment.id).toBe(environment.id)
+    expect(variable.data.versions[0].environment.slug).toBe(environment.slug)
 
     // Delete the variable
     const deleteVariable = await variableController.deleteVariable(
@@ -167,6 +169,12 @@ describe('Get Variable Tests', () => {
     expect(updateVariable.data.updatedVersions[0].environmentId).toBe(
       environment.id
     )
+    expect(updateVariable.data.updatedVersions[0].environment.id).toBe(
+      environment.id
+    )
+    expect(updateVariable.data.updatedVersions[0].environment.slug).toBe(
+      environment.slug
+    )
   })
 
   // Roll back a variable
@@ -222,6 +230,7 @@ describe('Get Variable Tests', () => {
       expect(value).toHaveProperty('environment')
       expect(value.environment).toHaveProperty('id')
       expect(value.environment).toHaveProperty('name')
+      expect(value.environment).toHaveProperty('slug')
       expect(value).toHaveProperty('value')
       expect(value).toHaveProperty('version')
     })