Skip to content
This repository has been archived by the owner on Apr 30, 2022. It is now read-only.

SoftAssert Aggregate Exceptions #588

Merged
merged 3 commits into from
Jan 11, 2021
Merged

Conversation

skalpin
Copy link
Contributor

@skalpin skalpin commented Dec 15, 2020

Including all exceptions thrown in SoftAsserts in the Aggregate
exception that is thrown on failure.

issue #562

Including all exceptions thrown in SoftAsserts in the Aggregate
exception that is thrown on failure.

issue #562
@skalpin skalpin requested a review from a team as a code owner December 15, 2020 02:52
Copy link
Contributor

@TroyWalshProf TroyWalshProf left a comment

Choose a reason for hiding this comment

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

Would use MicroAssert using in a couple of places, but otherwise like it.

Framework/BaseTestUnitTests/BaseFrameworkTests.cs Outdated Show resolved Hide resolved
Framework/BaseTestUnitTests/BaseFrameworkTests.cs Outdated Show resolved Hide resolved
Framework/BaseTestUnitTests/BaseFrameworkTests.cs Outdated Show resolved Hide resolved
@sonarcloud
Copy link

sonarcloud bot commented Jan 10, 2021

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

88.9% 88.9% Coverage
0.0% 0.0% Duplication

@skalpin skalpin merged commit be55af3 into master Jan 11, 2021
@skalpin skalpin deleted the issue/562-RemoveListOfExceptions branch January 11, 2021 14:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants