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

VSTest alias documentation contains holdover from <v0.17.0 #3355

Closed
2 tasks done
ghost opened this issue May 17, 2021 · 2 comments · Fixed by #3356
Closed
2 tasks done

VSTest alias documentation contains holdover from <v0.17.0 #3355

ghost opened this issue May 17, 2021 · 2 comments · Fixed by #3356
Assignees
Milestone

Comments

@ghost
Copy link

ghost commented May 17, 2021

Prerequisites

  • I have written a descriptive issue title
  • I have searched issues to ensure it has not already been reported

Cake runner

Cake runner for .NET Framework

Cake version

1.1.0

Operating system

Windows

Operating system architecture

64-Bit

CI Server

No response

What are you seeing?

The documentation of the VSTest alias still contains VSTestLogger.Trx which was removed in v0.17.0 ( #1138).

/// <summary>
/// Runs all VSTest unit tests in the assemblies matching the specified pattern.
/// </summary>
/// <example>
/// <code>
/// VSTest("./Tests/*.UnitTests.dll", new VSTestSettings() { Logger = VSTestLogger.Trx });
/// </code>
/// </example>
/// <param name="context">The context.</param>
/// <param name="pattern">The pattern.</param>
/// <param name="settings">The settings.</param>
[CakeMethodAlias]
public static void VSTest(this ICakeContext context, GlobPattern pattern, VSTestSettings settings)

What is expected?

/// VSTest("./Tests/*.UnitTests.dll", new VSTestSettings() { Logger = "trx" });

Steps to Reproduce

na

Output log

No response

@himanshu007-creator
Copy link
Contributor

HI, i would like to work on this issue.

@cake-build-bot
Copy link

🎉 This issue has been resolved in version v1.2.0 🎉

The release is available on:

Your GitReleaseManager bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants