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

[Issue_183][WFLY-19397] Add a Jakarta Data TCK runner #184

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

bstansberry
Copy link
Contributor

Resolves #183

My (slightly modified from what's currently pushed) branch for wildfly/wildfly#18227 passes, both when pointing at an existing WildFly Preview build via -Djboss.home=/the/path and when using -Dversion.org.wildfly.wildfly=34.0.0.Beta1-SNAPSHOT and provisioning a server with the proposed 'jakarta-layer' layer.

data/pom.xml Outdated Show resolved Hide resolved
data/tools/pom.xml Outdated Show resolved Hide resolved
data/tools/pom.xml Outdated Show resolved Hide resolved
*
* SPDX-License-Identifier: Apache-2.0
*/
package ee.jakarta.tck.data.tools.annp;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are we allowed to use this package name?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think so. The TCK guide points to the repo I copied this from as an example of how to run the TCK.

The build is specifically configured not to deploy this.

If I get time later I'll see how painful it would be to repackage this stuff.

<feature-pack>
<groupId>${wildfly.feature.pack.groupId}</groupId>
<artifactId>${wildfly.feature.pack.artifactId}</artifactId>
<version>${version.org.wildfly.wildfly}</version>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FWIW if we leave the version.org.wildfly.wildfly property empty and use a channel, we don't have to upgrade the version for every new release.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jamezp I'd prefer to leave this for the future. I want this to work with all three of wildfly, wildfly-ee and wildfly-preview feature packs and I don't have time now to incorporate channel wiring and test the combinations.

data/wildfly-runner/src/test/java/embedded/CdiTests.java Outdated Show resolved Hide resolved
@bstansberry
Copy link
Contributor Author

Note: This passes when run against a build of wildfly/wildfly#18434. Passes when run with any of the 3 'jboss.home' variants from that build (dist, ee-dist and preview/dist) as well as with the 3 feature packs (wildfly, wildfly-ee and wildfly-preview).

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.

Create a Jakarta Data TCK Runner
2 participants