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

PLANNER-2411: Add tests for Dev UI in Quarkus School Timetabling #122

Conversation

Christopher-Chianelli
Copy link
Contributor

Due to quarkusio/quarkus#17915, tests for Dev UI cannot be in the same maven project as the extension. Posting here to give an example of how the test looks like. Quickstarts is currently under a refactor by @ge0ffrey , so do not merge until their work is finished.

JIRA

https://issues.redhat.com/browse/PLANNER-2411

Referenced pull requests

https://github.com/kiegroup/optaplanner/pull/1381

@ge0ffrey
Copy link
Contributor

Dev UI testing should happen in https://github.com/kiegroup/optaplanner/tree/master/optaplanner-quarkus-integration/optaplanner-quarkus/integration-test or something similar I'd argue.

For quickstarts, it should just work, but no testing code needed.
Wdyt?

@Christopher-Chianelli
Copy link
Contributor Author

The issue is closed, so that should work now (previously was not possible)

@triceo
Copy link
Contributor

triceo commented Jun 30, 2021

@Christopher-Chianelli Now that the quickstarts repo is unfrozen again, do you intend to rebase this PR?

@Christopher-Chianelli
Copy link
Contributor Author

@triceo I'll try again on Quarkus on the optaplanner repo; if that works, I open the PR there; otherwise I will open it again here

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