Merge pull request #14 from ronaldbosma/small-patch

Add integration tests to PR trigger and use package reference update

Author: ronaldbosma

.github/workflows/azure-dev.yml

@@ -16,6 +16,7 @@ on:
       - 'hooks/**'
       - 'infra/**'
       - 'src/**'
+      - 'tests/AISQuick.IntegrationTests/**'
       - 'azure.yaml'
       - 'bicepconfig.json'
 
@@ -31,6 +32,7 @@ env:
   # Add a unique suffix to the environment name for pull requests to avoid name conflicts
   AZURE_ENV_NAME: ${{ github.event.pull_request.number && format('{0}-pr{1}', vars.AZURE_ENV_NAME, github.event.pull_request.number) || vars.AZURE_ENV_NAME }}
   AZURE_LOCATION: ${{ vars.AZURE_LOCATION }}
+  AZURE_SUBSCRIPTION_ID: ${{ secrets.AZURE_SUBSCRIPTION_ID || vars.AZURE_SUBSCRIPTION_ID }}
 
   # Individual resource inclusion flags
   INCLUDE_API_MANAGEMENT: ${{ vars.INCLUDE_API_MANAGEMENT }}
@@ -135,8 +137,6 @@ jobs:
     name: Deploy to Azure
     needs: build-verify-package
     runs-on: ubuntu-latest
-    env:
-      AZURE_SUBSCRIPTION_ID: ${{ secrets.AZURE_SUBSCRIPTION_ID || vars.AZURE_SUBSCRIPTION_ID }}
     outputs:
       azure-resource-group: ${{ steps.get-outputs.outputs.azure-resource-group }}
       azure-api-management-name: ${{ steps.get-outputs.outputs.azure-api-management-name }}
@@ -277,8 +277,6 @@ jobs:
         (needs.integration-tests.result == 'skipped' && needs.deploy.result == 'success')
       )
     runs-on: ubuntu-latest
-    env:
-      AZURE_SUBSCRIPTION_ID: ${{ secrets.AZURE_SUBSCRIPTION_ID || vars.AZURE_SUBSCRIPTION_ID }}
 
     steps:
     - name: Checkout code

azure.yaml

@@ -2,7 +2,7 @@
 
 name: azure-integration-services-quickstart
 metadata:
-  template: [email protected].0
+  template: [email protected].1
 hooks:
   predown:
     - shell: pwsh

tests/AISQuick.IntegrationTests/AISQuick.IntegrationTests.csproj

@@ -11,22 +11,25 @@
       -->
     <TestingPlatformShowTestsFailure>true</TestingPlatformShowTestsFailure>
   </PropertyGroup>
+  
+  <ItemGroup>
+    <PackageReference Update="Microsoft.NET.Test.Sdk" Version="18.0.0" />
+    <PackageReference Update="Microsoft.Testing.Extensions.CodeCoverage" Version="18.1.0" />
+    <PackageReference Update="Microsoft.Testing.Extensions.TrxReport" Version="2.0.1" />
+    <PackageReference Update="MSTest.Analyzers" Version="4.0.1">
+      <PrivateAssets>all</PrivateAssets>
+      <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
+    </PackageReference>
+    <PackageReference Update="MSTest.TestAdapter" Version="4.0.1" />
+    <PackageReference Update="MSTest.TestFramework" Version="4.0.1" />
+  </ItemGroup>
 
   <ItemGroup>
     <PackageReference Include="Azure.Identity" Version="1.17.0" />
     <PackageReference Include="Azure.Security.KeyVault.Secrets" Version="4.8.0" />
     <PackageReference Include="Microsoft.Extensions.Configuration" Version="9.0.10" />
     <PackageReference Include="Microsoft.Extensions.Configuration.EnvironmentVariables" Version="9.0.10" />
     <PackageReference Include="Microsoft.Extensions.Configuration.FileExtensions" Version="9.0.10" />
-    <PackageReference Include="Microsoft.NET.Test.Sdk" Version="18.0.0" />
-    <PackageReference Include="Microsoft.Testing.Extensions.CodeCoverage" Version="18.1.0" />
-    <PackageReference Include="Microsoft.Testing.Extensions.TrxReport" Version="2.0.1" />
-    <PackageReference Include="MSTest.Analyzers" Version="4.0.1">
-      <PrivateAssets>all</PrivateAssets>
-      <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
-    </PackageReference>
-    <PackageReference Include="MSTest.TestAdapter" Version="4.0.1" />
-    <PackageReference Include="MSTest.TestFramework" Version="4.0.1" />
     <PackageReference Include="Polly" Version="8.6.4" />
   </ItemGroup>