-
-
Notifications
You must be signed in to change notification settings - Fork 163
Request meta tests boilerplate #1746
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
mrmarcosmagalhaes
wants to merge
57
commits into
json-api-dotnet:master
Choose a base branch
from
mrmarcosmagalhaes:request-meta-tests-boilerplate
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 45 commits
Commits
Show all changes
57 commits
Select commit
Hold shift + click to select a range
cd9fa50
Basic boilerplate for writing meta tests
bkoelman f0208a0
test: accepts top level meta in POST resource request
mrmarcosmagalhaes 11a6580
test: accepts top level meta in PATCH relationship request
mrmarcosmagalhaes d34319e
test: accepts top level meta in POST relationship request
mrmarcosmagalhaes b68dd8f
test: accepts top level meta in DELETE relationship request
mrmarcosmagalhaes 1498a0b
test: accepts top level meta in atomic UPDATE resource request
mrmarcosmagalhaes eeebcd9
test: accepts top level meta in atomic ADD resource request
mrmarcosmagalhaes c33e9db
test: accepts top level meta in atomic REMOVE resource request
mrmarcosmagalhaes 6bf58d3
test: accepts meta in data of POST resource request
mrmarcosmagalhaes 7c768eb
test: accepts meta in relationship of POST resource request
mrmarcosmagalhaes b083bd2
test: accepts meta in data of atomic ADD resource request
mrmarcosmagalhaes b05bb8f
test: accepts meta in relationship of atomic UPDATE resource request
mrmarcosmagalhaes fd5d2bd
refactor after execute inspect code script
mrmarcosmagalhaes 6238c91
remove code style warnings
mrmarcosmagalhaes f8d646c
Merge branch 'master' into request-meta-tests-boilerplate
bkoelman 062c0bc
Rune code cleanup
bkoelman 3605490
Fix inspections
bkoelman 8dc82d3
Merge branch 'master' into request-meta-tests-boilerplate
mrmarcosmagalhaes e49e5dd
Align request meta integration tests with JSON:API requirements
mrmarcosmagalhaes 26502ea
fix some code styles
mrmarcosmagalhaes 377d9bf
fix missing code styles warnings
mrmarcosmagalhaes 1de1666
Fix clean up code warnings
mrmarcosmagalhaes 7164452
Fix inspect code warnings
mrmarcosmagalhaes 03aeac8
Fix clean up warnings
mrmarcosmagalhaes 481fd15
Merge branch 'master' into request-meta-tests-boilerplate
mrmarcosmagalhaes 19953bd
Add meta fakers and refactor
mrmarcosmagalhaes 806e475
Merge branch 'request-meta-tests-boilerplate' of https://github.com/m…
mrmarcosmagalhaes 2dd1daf
Add post resource request with to many relationships
mrmarcosmagalhaes b4823d7
Add relationship endpoints tests
mrmarcosmagalhaes cacd9c6
Isolate atomic operations request tests
mrmarcosmagalhaes 607ca90
Add missing meta data coverage
mrmarcosmagalhaes 925989f
Resolve code cleanup warnings
mrmarcosmagalhaes 96174ad
Merge branch 'master' into request-meta-tests-boilerplate
mrmarcosmagalhaes 5c540b5
Resolve remaining code cleanup warnings
mrmarcosmagalhaes bcc248b
Merge branch 'request-meta-tests-boilerplate' of https://github.com/m…
mrmarcosmagalhaes 0e069ff
Merge branch 'master' into request-meta-tests-boilerplate
mrmarcosmagalhaes 63754fd
Fix build
mrmarcosmagalhaes bb6ba33
Merge branch 'request-meta-tests-boilerplate' of https://github.com/m…
mrmarcosmagalhaes b5e99a6
fix warnings
mrmarcosmagalhaes 5ee4d79
Add test to check meta in operation level
mrmarcosmagalhaes 5b44ea4
Revert "Add test to check meta in operation level"
mrmarcosmagalhaes 0c6328f
Add test to check meta in operation level
mrmarcosmagalhaes 8c306ef
Fix compilation error
mrmarcosmagalhaes 35e6533
fix warnings
mrmarcosmagalhaes fdac076
fix cleanup-code task
mrmarcosmagalhaes 539382b
Pushed updates addressing review comments.
mrmarcosmagalhaes 689405b
Pushed updates addressing missing review comments.
mrmarcosmagalhaes a133a32
Fix failed code inspection
mrmarcosmagalhaes e430672
Merge branch 'master' into request-meta-tests-boilerplate
mrmarcosmagalhaes b072dd5
Pushed updates addressing review comments.
mrmarcosmagalhaes 2cb2e37
Merge branch 'request-meta-tests-boilerplate' of https://github.com/m…
mrmarcosmagalhaes 845d4ad
Pushed updates addressing missing review comments.
mrmarcosmagalhaes 45dc467
Pushed updates addressing review comments.
mrmarcosmagalhaes f9bd02f
Fix compilation error.
mrmarcosmagalhaes db6c3e8
Fix cleanup code errors.
mrmarcosmagalhaes 7ddd949
Fix build errors.
mrmarcosmagalhaes a4628ae
Fix build errors.
mrmarcosmagalhaes File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
13 changes: 13 additions & 0 deletions
13
test/JsonApiDotNetCoreTests/IntegrationTests/Meta/OperationsController.cs
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,13 @@ | ||
| using JsonApiDotNetCore.AtomicOperations; | ||
| using JsonApiDotNetCore.Configuration; | ||
| using JsonApiDotNetCore.Controllers; | ||
| using JsonApiDotNetCore.Middleware; | ||
| using JsonApiDotNetCore.Resources; | ||
| using Microsoft.Extensions.Logging; | ||
|
|
||
| namespace JsonApiDotNetCoreTests.IntegrationTests.Meta; | ||
|
|
||
| public sealed class OperationsController( | ||
| IJsonApiOptions options, IResourceGraph resourceGraph, ILoggerFactory loggerFactory, IOperationsProcessor processor, IJsonApiRequest request, | ||
| ITargetedFields targetedFields, IAtomicOperationFilter operationFilter) | ||
| : JsonApiOperationsController(options, resourceGraph, loggerFactory, processor, request, targetedFields, operationFilter); |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.