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

Bugfix 209 #217

Closed
wants to merge 3 commits into from
Closed

Bugfix 209 #217

wants to merge 3 commits into from

Conversation

carlospzurita
Copy link
Contributor

Reworked the mobileRouter.js to check existence of TestObject and redirect to normal TestRun dialog if it has been deleted.

@jonherrmann
Copy link
Collaborator

Codacy Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 1
           

See the complete overview on Codacy

@jonherrmann jonherrmann added the PR: UI Tests required Blocked as long as no UI tests are available for verification label Jul 1, 2020
@jonherrmann jonherrmann linked an issue Jul 1, 2020 that may be closed by this pull request
@carlospzurita
Copy link
Contributor Author

We are having some delays on the development of the UI tests for this PR. We have been trying to automatize the API request to create and delete TestObjects from Hightwatch, in order to have the whole cycle for testing, but we found it difficult to do. We hope to have a test ready for today

@carlospzurita
Copy link
Contributor Author

carlospzurita commented Jul 6, 2020

We have added a pull request https://github.com/etf-validator/etf-system-tests/pull/5 on the system tests repository to test this development.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: UI Tests required Blocked as long as no UI tests are available for verification
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bugfix for Rerun button feature
2 participants