add label support

Author: aprilk-ms

__tests__/feature-flag-client.test.ts

@@ -27,12 +27,13 @@ describe('Feature Flag Client', () => {
 
   it('should list feature flags', async () => {
     const appConfigEndpoint = 'https://example.com'
+    const label = 'test-label'
     const getMock = jest.spyOn(axios, 'get').mockResolvedValue({
       status: 200,
       data: featureListResponse
     })
 
-    const results = await listFeatureFlags(appConfigEndpoint)
+    const results = await listFeatureFlags(appConfigEndpoint, label)
     expect(results.items.length).toEqual(1)
     expect(results.items[0].key).toEqual(
       '.appconfig.featureflag/featureFlagId1'
@@ -41,27 +42,36 @@ describe('Feature Flag Client', () => {
 
   it('should throw error when list api fails', async () => {
     const appConfigEndpoint = 'https://example.com'
+    const label = 'test-label'
 
     const getMock = jest.spyOn(axios, 'get').mockResolvedValue({
       status: 500,
       statusText: 'Internal Server Error'
     })
 
-    await expect(listFeatureFlags(appConfigEndpoint)).rejects.toThrow(ApiError)
+    await expect(listFeatureFlags(appConfigEndpoint, label)).rejects.toThrow(
+      ApiError
+    )
   })
 
   it('create or update feature flag', async () => {
     const appConfigEndpoint = 'https://example.com'
     const featureFlagId = 'featureFlagId1'
+    const label = 'test-label'
     const value = getDummyFeatureFlagItem(featureFlagId)
 
     const getMock = jest.spyOn(axios, 'put').mockResolvedValue({
       status: 200
     })
 
-    await createOrUpdateFeatureFlag(appConfigEndpoint, featureFlagId, value)
+    await createOrUpdateFeatureFlag(
+      appConfigEndpoint,
+      featureFlagId,
+      value,
+      label
+    )
     expect(getMock).toBeCalledWith(
-      `${appConfigEndpoint}/kv/.appconfig.featureflag%2FfeatureFlagId1?api-version=2023-11-01`,
+      `${appConfigEndpoint}/kv/.appconfig.featureflag%2FfeatureFlagId1?api-version=2023-11-01&label=test-label`,
       {
         content_type:
           'application/vnd.microsoft.appconfig.ff+json;charset=utf-8',
@@ -80,6 +90,7 @@ describe('Feature Flag Client', () => {
   it('should throw error when create or update feature flag fails', async () => {
     const appConfigEndpoint = 'https://example.com'
     const featureFlagId = 'featureFlagId1'
+    const label = 'test-label'
     const value = getDummyFeatureFlagItem(featureFlagId)
 
     const getMock = jest.spyOn(axios, 'put').mockResolvedValue({
@@ -88,21 +99,22 @@ describe('Feature Flag Client', () => {
     })
 
     await expect(
-      createOrUpdateFeatureFlag(appConfigEndpoint, featureFlagId, value)
+      createOrUpdateFeatureFlag(appConfigEndpoint, featureFlagId, value, label)
     ).rejects.toThrow(ApiError)
   })
 
   it('should delete feature flag', async () => {
     const appConfigEndpoint = 'https://example.com'
     const featureFlagId = 'featureFlagId1'
+    const label = 'test-label'
 
     const getMock = jest.spyOn(axios, 'delete').mockResolvedValue({
       status: 200
     })
 
-    await deleteFeatureFlag(appConfigEndpoint, featureFlagId)
+    await deleteFeatureFlag(appConfigEndpoint, featureFlagId, label)
     expect(getMock).toBeCalledWith(
-      `${appConfigEndpoint}/kv/.appconfig.featureflag%2FfeatureFlagId1?api-version=2023-11-01`,
+      `${appConfigEndpoint}/kv/.appconfig.featureflag%2FfeatureFlagId1?api-version=2023-11-01&label=test-label`,
       {
         headers: {
           Accept: '*/*',
@@ -116,13 +128,14 @@ describe('Feature Flag Client', () => {
   it('should throw error when delete feature flag fails', async () => {
     const appConfigEndpoint = 'https://example.com'
     const featureFlagId = 'featureFlagId1'
+    const label = 'test-label'
 
     const getMock = jest.spyOn(axios, 'delete').mockResolvedValue({
       status: 500
     })
 
     await expect(
-      deleteFeatureFlag(appConfigEndpoint, featureFlagId)
+      deleteFeatureFlag(appConfigEndpoint, featureFlagId, label)
     ).rejects.toThrow(ApiError)
   })
 

__tests__/input.test.ts

@@ -25,7 +25,8 @@ describe('getActionInput', () => {
       configFile: 'configFile',
       strictSync: true,
       appConfigEndpoint: 'https://example.com',
-      operation: 'deploy'
+      operation: 'deploy',
+      label: ''
     })
   })
 
@@ -38,7 +39,8 @@ describe('getActionInput', () => {
       configFile: 'configFile',
       strictSync: false, // doesn't matter in validate mode
       appConfigEndpoint: '', // doesn't matter in validate mode
-      operation: 'validate'
+      operation: 'validate',
+      label: ''
     })
   })
 

__tests__/update-feature-flags.test.ts

@@ -53,7 +53,8 @@ describe('updateFeatureFlags', () => {
     expect(createOrUpdateFeatureFlag).toHaveBeenCalledWith(
       input.appConfigEndpoint,
       'featureFlagId2',
-      configs[1]
+      configs[1],
+      'test-label'
     )
     expect(infoMock).toHaveBeenCalledWith('Updated 1 feature flags')
   })
@@ -81,7 +82,8 @@ describe('updateFeatureFlags', () => {
     expect(createOrUpdateFeatureFlag).toHaveBeenCalledWith(
       input.appConfigEndpoint,
       'featureFlagId2',
-      configs[1]
+      configs[1],
+      'test-label'
     )
     expect(infoMock).toHaveBeenCalledWith('Updated 1 feature flags')
     expect(infoMock).toHaveBeenCalledWith(
@@ -90,7 +92,8 @@ describe('updateFeatureFlags', () => {
     expect(deleteFeatureFlag).toHaveBeenCalledTimes(1)
     expect(deleteFeatureFlag).toHaveBeenCalledWith(
       input.appConfigEndpoint,
-      'featureFlagId3'
+      'featureFlagId3',
+      'test-label'
     )
   })
 
@@ -99,7 +102,8 @@ describe('updateFeatureFlags', () => {
       configFile: 'configFile',
       strictSync: false,
       appConfigEndpoint: 'https://example.com',
-      operation: 'deploy'
+      operation: 'deploy',
+      label: 'test-label'
     }
   }
 

action.yml

@@ -19,6 +19,11 @@ inputs:
     description: 'Destination endpoint for the Azure App Configuration store'
     required: false
 
+  app-config-label:
+    description: 'Label for the config'
+    required: false
+    default: None
+
   operation: # Validate the configuration file only
     description:
       'Possible values: validate or deploy - deploy by default. validate: only