You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Assert.AreEqual(aggregator.RunContextAttachments.Count, 1, "RunContextAttachments List must have data.");
should be
Assert.AreEqual(1, aggregator.RunContextAttachments.Count, "RunContextAttachments List must have data.");
I would like to fix these if i'm allowed.
The text was updated successfully, but these errors were encountered:
HeroMaxPower
changed the title
Some Asserts.AreEqual in TestPlatform.sln have incorrect expected and actual parameters.
Some Asserts.AreEqual in TestPlatform.sln have incorrect expected and actual arguments.
Feb 14, 2020
@HeroMaxPower thanks for the report, you are definitely allowed to PR this. The inconsistency among different testing frameworks probably caused this. I personally always have to double-check which of the params is expected in the assertion library the framework is using 🙂
Im using static code analyser SonarLint for checking this project and i've come across a violation of this rule
SonarLint Rule 3415
in some of the tests for this project.
e.g.
Assert.AreEqual(aggregator.RunContextAttachments.Count, 1, "RunContextAttachments List must have data.");
should be
Assert.AreEqual(1, aggregator.RunContextAttachments.Count, "RunContextAttachments List must have data.");
I would like to fix these if i'm allowed.
The text was updated successfully, but these errors were encountered: