Skip to content

Make tests run on all platforms #1492

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

Merged

Conversation

0xced
Copy link
Contributor

@0xced 0xced commented Jun 11, 2023

An extra space in the "coverlet " path and a platform specific directory separator char were the culprits for failing the DotnetGlobalTools tests on Linux and macOS.

An extra space in the "coverlet " path and a platform specific directory separator char were the culprits for failing tests on Linux and macOS.
@0xced 0xced force-pushed the XPlatform-DotnetGlobalTools branch from 249e828 to 8bcff84 Compare June 11, 2023 21:52
@daveMueller
Copy link
Collaborator

Cool 😲, thanks a lot @0xced 🙏

@0xced 0xced changed the title Make the DotnetGlobalTools tests run on all platforms Make tests run on all platforms Jun 12, 2023
@MarcoRossignoli
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

No commit pushedDate could be found for PR 1492 in repo coverlet-coverage/coverlet

@MarcoRossignoli MarcoRossignoli merged commit 95b5cdf into coverlet-coverage:master Jul 23, 2023
MarcoRossignoli pushed a commit that referenced this pull request Aug 3, 2023
* use System.CommandLine for coverlet.console

* resolve review comments and add DotnetTool tests

* add space between CLI argument (#1499)

matches the same argument further down

* Fix resolving assemblies from frameworks not referenced by coverlet itself (#1449)

* Make tests run on all platforms (#1492)

* Integration test for assembly resolver for mvc.razor (#1502)

* Update src/coverlet.console/Program.cs

Co-authored-by: David Müller <muellerdavid4@gmail.com>

* Update src/coverlet.console/Program.cs

Co-authored-by: David Müller <muellerdavid4@gmail.com>

* Update src/coverlet.console/Program.cs

Co-authored-by: David Müller <muellerdavid4@gmail.com>

* Update src/coverlet.console/Program.cs

Co-authored-by: David Müller <muellerdavid4@gmail.com>

* Update src/coverlet.console/Program.cs

Co-authored-by: David Müller <muellerdavid4@gmail.com>

* Update src/coverlet.console/Program.cs

Co-authored-by: David Müller <muellerdavid4@gmail.com>

* Update src/coverlet.console/Program.cs

Co-authored-by: David Müller <muellerdavid4@gmail.com>

* Update src/coverlet.console/Program.cs

Co-authored-by: David Müller <muellerdavid4@gmail.com>

* Update src/coverlet.console/Program.cs

Co-authored-by: David Müller <muellerdavid4@gmail.com>

* remove spelling configuration

* use System.CommandLine for coverlet.console

* resolve review comments and add DotnetTool tests

* Update src/coverlet.console/Program.cs

Co-authored-by: David Müller <muellerdavid4@gmail.com>

* Update src/coverlet.console/Program.cs

Co-authored-by: David Müller <muellerdavid4@gmail.com>

* Update src/coverlet.console/Program.cs

Co-authored-by: David Müller <muellerdavid4@gmail.com>

* Update src/coverlet.console/Program.cs

Co-authored-by: David Müller <muellerdavid4@gmail.com>

* Update src/coverlet.console/Program.cs

Co-authored-by: David Müller <muellerdavid4@gmail.com>

* Update src/coverlet.console/Program.cs

Co-authored-by: David Müller <muellerdavid4@gmail.com>

* Update src/coverlet.console/Program.cs

Co-authored-by: David Müller <muellerdavid4@gmail.com>

* Update src/coverlet.console/Program.cs

Co-authored-by: David Müller <muellerdavid4@gmail.com>

* Update src/coverlet.console/Program.cs

Co-authored-by: David Müller <muellerdavid4@gmail.com>

* remove spelling configuration

---------

Co-authored-by: David Markowitz <39972741+EmosewaMC@users.noreply.github.com>
Co-authored-by: Lukas Grützmacher <44983012+lg2de@users.noreply.github.com>
Co-authored-by: Cédric Luthi <cedric.luthi@gmail.com>
Co-authored-by: David Müller <muellerdavid4@gmail.com>
@0xced 0xced deleted the XPlatform-DotnetGlobalTools branch August 8, 2023 21:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants