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

Spike black box tests #726

Closed
HEskandari opened this issue May 12, 2017 · 2 comments
Closed

Spike black box tests #726

HEskandari opened this issue May 12, 2017 · 2 comments

Comments

@HEskandari
Copy link
Contributor

HEskandari commented May 12, 2017

Related to issue https://github.com/Particular/PlatformDevelopment/issues/925. This is a spin-off of a comment here.

Since ServiceInsight has very limited unit test coverage. The Maintainer Group feel that in cases of major changes/feature they need to have better coverage to ensure nothing is broken. This issue is to run a spike on that.

@HEskandari
Copy link
Contributor Author

HEskandari commented May 12, 2017

Maybe not directly related but historically we've had two approaches (not including the unit tests) in the code base:

  • Our approach to testing the UI in isolation in the Approval Tests. This solves the challenge of creating a self-relying UserControl and showing it for test purposes. The downside is that this is monitor/resolution/OS dependant and the tests might pass in one environment and not the other.

  • My earlier attempt on BBD style testing by running the whole and UI automation. This approach also has its own quirks. These tests were removed from the repo as of commit 4c2634f since they were not running on TeamCity.

@adamralph adamralph changed the title Add more automated test to ServiceInsight Spike black box tests May 12, 2017
@sergioc
Copy link

sergioc commented Aug 28, 2020

Discussed during enhancement release triage, closing for as we don't see this being needed any time soon.

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

No branches or pull requests

2 participants