-
Notifications
You must be signed in to change notification settings - Fork 13
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
FISH-9002 EE11 Authentication TCK Runner #118
Merged
Pandrex247
merged 23 commits into
payara:EE11
from
Pandrex247:FISH-9002-EE11-Authentication-TCK
Dec 2, 2024
Merged
Changes from all commits
Commits
Show all changes
23 commits
Select commit
Hold shift + click to select a range
073955a
FISH-9002 First stab
Pandrex247 c596893
FISH-9002 Correct linux script
Pandrex247 5cfadd2
FISH-9002 Remove unnecessary plugin versions
Pandrex247 5c9b73f
FISH-9002 Extra adjustments
Pandrex247 74aa062
Test
Pandrex247 aff52d6
Fix Windows runner
Pandrex247 852186d
Windows remote profile mostly working
Pandrex247 10e352f
FISH-9002 Fix replace script for Linux
Pandrex247 e294ff9
FISH-9002 Only run commands once
Pandrex247 51ba259
FISH-9002 Add epicyro dependency
Pandrex247 46c69c2
FISH-9002 Add payara-bom
Pandrex247 5c7cd43
FISH-9002 Add epicyro version property
Pandrex247 6a7af2c
FISH-9002 Add repositories
Pandrex247 bc7dd05
FISH-9002 Override webservice client class spi provider and add to cl…
Pandrex247 dec8778
FISH-9002 Use our existing class for now
Pandrex247 1fa16b8
FISH-9002 Also update readme
Pandrex247 baef03f
FISH-9002 Add explanative comment about tckDir
Pandrex247 0b8eed0
FISH-9002 Add Windows example
Pandrex247 6441dbe
FISH-9002 Add little TODO comment
Pandrex247 d2da292
FISH-9002 Add missing inherited=false
Pandrex247 074ed21
FISH-9002 Remove unnecessary test delay
Pandrex247 ff38c4c
Merge branch 'EE11' of https://github.com/payara/jakartaee-10-tck-run…
Pandrex247 4dd23d8
FISH-9002 Comment out Epicyro for now as we don't actually use it yet
Pandrex247 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change | ||||
---|---|---|---|---|---|---|
@@ -0,0 +1,46 @@ | ||||||
# Jakarta Authentication TCK Runner | ||||||
|
||||||
## Prerequisite | ||||||
|
||||||
Download and install the TCK into your local Maven repo. | ||||||
From the top-level directory: `mvn clean install -pl . -pl tck-download -pl tck-download/jakarta-authentication-tck -Pjakarta-staging` | ||||||
|
||||||
## Test Execution | ||||||
|
||||||
### Managed Server Profile | ||||||
NOTE: The payara-server-managed doesn't work on Windows. | ||||||
The `create-system-properties` asadmin command doesn't play well with Windows. | ||||||
|
||||||
To execute the full TCK against a managed Payara Server, run from the module directory, replacing the payara.version property. | ||||||
|
||||||
``` | ||||||
mvn clean verify -Ppayara-server-managed -Dpayara.version=${payaraVersion} | ||||||
``` | ||||||
|
||||||
### Remote Server Profile | ||||||
|
||||||
To execute the full TCK against a remote Payara Server installation, start your Payara Server and run the follow asadmin command against it, replacing ${tckDir} and ${payaraHome}: | ||||||
|
||||||
``` | ||||||
create-system-properties j2eelogin.name=j2ee:j2eelogin.password=j2ee:provider.configuration.file=${tckDir}/spi/common/ProviderConfiguration.xml:vendor.authconfig.factory=com.sun.jaspic.config.factory.AuthConfigFileFactory:log.file.location=${payaraHome}/glassfish/domains/domain1/logs | ||||||
``` | ||||||
|
||||||
${tckDir} is the directory where the unpacked TCK exists, for example: /home/user/Git/JakartaEE-10-TCK-Runners/authentication-tck/target/authentication-tck-3.1.0/tck | ||||||
|
||||||
NOTE: On Windows you will need to escape the ':' and '\' characters that will be present on your path variables, due to how the `create-system-properties` parses these characters | ||||||
For example: | ||||||
``` | ||||||
create-system-properties "j2eelogin.name=j2ee:j2eelogin.password=j2ee:provider.configuration.file=D\:\\Git\\JakartaEE10-TCK-Runners\\authentication-tck\\target\\authentication-tck-3.1.0\\tck\\spi\\common\\ProviderConfiguration.xml:vendor.authconfig.factory=com.sun.jaspic.config.factory.AuthConfigFileFactory:log.file.location=D\:\\Git\\Payara\\appserver\\distributions\\payara\\target\\stage\\payara7\\glassfish\\domains\\domain1\\logs" | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. same as before:
Suggested change
|
||||||
``` | ||||||
|
||||||
Run the TCK with the following command from the module directory, providing the path to the payaraHome and replacing the payara.version property: | ||||||
``` | ||||||
mvn clean verify -Ppayara-server-managed -Dpayara.home=${payaraHome} -Dpayara.version=${payaraVersion} | ||||||
``` | ||||||
|
||||||
If on Windows, you may encounter an error where the powershell script cannot be executed due to the local execution policy. | ||||||
The following will get around this for the terminal session: | ||||||
|
||||||
``` | ||||||
Set-ExecutionPolicy -Scope Process -ExecutionPolicy Bypass | ||||||
``` |
This file was deleted.
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Now the provider come from the epicyro implementation