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.

Sign up for GitHub

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

Jump to bottom

[QACI-542 ECOSYS-68] Support parallel testing, do not expose dependencies to clients #105

Merged
merged 13 commits into from
Feb 16, 2021
Merged

[QACI-542 ECOSYS-68] Support parallel testing, do not expose dependencies to clients #105

merged 13 commits into from
Feb 16, 2021

Conversation

lprimak
Copy link
Contributor

@lprimak lprimak commented Jan 31, 2021 •
edited
Loading

Facilitated parallel testing support:

  • Update to Arquillian 1.7 latest alpha to support parallel tests
  • Added ConcurrentHashMap for deploy aliases
  • Added better logs for debugging support when application deploy fails
  • moved fields to local variables
  • made more fields final
  • general refactoring for "moderm" Java 8

Updated shade plugin configuration as to not leak dependencies to clients
Stabilized integration tests on Micro (workaround for MP-CONFIG incompatibility with micro remote deployer)

Also Fixes #95

@lprimak lprimak changed the title Support parallel testing [QACI-542 ECOSYS-68] Support parallel testing, do not expose dependencies to clients Feb 1, 2021
@lprimak
Copy link
Contributor Author

lprimak commented Feb 1, 2021

This PR is available for testing in the Maven Central snapshot repository now
https://oss.sonatype.org/content/repositories/snapshots

@lprimak lprimak marked this pull request as draft February 6, 2021 20:21
@lprimak lprimak marked this pull request as ready for review February 8, 2021 06:31
@lprimak lprimak requested a review from cubastanley February 8, 2021 06:31
@lprimak lprimak merged commit ae0ccbc into payara:master Feb 16, 2021
@lprimak lprimak deleted the ParallelTesting branch February 16, 2021 06:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jGauravGupta jGauravGupta jGauravGupta approved these changes

@pdudits pdudits Awaiting requested review from pdudits

@Pandrex247 Pandrex247 Awaiting requested review from Pandrex247

@cubastanley cubastanley Awaiting requested review from cubastanley

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add Jakarta EE 9 Support
2 participants
@lprimak @jGauravGupta