Skip to content

Merge branch 'master' into feature/docker-and-package-upgrades #1969

Merge branch 'master' into feature/docker-and-package-upgrades

Merge branch 'master' into feature/docker-and-package-upgrades #1969

Triggered via push August 17, 2023 15:21
Status Failure
Total duration 1h 55m 45s
Artifacts 30
This run and associated checks have been archived and are scheduled for deletion. Learn more about checks retention
Matrix: tests
build-and-push-docker-image
1m 45s
build-and-push-docker-image
deploy-output
0s
deploy-output
Fit to window
Zoom out
Zoom in

Annotations

2 errors and 570 warnings
tests (controlled, all-features, simple)
Process completed with exit code 1.
tests (random, facilityuse-has-slots, simple)
Process completed with exit code 1.
tests (controlled, all-features, simple)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1, actions/setup-node@v1, actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
tests (controlled, all-features, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, all-features, simple)
Avoid empty interfaces [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, all-features, simple)
Exceptions should be public [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, all-features, simple)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, all-features, simple)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message, Exception innerException). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, all-features, simple)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, all-features, simple)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, all-features, simple)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, all-features, simple)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, all-features, simple)
In externally visible method 'Task<Order> StoreBookingEngine.ProcessOrderCreationFromOrderProposal(OrderIdComponents orderId, OrderIdTemplate orderIdTemplate, ILegalEntity seller, SimpleIdComponents sellerId, SimpleIdComponents customerAccountIdComponents, Order order)', validate parameter 'order' is non-null before using it. If appropriate, throw an ArgumentNullException when the argument is null or add a Code Contract precondition asserting non-null argument. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, all-features, simple)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (controlled, all-features, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, all-features, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, all-features, simple)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, all-features, simple)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (controlled, all-features, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, all-features, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, all-features, simple)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, facilityuse-has-slots, simple)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1, actions/setup-node@v1, actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
tests (controlled, facilityuse-has-slots, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, facilityuse-has-slots, simple)
Avoid empty interfaces [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, facilityuse-has-slots, simple)
Exceptions should be public [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, facilityuse-has-slots, simple)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, facilityuse-has-slots, simple)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message, Exception innerException). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, facilityuse-has-slots, simple)
Call System.IDisposable.Dispose on object created by 'new SemaphoreSlim(1, 1)' before all references to it are out of scope. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, facilityuse-has-slots, simple)
In externally visible method 'string IdempotencyStore.CalculateIdempotencyKey(OrderIdComponents orderId, string orderJson)', validate parameter 'orderId' is non-null before using it. If appropriate, throw an ArgumentNullException when the argument is null or add a Code Contract precondition asserting non-null argument. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, facilityuse-has-slots, simple)
Member CalculateIdempotencyKey does not access instance data and can be marked as static (Shared in VisualBasic) [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, facilityuse-has-slots, simple)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, facilityuse-has-slots, simple)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, facilityuse-has-slots, simple)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (controlled, facilityuse-has-slots, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, facilityuse-has-slots, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, facilityuse-has-slots, simple)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, facilityuse-has-slots, simple)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (controlled, facilityuse-has-slots, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, facilityuse-has-slots, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, facilityuse-has-slots, simple)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, no-auth, approval)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1, actions/setup-node@v1, actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
tests (controlled, no-auth, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-auth, approval)
Avoid empty interfaces [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-auth, approval)
Exceptions should be public [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-auth, approval)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-auth, approval)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message, Exception innerException). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-auth, approval)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-auth, approval)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-auth, approval)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-auth, approval)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-auth, approval)
In externally visible method 'Task<Order> StoreBookingEngine.ProcessOrderCreationFromOrderProposal(OrderIdComponents orderId, OrderIdTemplate orderIdTemplate, ILegalEntity seller, SimpleIdComponents sellerId, SimpleIdComponents customerAccountIdComponents, Order order)', validate parameter 'order' is non-null before using it. If appropriate, throw an ArgumentNullException when the argument is null or add a Code Contract precondition asserting non-null argument. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-auth, approval)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (controlled, no-auth, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, no-auth, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, no-auth, approval)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, no-auth, approval)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (controlled, no-auth, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, no-auth, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, no-auth, approval)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, no-auth, simple)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1, actions/setup-node@v1, actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
tests (controlled, no-auth, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-auth, simple)
Avoid empty interfaces [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-auth, simple)
Exceptions should be public [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-auth, simple)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-auth, simple)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message, Exception innerException). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-auth, simple)
Call System.IDisposable.Dispose on object created by 'new SemaphoreSlim(1, 1)' before all references to it are out of scope. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-auth, simple)
Member CalculateIdempotencyKey does not access instance data and can be marked as static (Shared in VisualBasic) [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-auth, simple)
In externally visible method 'string IdempotencyStore.CalculateIdempotencyKey(OrderIdComponents orderId, string orderJson)', validate parameter 'orderId' is non-null before using it. If appropriate, throw an ArgumentNullException when the argument is null or add a Code Contract precondition asserting non-null argument. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-auth, simple)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-auth, simple)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-auth, simple)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (controlled, no-auth, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, no-auth, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, no-auth, simple)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, no-auth, simple)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (controlled, no-auth, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, no-auth, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, no-auth, simple)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, all-features, approval)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1, actions/setup-node@v1, actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
tests (controlled, all-features, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, all-features, approval)
Avoid empty interfaces [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, all-features, approval)
Exceptions should be public [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, all-features, approval)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, all-features, approval)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message, Exception innerException). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, all-features, approval)
Call System.IDisposable.Dispose on object created by 'new SemaphoreSlim(1, 1)' before all references to it are out of scope. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, all-features, approval)
Member CalculateIdempotencyKey does not access instance data and can be marked as static (Shared in VisualBasic) [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, all-features, approval)
In externally visible method 'string IdempotencyStore.CalculateIdempotencyKey(OrderIdComponents orderId, string orderJson)', validate parameter 'orderId' is non-null before using it. If appropriate, throw an ArgumentNullException when the argument is null or add a Code Contract precondition asserting non-null argument. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, all-features, approval)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, all-features, approval)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, all-features, approval)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (controlled, all-features, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, all-features, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, all-features, approval)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, all-features, approval)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (controlled, all-features, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, all-features, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, all-features, approval)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, facilityuse-has-slots, approval)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1, actions/setup-node@v1, actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
tests (controlled, facilityuse-has-slots, approval)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (controlled, facilityuse-has-slots, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, facilityuse-has-slots, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, facilityuse-has-slots, approval)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, facilityuse-has-slots, approval)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (controlled, facilityuse-has-slots, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, facilityuse-has-slots, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, facilityuse-has-slots, approval)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, facilityuse-has-slots, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, facilityuse-has-slots, approval)
Avoid empty interfaces [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, facilityuse-has-slots, approval)
Exceptions should be public [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, facilityuse-has-slots, approval)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, facilityuse-has-slots, approval)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message, Exception innerException). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, facilityuse-has-slots, approval)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, facilityuse-has-slots, approval)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, facilityuse-has-slots, approval)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, facilityuse-has-slots, approval)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, facilityuse-has-slots, approval)
In externally visible method 'Task<Order> StoreBookingEngine.ProcessOrderCreationFromOrderProposal(OrderIdComponents orderId, OrderIdTemplate orderIdTemplate, ILegalEntity seller, SimpleIdComponents sellerId, SimpleIdComponents customerAccountIdComponents, Order order)', validate parameter 'order' is non-null before using it. If appropriate, throw an ArgumentNullException when the argument is null or add a Code Contract precondition asserting non-null argument. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-payment-reconciliation, approval)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1, actions/setup-node@v1, actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
tests (controlled, no-payment-reconciliation, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-payment-reconciliation, approval)
Avoid empty interfaces [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-payment-reconciliation, approval)
Exceptions should be public [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-payment-reconciliation, approval)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-payment-reconciliation, approval)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message, Exception innerException). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-payment-reconciliation, approval)
In externally visible method 'string IdempotencyStore.CalculateIdempotencyKey(OrderIdComponents orderId, string orderJson)', validate parameter 'orderId' is non-null before using it. If appropriate, throw an ArgumentNullException when the argument is null or add a Code Contract precondition asserting non-null argument. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-payment-reconciliation, approval)
Member CalculateIdempotencyKey does not access instance data and can be marked as static (Shared in VisualBasic) [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-payment-reconciliation, approval)
Call System.IDisposable.Dispose on object created by 'new SemaphoreSlim(1, 1)' before all references to it are out of scope. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-payment-reconciliation, approval)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-payment-reconciliation, approval)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-payment-reconciliation, approval)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (controlled, no-payment-reconciliation, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, no-payment-reconciliation, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, no-payment-reconciliation, approval)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, no-payment-reconciliation, approval)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (controlled, no-payment-reconciliation, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, no-payment-reconciliation, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, no-payment-reconciliation, approval)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, no-tax-calculation, approval)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1, actions/setup-node@v1, actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
tests (controlled, no-tax-calculation, approval)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (controlled, no-tax-calculation, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, no-tax-calculation, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, no-tax-calculation, approval)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, no-tax-calculation, approval)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (controlled, no-tax-calculation, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, no-tax-calculation, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, no-tax-calculation, approval)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, no-tax-calculation, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-tax-calculation, approval)
Avoid empty interfaces [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-tax-calculation, approval)
Exceptions should be public [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-tax-calculation, approval)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-tax-calculation, approval)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message, Exception innerException). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-tax-calculation, approval)
In externally visible method 'string IdempotencyStore.CalculateIdempotencyKey(OrderIdComponents orderId, string orderJson)', validate parameter 'orderId' is non-null before using it. If appropriate, throw an ArgumentNullException when the argument is null or add a Code Contract precondition asserting non-null argument. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-tax-calculation, approval)
Member CalculateIdempotencyKey does not access instance data and can be marked as static (Shared in VisualBasic) [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-tax-calculation, approval)
Call System.IDisposable.Dispose on object created by 'new SemaphoreSlim(1, 1)' before all references to it are out of scope. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-tax-calculation, approval)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-tax-calculation, approval)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-tax-calculation, simple)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1, actions/setup-node@v1, actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
tests (controlled, no-tax-calculation, simple)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (controlled, no-tax-calculation, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, no-tax-calculation, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, no-tax-calculation, simple)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, no-tax-calculation, simple)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (controlled, no-tax-calculation, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, no-tax-calculation, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, no-tax-calculation, simple)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, no-tax-calculation, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-tax-calculation, simple)
Avoid empty interfaces [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-tax-calculation, simple)
Exceptions should be public [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-tax-calculation, simple)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-tax-calculation, simple)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message, Exception innerException). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-tax-calculation, simple)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-tax-calculation, simple)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-tax-calculation, simple)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-tax-calculation, simple)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-tax-calculation, simple)
In externally visible method 'Task<Order> StoreBookingEngine.ProcessOrderCreationFromOrderProposal(OrderIdComponents orderId, OrderIdTemplate orderIdTemplate, ILegalEntity seller, SimpleIdComponents sellerId, SimpleIdComponents customerAccountIdComponents, Order order)', validate parameter 'order' is non-null before using it. If appropriate, throw an ArgumentNullException when the argument is null or add a Code Contract precondition asserting non-null argument. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-payment-reconciliation, simple)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1, actions/setup-node@v1, actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
tests (controlled, no-payment-reconciliation, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-payment-reconciliation, simple)
Avoid empty interfaces [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-payment-reconciliation, simple)
Exceptions should be public [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-payment-reconciliation, simple)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-payment-reconciliation, simple)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message, Exception innerException). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-payment-reconciliation, simple)
In externally visible method 'string IdempotencyStore.CalculateIdempotencyKey(OrderIdComponents orderId, string orderJson)', validate parameter 'orderId' is non-null before using it. If appropriate, throw an ArgumentNullException when the argument is null or add a Code Contract precondition asserting non-null argument. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-payment-reconciliation, simple)
Member CalculateIdempotencyKey does not access instance data and can be marked as static (Shared in VisualBasic) [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-payment-reconciliation, simple)
Call System.IDisposable.Dispose on object created by 'new SemaphoreSlim(1, 1)' before all references to it are out of scope. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-payment-reconciliation, simple)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-payment-reconciliation, simple)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, no-payment-reconciliation, simple)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (controlled, no-payment-reconciliation, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, no-payment-reconciliation, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, no-payment-reconciliation, simple)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, no-payment-reconciliation, simple)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (controlled, no-payment-reconciliation, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, no-payment-reconciliation, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, no-payment-reconciliation, simple)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, prepayment-always-required, simple)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1, actions/setup-node@v1, actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
tests (controlled, prepayment-always-required, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, prepayment-always-required, simple)
Avoid empty interfaces [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, prepayment-always-required, simple)
Exceptions should be public [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, prepayment-always-required, simple)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, prepayment-always-required, simple)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message, Exception innerException). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, prepayment-always-required, simple)
Call System.IDisposable.Dispose on object created by 'new SemaphoreSlim(1, 1)' before all references to it are out of scope. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, prepayment-always-required, simple)
Member CalculateIdempotencyKey does not access instance data and can be marked as static (Shared in VisualBasic) [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, prepayment-always-required, simple)
In externally visible method 'string IdempotencyStore.CalculateIdempotencyKey(OrderIdComponents orderId, string orderJson)', validate parameter 'orderId' is non-null before using it. If appropriate, throw an ArgumentNullException when the argument is null or add a Code Contract precondition asserting non-null argument. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, prepayment-always-required, simple)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, prepayment-always-required, simple)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, prepayment-always-required, simple)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (controlled, prepayment-always-required, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, prepayment-always-required, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, prepayment-always-required, simple)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, prepayment-always-required, simple)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (controlled, prepayment-always-required, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, prepayment-always-required, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, prepayment-always-required, simple)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, all-features, both)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1, actions/setup-node@v1, actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
tests (controlled, all-features, both)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (controlled, all-features, both)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, all-features, both)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, all-features, both)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, all-features, both)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (controlled, all-features, both)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, all-features, both)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, all-features, both)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, all-features, both)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, all-features, both)
Avoid empty interfaces [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, all-features, both)
Exceptions should be public [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, all-features, both)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, all-features, both)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message, Exception innerException). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, all-features, both)
Member CalculateIdempotencyKey does not access instance data and can be marked as static (Shared in VisualBasic) [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, all-features, both)
In externally visible method 'string IdempotencyStore.CalculateIdempotencyKey(OrderIdComponents orderId, string orderJson)', validate parameter 'orderId' is non-null before using it. If appropriate, throw an ArgumentNullException when the argument is null or add a Code Contract precondition asserting non-null argument. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, all-features, both)
Call System.IDisposable.Dispose on object created by 'new SemaphoreSlim(1, 1)' before all references to it are out of scope. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, all-features, both)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, all-features, both)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, prepayment-always-required, approval)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1, actions/setup-node@v1, actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
tests (controlled, prepayment-always-required, approval)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (controlled, prepayment-always-required, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, prepayment-always-required, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, prepayment-always-required, approval)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, prepayment-always-required, approval)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (controlled, prepayment-always-required, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, prepayment-always-required, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, prepayment-always-required, approval)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, prepayment-always-required, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, prepayment-always-required, approval)
Avoid empty interfaces [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, prepayment-always-required, approval)
Exceptions should be public [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, prepayment-always-required, approval)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, prepayment-always-required, approval)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message, Exception innerException). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, prepayment-always-required, approval)
Call System.IDisposable.Dispose on object created by 'new SemaphoreSlim(1, 1)' before all references to it are out of scope. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, prepayment-always-required, approval)
In externally visible method 'string IdempotencyStore.CalculateIdempotencyKey(OrderIdComponents orderId, string orderJson)', validate parameter 'orderId' is non-null before using it. If appropriate, throw an ArgumentNullException when the argument is null or add a Code Contract precondition asserting non-null argument. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, prepayment-always-required, approval)
Member CalculateIdempotencyKey does not access instance data and can be marked as static (Shared in VisualBasic) [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, prepayment-always-required, approval)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, prepayment-always-required, approval)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, single-seller, simple)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1, actions/setup-node@v1, actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
tests (controlled, single-seller, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, single-seller, simple)
Avoid empty interfaces [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, single-seller, simple)
Exceptions should be public [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, single-seller, simple)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, single-seller, simple)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message, Exception innerException). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, single-seller, simple)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, single-seller, simple)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, single-seller, simple)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, single-seller, simple)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, single-seller, simple)
In externally visible method 'Task<Order> StoreBookingEngine.ProcessOrderCreationFromOrderProposal(OrderIdComponents orderId, OrderIdTemplate orderIdTemplate, ILegalEntity seller, SimpleIdComponents sellerId, SimpleIdComponents customerAccountIdComponents, Order order)', validate parameter 'order' is non-null before using it. If appropriate, throw an ArgumentNullException when the argument is null or add a Code Contract precondition asserting non-null argument. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, single-seller, simple)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (controlled, single-seller, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, single-seller, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, single-seller, simple)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, single-seller, simple)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (controlled, single-seller, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, single-seller, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, single-seller, simple)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, single-seller, approval)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1, actions/setup-node@v1, actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
tests (controlled, single-seller, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, single-seller, approval)
Avoid empty interfaces [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, single-seller, approval)
Exceptions should be public [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, single-seller, approval)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, single-seller, approval)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message, Exception innerException). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, single-seller, approval)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, single-seller, approval)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, single-seller, approval)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, single-seller, approval)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, single-seller, approval)
In externally visible method 'Task<Order> StoreBookingEngine.ProcessOrderCreationFromOrderProposal(OrderIdComponents orderId, OrderIdTemplate orderIdTemplate, ILegalEntity seller, SimpleIdComponents sellerId, SimpleIdComponents customerAccountIdComponents, Order order)', validate parameter 'order' is non-null before using it. If appropriate, throw an ArgumentNullException when the argument is null or add a Code Contract precondition asserting non-null argument. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (controlled, single-seller, approval)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (controlled, single-seller, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, single-seller, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, single-seller, approval)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, single-seller, approval)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (controlled, single-seller, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, single-seller, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (controlled, single-seller, approval)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, facilityuse-has-slots, simple)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1, actions/setup-node@v1, actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
tests (random, facilityuse-has-slots, simple)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (random, facilityuse-has-slots, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, facilityuse-has-slots, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, facilityuse-has-slots, simple)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, facilityuse-has-slots, simple)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (random, facilityuse-has-slots, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, facilityuse-has-slots, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, facilityuse-has-slots, simple)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, facilityuse-has-slots, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, facilityuse-has-slots, simple)
Avoid empty interfaces [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, facilityuse-has-slots, simple)
Exceptions should be public [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, facilityuse-has-slots, simple)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, facilityuse-has-slots, simple)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message, Exception innerException). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, facilityuse-has-slots, simple)
Call System.IDisposable.Dispose on object created by 'new SemaphoreSlim(1, 1)' before all references to it are out of scope. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, facilityuse-has-slots, simple)
Member CalculateIdempotencyKey does not access instance data and can be marked as static (Shared in VisualBasic) [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, facilityuse-has-slots, simple)
In externally visible method 'string IdempotencyStore.CalculateIdempotencyKey(OrderIdComponents orderId, string orderJson)', validate parameter 'orderId' is non-null before using it. If appropriate, throw an ArgumentNullException when the argument is null or add a Code Contract precondition asserting non-null argument. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, facilityuse-has-slots, simple)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, facilityuse-has-slots, simple)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, all-features, simple)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1, actions/setup-node@v1, actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
tests (random, all-features, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, all-features, simple)
Avoid empty interfaces [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, all-features, simple)
Exceptions should be public [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, all-features, simple)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, all-features, simple)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message, Exception innerException). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, all-features, simple)
Member CalculateIdempotencyKey does not access instance data and can be marked as static (Shared in VisualBasic) [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, all-features, simple)
In externally visible method 'string IdempotencyStore.CalculateIdempotencyKey(OrderIdComponents orderId, string orderJson)', validate parameter 'orderId' is non-null before using it. If appropriate, throw an ArgumentNullException when the argument is null or add a Code Contract precondition asserting non-null argument. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, all-features, simple)
Call System.IDisposable.Dispose on object created by 'new SemaphoreSlim(1, 1)' before all references to it are out of scope. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, all-features, simple)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, all-features, simple)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, all-features, simple)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (random, all-features, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, all-features, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, all-features, simple)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, all-features, simple)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (random, all-features, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, all-features, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, all-features, simple)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, all-features, approval)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1, actions/setup-node@v1, actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
tests (random, all-features, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, all-features, approval)
Avoid empty interfaces [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, all-features, approval)
Exceptions should be public [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, all-features, approval)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, all-features, approval)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message, Exception innerException). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, all-features, approval)
In externally visible method 'string IdempotencyStore.CalculateIdempotencyKey(OrderIdComponents orderId, string orderJson)', validate parameter 'orderId' is non-null before using it. If appropriate, throw an ArgumentNullException when the argument is null or add a Code Contract precondition asserting non-null argument. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, all-features, approval)
Member CalculateIdempotencyKey does not access instance data and can be marked as static (Shared in VisualBasic) [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, all-features, approval)
Call System.IDisposable.Dispose on object created by 'new SemaphoreSlim(1, 1)' before all references to it are out of scope. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, all-features, approval)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, all-features, approval)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, all-features, approval)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (random, all-features, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, all-features, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, all-features, approval)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, all-features, approval)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (random, all-features, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, all-features, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, all-features, approval)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, no-auth, simple)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1, actions/setup-node@v1, actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
tests (random, no-auth, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-auth, simple)
Avoid empty interfaces [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-auth, simple)
Exceptions should be public [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-auth, simple)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-auth, simple)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message, Exception innerException). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-auth, simple)
Call System.IDisposable.Dispose on object created by 'new SemaphoreSlim(1, 1)' before all references to it are out of scope. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-auth, simple)
In externally visible method 'string IdempotencyStore.CalculateIdempotencyKey(OrderIdComponents orderId, string orderJson)', validate parameter 'orderId' is non-null before using it. If appropriate, throw an ArgumentNullException when the argument is null or add a Code Contract precondition asserting non-null argument. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-auth, simple)
Member CalculateIdempotencyKey does not access instance data and can be marked as static (Shared in VisualBasic) [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-auth, simple)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-auth, simple)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-auth, simple)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (random, no-auth, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, no-auth, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, no-auth, simple)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, no-auth, simple)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (random, no-auth, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, no-auth, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, no-auth, simple)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, no-payment-reconciliation, simple)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1, actions/setup-node@v1, actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
tests (random, no-payment-reconciliation, simple)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (random, no-payment-reconciliation, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, no-payment-reconciliation, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, no-payment-reconciliation, simple)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, no-payment-reconciliation, simple)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (random, no-payment-reconciliation, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, no-payment-reconciliation, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, no-payment-reconciliation, simple)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, no-payment-reconciliation, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-payment-reconciliation, simple)
Avoid empty interfaces [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-payment-reconciliation, simple)
Exceptions should be public [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-payment-reconciliation, simple)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-payment-reconciliation, simple)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message, Exception innerException). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-payment-reconciliation, simple)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-payment-reconciliation, simple)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-payment-reconciliation, simple)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-payment-reconciliation, simple)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-payment-reconciliation, simple)
In externally visible method 'Task<Order> StoreBookingEngine.ProcessOrderCreationFromOrderProposal(OrderIdComponents orderId, OrderIdTemplate orderIdTemplate, ILegalEntity seller, SimpleIdComponents sellerId, SimpleIdComponents customerAccountIdComponents, Order order)', validate parameter 'order' is non-null before using it. If appropriate, throw an ArgumentNullException when the argument is null or add a Code Contract precondition asserting non-null argument. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, prepayment-always-required, approval)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1, actions/setup-node@v1, actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
tests (random, prepayment-always-required, approval)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (random, prepayment-always-required, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, prepayment-always-required, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, prepayment-always-required, approval)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, prepayment-always-required, approval)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (random, prepayment-always-required, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, prepayment-always-required, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, prepayment-always-required, approval)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, prepayment-always-required, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, prepayment-always-required, approval)
Avoid empty interfaces [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, prepayment-always-required, approval)
Exceptions should be public [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, prepayment-always-required, approval)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, prepayment-always-required, approval)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message, Exception innerException). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, prepayment-always-required, approval)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, prepayment-always-required, approval)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, prepayment-always-required, approval)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, prepayment-always-required, approval)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, prepayment-always-required, approval)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, prepayment-always-required, simple)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1, actions/setup-node@v1, actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
tests (random, prepayment-always-required, simple)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (random, prepayment-always-required, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, prepayment-always-required, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, prepayment-always-required, simple)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, prepayment-always-required, simple)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (random, prepayment-always-required, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, prepayment-always-required, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, prepayment-always-required, simple)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, prepayment-always-required, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, prepayment-always-required, simple)
Avoid empty interfaces [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, prepayment-always-required, simple)
Exceptions should be public [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, prepayment-always-required, simple)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, prepayment-always-required, simple)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message, Exception innerException). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, prepayment-always-required, simple)
Call System.IDisposable.Dispose on object created by 'new SemaphoreSlim(1, 1)' before all references to it are out of scope. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, prepayment-always-required, simple)
In externally visible method 'string IdempotencyStore.CalculateIdempotencyKey(OrderIdComponents orderId, string orderJson)', validate parameter 'orderId' is non-null before using it. If appropriate, throw an ArgumentNullException when the argument is null or add a Code Contract precondition asserting non-null argument. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, prepayment-always-required, simple)
Member CalculateIdempotencyKey does not access instance data and can be marked as static (Shared in VisualBasic) [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, prepayment-always-required, simple)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, prepayment-always-required, simple)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-payment-reconciliation, approval)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1, actions/setup-node@v1, actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
tests (random, no-payment-reconciliation, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-payment-reconciliation, approval)
Avoid empty interfaces [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-payment-reconciliation, approval)
Exceptions should be public [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-payment-reconciliation, approval)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-payment-reconciliation, approval)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message, Exception innerException). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-payment-reconciliation, approval)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-payment-reconciliation, approval)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-payment-reconciliation, approval)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-payment-reconciliation, approval)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-payment-reconciliation, approval)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-payment-reconciliation, approval)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (random, no-payment-reconciliation, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, no-payment-reconciliation, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, no-payment-reconciliation, approval)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, no-payment-reconciliation, approval)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (random, no-payment-reconciliation, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, no-payment-reconciliation, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, no-payment-reconciliation, approval)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, no-tax-calculation, approval)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1, actions/setup-node@v1, actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
tests (random, no-tax-calculation, approval)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (random, no-tax-calculation, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, no-tax-calculation, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, no-tax-calculation, approval)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, no-tax-calculation, approval)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (random, no-tax-calculation, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, no-tax-calculation, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, no-tax-calculation, approval)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, no-tax-calculation, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-tax-calculation, approval)
Avoid empty interfaces [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-tax-calculation, approval)
Exceptions should be public [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-tax-calculation, approval)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-tax-calculation, approval)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message, Exception innerException). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-tax-calculation, approval)
In externally visible method 'string IdempotencyStore.CalculateIdempotencyKey(OrderIdComponents orderId, string orderJson)', validate parameter 'orderId' is non-null before using it. If appropriate, throw an ArgumentNullException when the argument is null or add a Code Contract precondition asserting non-null argument. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-tax-calculation, approval)
Member CalculateIdempotencyKey does not access instance data and can be marked as static (Shared in VisualBasic) [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-tax-calculation, approval)
Call System.IDisposable.Dispose on object created by 'new SemaphoreSlim(1, 1)' before all references to it are out of scope. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-tax-calculation, approval)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-tax-calculation, approval)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, facilityuse-has-slots, approval)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1, actions/setup-node@v1, actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
tests (random, facilityuse-has-slots, approval)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (random, facilityuse-has-slots, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, facilityuse-has-slots, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, facilityuse-has-slots, approval)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, facilityuse-has-slots, approval)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (random, facilityuse-has-slots, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, facilityuse-has-slots, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, facilityuse-has-slots, approval)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, facilityuse-has-slots, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, facilityuse-has-slots, approval)
Avoid empty interfaces [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, facilityuse-has-slots, approval)
Exceptions should be public [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, facilityuse-has-slots, approval)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, facilityuse-has-slots, approval)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message, Exception innerException). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, facilityuse-has-slots, approval)
Call System.IDisposable.Dispose on object created by 'new SemaphoreSlim(1, 1)' before all references to it are out of scope. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, facilityuse-has-slots, approval)
In externally visible method 'string IdempotencyStore.CalculateIdempotencyKey(OrderIdComponents orderId, string orderJson)', validate parameter 'orderId' is non-null before using it. If appropriate, throw an ArgumentNullException when the argument is null or add a Code Contract precondition asserting non-null argument. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, facilityuse-has-slots, approval)
Member CalculateIdempotencyKey does not access instance data and can be marked as static (Shared in VisualBasic) [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, facilityuse-has-slots, approval)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, facilityuse-has-slots, approval)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, single-seller, approval)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1, actions/setup-node@v1, actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
tests (random, single-seller, approval)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (random, single-seller, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, single-seller, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, single-seller, approval)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, single-seller, approval)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (random, single-seller, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, single-seller, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, single-seller, approval)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, single-seller, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, single-seller, approval)
Avoid empty interfaces [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, single-seller, approval)
Exceptions should be public [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, single-seller, approval)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, single-seller, approval)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message, Exception innerException). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, single-seller, approval)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, single-seller, approval)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, single-seller, approval)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, single-seller, approval)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, single-seller, approval)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-auth, approval)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1, actions/setup-node@v1, actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
tests (random, no-auth, approval)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (random, no-auth, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, no-auth, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, no-auth, approval)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, no-auth, approval)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (random, no-auth, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, no-auth, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, no-auth, approval)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, no-auth, approval)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-auth, approval)
Avoid empty interfaces [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-auth, approval)
Exceptions should be public [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-auth, approval)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-auth, approval)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message, Exception innerException). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-auth, approval)
Call System.IDisposable.Dispose on object created by 'new SemaphoreSlim(1, 1)' before all references to it are out of scope. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-auth, approval)
In externally visible method 'string IdempotencyStore.CalculateIdempotencyKey(OrderIdComponents orderId, string orderJson)', validate parameter 'orderId' is non-null before using it. If appropriate, throw an ArgumentNullException when the argument is null or add a Code Contract precondition asserting non-null argument. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-auth, approval)
Member CalculateIdempotencyKey does not access instance data and can be marked as static (Shared in VisualBasic) [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-auth, approval)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-auth, approval)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-tax-calculation, simple)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1, actions/setup-node@v1, actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
tests (random, no-tax-calculation, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-tax-calculation, simple)
Avoid empty interfaces [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-tax-calculation, simple)
Exceptions should be public [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-tax-calculation, simple)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-tax-calculation, simple)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message, Exception innerException). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-tax-calculation, simple)
Call System.IDisposable.Dispose on object created by 'new SemaphoreSlim(1, 1)' before all references to it are out of scope. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-tax-calculation, simple)
In externally visible method 'string IdempotencyStore.CalculateIdempotencyKey(OrderIdComponents orderId, string orderJson)', validate parameter 'orderId' is non-null before using it. If appropriate, throw an ArgumentNullException when the argument is null or add a Code Contract precondition asserting non-null argument. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-tax-calculation, simple)
Member CalculateIdempotencyKey does not access instance data and can be marked as static (Shared in VisualBasic) [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-tax-calculation, simple)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-tax-calculation, simple)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, no-tax-calculation, simple)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (random, no-tax-calculation, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, no-tax-calculation, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, no-tax-calculation, simple)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, no-tax-calculation, simple)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (random, no-tax-calculation, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, no-tax-calculation, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, no-tax-calculation, simple)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, single-seller, simple)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1, actions/setup-node@v1, actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
tests (random, single-seller, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, single-seller, simple)
Avoid empty interfaces [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, single-seller, simple)
Exceptions should be public [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, single-seller, simple)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, single-seller, simple)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message, Exception innerException). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, single-seller, simple)
Call System.IDisposable.Dispose on object created by 'new SemaphoreSlim(1, 1)' before all references to it are out of scope. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, single-seller, simple)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, single-seller, simple)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, single-seller, simple)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, single-seller, simple)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, single-seller, simple)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (random, single-seller, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, single-seller, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, single-seller, simple)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, single-seller, simple)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (random, single-seller, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, single-seller, simple)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, single-seller, simple)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, all-features, both)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1, actions/setup-node@v1, actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
tests (random, all-features, both)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, all-features, both)
Avoid empty interfaces [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, all-features, both)
Exceptions should be public [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, all-features, both)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, all-features, both)
Add the following constructor to SilentRollbackException: public SilentRollbackException(string message, Exception innerException). [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, all-features, both)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, all-features, both)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, all-features, both)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, all-features, both)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, all-features, both)
Consider calling ConfigureAwait on the awaited task [/home/runner/work/openactive-test-suite/openactive-test-suite/server/OpenActive.Server.NET/OpenActive.Server.NET.csproj]
tests (random, all-features, both)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (random, all-features, both)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, all-features, both)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, all-features, both)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, all-features, both)
The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Fakes/OpenActive.FakeDatabase.NET/OpenActive.FakeDatabase.NET.csproj]
tests (random, all-features, both)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, all-features, both)
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]
tests (random, all-features, both)
Unreachable code detected [/home/runner/work/openactive-test-suite/openactive-test-suite/server/Examples/BookingSystem.AspNetCore.IdentityServer/BookingSystem.AspNetCore.IdentityServer.csproj]

Artifacts

Produced during runtime
Name Size
controlled.all-features.approval Expired
191 MB
controlled.all-features.both Expired
349 MB
controlled.all-features.simple Expired
958 KB
controlled.facilityuse-has-slots.approval Expired
178 MB
controlled.facilityuse-has-slots.simple Expired
149 MB
controlled.no-auth.approval Expired
185 MB
controlled.no-auth.simple Expired
155 MB
controlled.no-payment-reconciliation.approval Expired
187 MB
controlled.no-payment-reconciliation.simple Expired
155 MB
controlled.no-tax-calculation.approval Expired
181 MB
controlled.no-tax-calculation.simple Expired
151 MB
controlled.prepayment-always-required.approval Expired
178 MB
controlled.prepayment-always-required.simple Expired
148 MB
controlled.single-seller.approval Expired
179 MB
controlled.single-seller.simple Expired
150 MB
random.all-features.approval Expired
191 MB
random.all-features.both Expired
351 MB
random.all-features.simple Expired
160 MB
random.facilityuse-has-slots.approval Expired
180 MB
random.facilityuse-has-slots.simple Expired
150 MB
random.no-auth.approval Expired
187 MB
random.no-auth.simple Expired
157 MB
random.no-payment-reconciliation.approval Expired
188 MB
random.no-payment-reconciliation.simple Expired
156 MB
random.no-tax-calculation.approval Expired
182 MB
random.no-tax-calculation.simple Expired
151 MB
random.prepayment-always-required.approval Expired
179 MB
random.prepayment-always-required.simple Expired
148 MB
random.single-seller.approval Expired
182 MB
random.single-seller.simple Expired
152 MB