Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
69 commits
Select commit Hold shift + click to select a range
df290c2
add initial integration test file
amerjusupovic Mar 3, 2025
e99a659
update auth
amerjusupovic Mar 3, 2025
ba1d8f0
add skipping
amerjusupovic Mar 3, 2025
366687c
organize tests into unit and integration
amerjusupovic Mar 3, 2025
421862a
update formatting
amerjusupovic Mar 3, 2025
8d6db8c
WIP create config store
amerjusupovic Mar 4, 2025
10ccabc
edit
amerjusupovic Mar 4, 2025
6dd421a
update integration setup
amerjusupovic Mar 4, 2025
6b24ac9
copilot updates
amerjusupovic Mar 6, 2025
28a14cc
initial working version of integration tests
amerjusupovic Mar 7, 2025
68a005c
remove obsolete tags and update formatting
amerjusupovic Mar 7, 2025
1c992a5
updating tests
amerjusupovic Mar 12, 2025
5cb73ad
format
amerjusupovic Mar 12, 2025
5f114da
remove skipping, add tests
amerjusupovic Mar 14, 2025
c495fc4
add cleanup stale resource call on startup and dispose
amerjusupovic Mar 14, 2025
131beba
Merge branch 'main' of https://github.com/Azure/AppConfiguration-Dotn…
amerjusupovic Mar 14, 2025
5dbb856
add back removed line
amerjusupovic Mar 17, 2025
b564f09
use persistent resource group
amerjusupovic Mar 17, 2025
b305239
add ordering of apis test
amerjusupovic Mar 18, 2025
25502c4
add refresh test
amerjusupovic Mar 18, 2025
e85e4b6
add tests
amerjusupovic Mar 18, 2025
2453aa6
add failover test
amerjusupovic Mar 19, 2025
2d8b9a6
Merge branch 'main' of https://github.com/Azure/AppConfiguration-Dotn…
amerjusupovic Mar 19, 2025
f4ee675
adding snapshot tests first draft
amerjusupovic Mar 27, 2025
d9f18ef
fix snapshot tests
amerjusupovic Mar 27, 2025
fb8aa36
add key vault tests
amerjusupovic Mar 28, 2025
db881d5
add more keyvault tests
amerjusupovic Mar 31, 2025
ff5596a
fix keyvault tests
amerjusupovic Apr 2, 2025
706ed81
add cleanup of key values/secrets/snapshot
amerjusupovic Apr 2, 2025
c4ce614
fix key vault isolation
amerjusupovic Apr 2, 2025
41438c1
remove base exception catches
amerjusupovic Apr 4, 2025
a634dd5
update ci.yml to work with integration tests and use github repo secrets
amerjusupovic Apr 7, 2025
b94c75c
add id token write permission
amerjusupovic Apr 9, 2025
0394f7b
update ci.yml step to create subscription file
amerjusupovic Apr 9, 2025
63c148e
edit ci.yml
amerjusupovic Apr 9, 2025
6586423
fix github action
amerjusupovic Apr 9, 2025
36a045f
update comment in powershell script
amerjusupovic Apr 10, 2025
2b2f1f5
remove unused packages
amerjusupovic Apr 11, 2025
965f348
update comment, remove unused methods
amerjusupovic Apr 11, 2025
14445c0
add request tracing test
amerjusupovic Apr 14, 2025
8371a76
Merge branch 'main' of https://github.com/Azure/AppConfiguration-Dotn…
amerjusupovic Apr 14, 2025
70540df
check for status code in catch, move cleanupstaleresources call to di…
amerjusupovic Apr 14, 2025
683a079
PR comments
amerjusupovic Apr 15, 2025
b3b42df
comments
amerjusupovic Apr 15, 2025
811aa7d
comments
amerjusupovic Apr 15, 2025
1d3ae8e
Merge branch 'main' of https://github.com/Azure/AppConfiguration-Dotn…
amerjusupovic Apr 21, 2025
fa5249b
Merge branch 'main' of https://github.com/Azure/AppConfiguration-Dotn…
amerjusupovic Apr 21, 2025
a9f7612
Merge branch 'main' into ajusupovic/additional-unit-tests
amerjusupovic Apr 24, 2025
502ae9f
update copilot instructions
amerjusupovic Apr 25, 2025
d8f03d8
Merge branch 'ajusupovic/additional-unit-tests' of https://github.com…
amerjusupovic Apr 25, 2025
2c8a630
update copilot instructions
amerjusupovic Apr 25, 2025
78e73f7
fix try blocks to only surround throwing code
amerjusupovic Apr 30, 2025
dc4db90
fix setuptestkeys, simplify comments and setup code
amerjusupovic Apr 30, 2025
0f4a1d6
Merge branch 'main' into ajusupovic/additional-unit-tests
amerjusupovic May 2, 2025
f20fc71
Merge branch 'main' of https://github.com/Azure/AppConfiguration-Dotn…
amerjusupovic May 12, 2025
e8fd105
Merge branch 'ajusupovic/additional-unit-tests' of https://github.com…
amerjusupovic May 12, 2025
eb081ba
tagfilters integration test in progress
amerjusupovic May 13, 2025
c2933e3
tagfilters integration test in progress
amerjusupovic May 13, 2025
d85201e
Merge branch 'main' of https://github.com/Azure/AppConfiguration-Dotn…
amerjusupovic May 13, 2025
8af23a3
correct api version
amerjusupovic May 13, 2025
f16c1ed
fixed tagfilter test
amerjusupovic May 13, 2025
5558b53
Merge branch 'main' of https://github.com/Azure/AppConfiguration-Dotn…
amerjusupovic May 14, 2025
7d11cd8
fix requesttracing test
amerjusupovic May 14, 2025
52fbf74
allow commas in json reader
amerjusupovic May 14, 2025
671433d
stale threshold
amerjusupovic May 16, 2025
6e2a2d1
remove copilot instructions
amerjusupovic May 16, 2025
2a0a4e5
use outputhelper
amerjusupovic May 16, 2025
6de8e8b
fix cleanupstale
amerjusupovic May 19, 2025
b9157e3
update setup method, remove flag
amerjusupovic May 20, 2025
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 10 additions & 0 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@ on:

permissions:
security-events: write
id-token: write

jobs:
build:
Expand All @@ -40,8 +41,17 @@ jobs:
- name: Dotnet Pack
run: pwsh pack.ps1

- name: Azure Login with OIDC
uses: azure/login@v1
with:
client-id: ${{ secrets.AZURE_CLIENT_ID }}
tenant-id: ${{ secrets.AZURE_TENANT_ID }}
subscription-id: ${{ secrets.AZURE_SUBSCRIPTION_ID }}

- name: Dotnet Test
run: pwsh test.ps1
env:
AZURE_SUBSCRIPTION_ID: ${{ secrets.AZURE_SUBSCRIPTION_ID }}

- name: Publish Test Results
uses: actions/upload-artifact@v4
Expand Down
3 changes: 3 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,9 @@
# Azure Functions localsettings file
local.settings.json

# Integration test secrets
appsettings.Secrets.json

# User-specific files
*.suo
*.user
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -426,7 +426,12 @@ private FeatureFlag ParseFeatureFlag(string settingKey, string settingValue)
{
var featureFlag = new FeatureFlag();

var reader = new Utf8JsonReader(System.Text.Encoding.UTF8.GetBytes(settingValue));
var reader = new Utf8JsonReader(
System.Text.Encoding.UTF8.GetBytes(settingValue),
new JsonReaderOptions
{
AllowTrailingCommas = true
});

try
{
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,47 @@
#!/usr/bin/env pwsh

# GetAzureSubscription.ps1
# This script gets the AppConfig - Dev subscription ID and saves it to a JSON file

$outputPath = Join-Path $PSScriptRoot "appsettings.Secrets.json"

Write-Host "Checking for active Azure CLI login"

az account show | Out-Null

if ($LASTEXITCODE -ne 0) {
Write-Host "Must be logged in with the Azure CLI to proceed"

az login

if ($LASTEXITCODE -ne 0) {
Write-Host "Azure login failed"
return
}
}

az account set --name "AppConfig - Dev"

# Get current subscription from az CLI
$subscriptionId = az account show --query id -o tsv 2>&1

if ($LASTEXITCODE -ne 0) {
Write-Host "Azure CLI command failed with exit code $LASTEXITCODE. Output: $subscriptionId"
return
}

# Check if the output is empty
if ([string]::IsNullOrWhiteSpace($subscriptionId)) {
Write-Host "No active Azure subscription found. Please run 'az login' first."

exit 1
}

# If successful, save the subscription ID to a JSON file
$result = @{
SubscriptionId = $subscriptionId.Trim()
}

$result | ConvertTo-Json | Out-File $outputPath -Encoding utf8
Write-Host "Subscription information saved to: $outputPath"
exit 0
Loading