Skip to content

Conversation

@sungam3r
Copy link
Member

No description provided.

@sungam3r sungam3r requested a review from Shane32 December 28, 2021 22:01
@sungam3r sungam3r self-assigned this Dec 28, 2021
@sungam3r sungam3r added bugfix Pull request that fixes a bug dependencies Pull requests that update a dependency file labels Dec 28, 2021
@github-actions github-actions bot added the test Pull request that adds new or changes existing tests label Dec 28, 2021
@sungam3r sungam3r added this to the 5.0 milestone Dec 28, 2021
@sungam3r sungam3r removed the test Pull request that adds new or changes existing tests label Dec 28, 2021
@github-actions github-actions bot added CI CI configuration issue or pull request test Pull request that adds new or changes existing tests labels Dec 28, 2021
@codecov-commenter
Copy link

Codecov Report

Merging #191 (78e2731) into develop (02ed319) will decrease coverage by 3.25%.
The diff coverage is 71.23%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #191      +/-   ##
===========================================
- Coverage    83.60%   80.34%   -3.26%     
===========================================
  Files            9       10       +1     
  Lines          250      290      +40     
  Branches        41       46       +5     
===========================================
+ Hits           209      233      +24     
- Misses          32       45      +13     
- Partials         9       12       +3     
Impacted Files Coverage Δ
.../GraphQL.Authorization/GraphQLBuilderExtensions.cs 0.00% <0.00%> (ø)
...ation/Requirements/AuthenticatedUserRequirement.cs 75.00% <0.00%> (ø)
src/GraphQL.Authorization/AuthorizationSettings.cs 33.33% <50.00%> (ø)
...tion/Requirements/ClaimAuthorizationRequirement.cs 86.95% <81.25%> (+0.91%) ⬆️
...aphQL.Authorization/AuthorizationValidationRule.cs 94.95% <87.50%> (-2.92%) ⬇️
src/GraphQL.Authorization/AuthorizationContext.cs 83.33% <100.00%> (ø)
...rc/GraphQL.Authorization/AuthorizationEvaluator.cs 100.00% <100.00%> (ø)
src/GraphQL.Authorization/AuthorizationPolicy.cs 85.71% <100.00%> (ø)
...raphQL.Authorization/AuthorizationPolicyBuilder.cs 45.45% <100.00%> (ø)
src/GraphQL.Authorization/AuthorizationResult.cs 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ee291f1...78e2731. Read the comment docs.

Copy link
Member

@Shane32 Shane32 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think, either in this PR or another, we should also include the input extensions

public System.Collections.Generic.IReadOnlyDictionary<string, object>? Inputs { get; set; }
public System.Security.Claims.ClaimsPrincipal? User { get; set; }
public System.Collections.Generic.IDictionary<string, object?>? UserContext { get; set; }
public GraphQL.Inputs? Variables { get; set; }
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Needs extensions also?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IDocumentValidator has no access to extensions. Another PR.

@sungam3r sungam3r merged commit d04b29f into develop Jan 1, 2022
@sungam3r sungam3r deleted the fix-develop branch January 1, 2022 13:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix Pull request that fixes a bug CI CI configuration issue or pull request dependencies Pull requests that update a dependency file test Pull request that adds new or changes existing tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants