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

Create true integration tests for Cake.Warp aliases #15

Open
AdmiringWorm opened this issue May 12, 2019 · 4 comments
Open

Create true integration tests for Cake.Warp aliases #15

AdmiringWorm opened this issue May 12, 2019 · 4 comments
Labels
Build Build related only change help wanted Extra attention is needed

Comments

@AdmiringWorm
Copy link
Member

Is your feature related to a problem? Please describe

The problem is that there currently is no integration tests that actually
checks if the cake aliases work as they should.

Desribe the solution you'd like

The ideal solution would be to run Cake itself against a dynamically created script
in the Cake.Warp.IntegrationTests library.
This would need to be able to test both during .NET/Mono version of Cake, as well as against the .NET Core edition of Cake.

Describe alternatives you've considered

Considered calling the aliases directly, but this would mean we needed to mock out some of the
classes needed for it to work. This would then mean it isn't a true integration test.

Additional context

@AdmiringWorm AdmiringWorm added Feature New non-breaking functionality help wanted Extra attention is needed labels May 12, 2019
@AdmiringWorm AdmiringWorm pinned this issue May 13, 2019
@AdmiringWorm AdmiringWorm added Build Build related only change and removed Feature New non-breaking functionality labels May 30, 2019
@AdmiringWorm
Copy link
Member Author

hmm, maybe using cake.frosting could be an option.
Something I would definitely need to consider.

@gep13
Copy link
Member

gep13 commented Jul 26, 2020

Have you seen the integration tests that @pascalberger has just added to Cake.Issues.Reporting.Generic?

@AdmiringWorm
Copy link
Member Author

I have now, but that wouldn't work as I want it.

Even for integration tests, I would still want these to be reported whether they are covered on coverage reports.
This doesn't seem to be available with the integration tests/demos that were added in that repo.

@gep13
Copy link
Member

gep13 commented Jul 26, 2020

Ah, I see.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Build Build related only change help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants