Skip to content
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

Bump the xunit group with 1 update #1769

Merged
merged 5 commits into from
Nov 2, 2023

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Nov 2, 2023

Bumps the xunit group with 1 update: xunit.

Commits
  • 6a07b95 v2.6.0
  • 5a7297e #2806: Performance improvements in Assert.Equal (v2)
  • a6b5cc4 Add net6.0 binary to xunit.assert
  • abc1dd3 Update Microsoft.NETCore.App.Ref to 6.0
  • 52b6638 Move CA1304 and CA1305 enablement to production code only (not test code)
  • e139d77 Ensure all throws of NotImplementedException contain messages
  • 8a84fdb Ensure all GetHashCode functions are implemented (#2804) (v2)
  • c853205 Unit tests for #2803 (v2)
  • 8ffe56a Unit tests for #2800 (v2)
  • b2e660e Replace (most) concatenation and interpolation (in shipped code) with String....
  • Additional commits viewable in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore <dependency name> major version will close this group update PR and stop Dependabot creating any more for the specific dependency's major version (unless you unignore this specific dependency's major version or upgrade to it yourself)
  • @dependabot ignore <dependency name> minor version will close this group update PR and stop Dependabot creating any more for the specific dependency's minor version (unless you unignore this specific dependency's minor version or upgrade to it yourself)
  • @dependabot ignore <dependency name> will close this group update PR and stop Dependabot creating any more for the specific dependency (unless you unignore this specific dependency or upgrade to it yourself)
  • @dependabot unignore <dependency name> will remove all of the ignore conditions of the specified dependency
  • @dependabot unignore <dependency name> <ignore condition> will remove the ignore condition of the specified dependency and ignore conditions

Bumps the xunit group with 1 update: [xunit](https://github.com/xunit/xunit).

- [Commits](xunit/xunit@2.5.3...2.6.0)

---
updated-dependencies:
- dependency-name: xunit
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: xunit
...

Signed-off-by: dependabot[bot] <[email protected]>
@dependabot dependabot bot added .NET Pull requests that update .NET code dependencies Pull requests that update a dependency file labels Nov 2, 2023
@polly-reviewer-bot polly-reviewer-bot bot enabled auto-merge (squash) November 2, 2023 16:39
Fix CS0121 errors caused by ambiguity between `Task` and `ValueTask`.
Fix compiler errors again by forcing the Func<Task> overload to be used.
Suppress xUnit1031 warnings that are OK because we explicitly want to wait for the operation to not succeed.
Rename method and drop type argument.
Copy link

codecov bot commented Nov 2, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (c5d268d) 84.53% compared to head (7b33b29) 84.53%.
Report is 5 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1769   +/-   ##
=======================================
  Coverage   84.53%   84.53%           
=======================================
  Files         307      307           
  Lines        6777     6777           
  Branches     1043     1043           
=======================================
  Hits         5729     5729           
  Misses        839      839           
  Partials      209      209           
Flag Coverage Δ
linux 84.53% <ø> (?)
macos 84.53% <ø> (?)
windows ?

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@polly-reviewer-bot polly-reviewer-bot bot merged commit 9e86a0a into main Nov 2, 2023
14 checks passed
@polly-reviewer-bot polly-reviewer-bot bot deleted the dependabot/nuget/xunit-7cf140af72 branch November 2, 2023 18:23
Comment on lines +108 to +117
// xunit 2.6.0 has Func<ValueTask> and Func<Task> overloads that cause ambiguity which
// is messy to resolve for multiple different TFMs, so force the Task version to be used.
static async Task AssertThrowsObjectDisposedExceptionAsync(Func<Task> func)
=> await Assert.ThrowsAsync<ObjectDisposedException>(func);

await AssertThrowsObjectDisposedExceptionAsync(async () => await controller.CloseCircuitAsync(ResilienceContextPool.Shared.Get()));
await AssertThrowsObjectDisposedExceptionAsync(async () => await controller.IsolateCircuitAsync(ResilienceContextPool.Shared.Get()));
await AssertThrowsObjectDisposedExceptionAsync(async () => await controller.OnActionPreExecuteAsync(ResilienceContextPool.Shared.Get()));
await AssertThrowsObjectDisposedExceptionAsync(async () => await controller.OnActionSuccessAsync(Outcome.FromResult(10), ResilienceContextPool.Shared.Get()));
await AssertThrowsObjectDisposedExceptionAsync(async () => await controller.OnActionFailureAsync(Outcome.FromResult(10), ResilienceContextPool.Shared.Get()));
Copy link
Member

Choose a reason for hiding this comment

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

Reverted in #1774.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file .NET Pull requests that update .NET code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant