Skip to content

feat: Code formatting cleanup and Postman test reorganization #240

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
pm7y merged 2 commits into from
Dec 26, 2025
Merged

feat: Code formatting cleanup and Postman test reorganization #240

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
d4c6b6b
feat: Code formatting cleanup and Postman test reorganization
pm7y Dec 26, 2025
82a2bec
chore: Refactor middleware
pm7y Dec 26, 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
9 changes: 6 additions & 3 deletions .gitignore
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -221,7 +221,7 @@ ClientBin/
*.publishsettings
orleans.codegen.cs

# Including strong name files can present a security risk
# Including strong name files can present a security risk
# (https://github.com/github/gitignore/pull/2483#issue-259490424)
#*.snk

Expand Down Expand Up @@ -317,7 +317,7 @@ __pycache__/
# OpenCover UI analysis results
OpenCover/

# Azure Stream Analytics local run output
# Azure Stream Analytics local run output
ASALocalRun/

# MSBuild Binary and Structured Log
Expand All @@ -326,12 +326,13 @@ ASALocalRun/
# NVidia Nsight GPU debugger configuration file
*.nvuser

# MFractors (Xamarin productivity tool) working folder
# MFractors (Xamarin productivity tool) working folder
.mfractor/

# Build artifacts
/win-x64
/src/AzureEventGridSimulator/appsettings.development.json
/src/AzureEventGridSimulator.ConformanceTests/*.development.json
/build-win64
/build-osx-x64
/build-linux-x64
Expand All @@ -355,3 +356,5 @@ my.env
.scannerwork
.claude
.specify
**/test-results/
specs
1 change: 1 addition & 0 deletions docker/Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ WORKDIR /source
# copy build configuration files first
COPY /src/Directory.Build.props .
COPY /src/Directory.Packages.props .
COPY /src/.editorconfig .

# copy source
COPY /src/AzureEventGridSimulator ./AzureEventGridSimulator
Expand Down
18 changes: 17 additions & 1 deletion docker/appsettings.docker.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,9 +20,10 @@
},
"topics": [
{
"name": "ExampleTopic",
"name": "EventGridTopic",
"port": 60101,
"key": "TheLocal+DevelopmentKey=",
"inputSchema": "EventGridSchema",
"subscribers": {
"http": [
{
Expand Down Expand Up @@ -88,6 +89,21 @@
}
]
}
},
{
"name": "CloudEventsTopic",
"port": 60102,
"key": "TheLocal+DevelopmentKey=",
"inputSchema": "CloudEventV1_0",
"subscribers": {
"http": [
{
"name": "CloudEventsRequestCatcherSubscription",
"endpoint": "https://azureeventgridsimulator.requestcatcher.com/",
"disableValidation": true
}
]
}
}
]
}
1 change: 1 addition & 0 deletions docker/docker-compose.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ services:
ports:
# add a port mapping for each topic in the settings file
- "60101:60101"
- "60102:60102"
volumes:
# map this folder to a read-only folder '/aegs' in the container
# this allows us to access files (e.g. settings or certificates) from within the container
Expand Down
62 changes: 62 additions & 0 deletions src/.editorconfig
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,62 @@
# EditorConfig for AzureEventGridSimulator
# https://editorconfig.org

root = true

[*]
indent_style = space
indent_size = 4
end_of_line = lf
charset = utf-8
trim_trailing_whitespace = true
insert_final_newline = true

[*.cs]
# Meziantou.Analyzer rules - downgrade to suggestions for existing code
# These can be gradually fixed over time

# MA0002: Use an overload that has a IEqualityComparer<string> or IComparer<string> parameter
dotnet_diagnostic.MA0002.severity = suggestion

# MA0004: Use Task.ConfigureAwait(false)
# Not needed in ASP.NET Core applications
dotnet_diagnostic.MA0004.severity = none

# MA0006: Use string.Equals instead of == operator
dotnet_diagnostic.MA0006.severity = suggestion

# MA0011: Use overload with IFormatProvider
dotnet_diagnostic.MA0011.severity = suggestion

# MA0015: Use overload of ArgumentException with parameter name
dotnet_diagnostic.MA0015.severity = suggestion

# MA0016: Prefer using collection abstraction instead of implementation
dotnet_diagnostic.MA0016.severity = suggestion

# MA0026: TODO comments
dotnet_diagnostic.MA0026.severity = suggestion

# MA0039: Do not write your own certificate validation method
# This is intentionally disabled because the simulator has a user-opt-in
# "dangerousAcceptAnyServerCertificateValidator" setting for testing purposes
dotnet_diagnostic.MA0039.severity = none

# MA0046: Event delegate parameters
dotnet_diagnostic.MA0046.severity = suggestion

# MA0048: File name must match type name
dotnet_diagnostic.MA0048.severity = suggestion

# MA0049: Type name should not match containing namespace
dotnet_diagnostic.MA0049.severity = suggestion

# MA0051: Method is too long
dotnet_diagnostic.MA0051.severity = suggestion

# MA0055: Do not use finalizer
# Finalizers are used in test fixtures for cleanup, which is acceptable
dotnet_diagnostic.MA0055.severity = suggestion

# MA0158: Use System.Threading.Lock
dotnet_diagnostic.MA0158.severity = suggestion
Loading
Loading