Skip to content

Commit

Permalink
Merge pull request #201 from QA-Automation-Starter/198-drop-jdk8-supp…
Browse files Browse the repository at this point in the history
…ort-and-migrate-to-jgiven-130-and-testng-780

jdk11 -- dropped jdk8 support
  • Loading branch information
adrian-herscu authored Nov 9, 2023
2 parents 98bf9d8 + 7486e1a commit ad6aac9
Show file tree
Hide file tree
Showing 266 changed files with 4,717 additions and 1,532 deletions.
19 changes: 13 additions & 6 deletions .github/workflows/on-main-push.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,8 @@ jobs:

strategy:
matrix:
java-version: [ 17, 11, 8 ]
# ISSUE on JDK 21 -- see on-pull-request.yml
java-version: [ 21, 17, 11 ]
os: [ ubuntu-latest, macos-latest, windows-latest ]

steps:
Expand All @@ -47,8 +48,8 @@ jobs:
mvnw install ^
-s %GITHUB_WORKSPACE%\settings.xml
deploy-for-jdk8:
name: jdk-8 snapshot publish
deploy-for-jdk11:
name: jdk-11 snapshot publish
# ISSUE ChromeDriver/GeckoDriver fail to initialize on Ubuntu
# see https://github.com/QA-Automation-Starter/qa-automation/actions/runs/3294884059/jobs/5432863668
# see https://docs.github.com/en/actions/using-github-hosted-runners/about-github-hosted-runners
Expand All @@ -61,7 +62,7 @@ jobs:
- uses: actions/checkout@v3
- uses: actions/setup-java@v3
with:
java-version: '8'
java-version: '11'
distribution: 'temurin'
cache: maven
settings-path: ${{ github.workspace }}
Expand Down Expand Up @@ -97,10 +98,15 @@ jobs:
# same as in on-pull-request.yml
generate-automation-project:
name: Generates new automation project and builds it
needs: deploy-for-jdk8
needs: deploy-for-jdk11
runs-on: ubuntu-latest

steps:
- uses: actions/setup-java@v3
with:
java-version: '11'
distribution: 'temurin'

- name: setup maven for sonatype snapshots
uses: whelk-io/maven-settings-xml-action@v20
with:
Expand Down Expand Up @@ -135,11 +141,12 @@ jobs:
- name: generate testing project
# ISSUE https://stackoverflow.com/questions/74581510/using-latest-for-archetypeversion-always-resolves-to-latest-release
# NOTE will use last released archetype
# TODO after 1.0.0 release should change DarchetypeVersion to LATEST
run: |
mvn --batch-mode archetype:generate \
-DarchetypeGroupId=dev.aherscu.qa \
-DarchetypeArtifactId=qa-testing-archetype \
-DarchetypeVersion=LATEST \
-DarchetypeVersion=1.0.0-SNAPSHOT \
-DgroupId=com.acme \
-DartifactId=testing \
-Dversion=1.0-SNAPSHOT \
Expand Down
27 changes: 21 additions & 6 deletions .github/workflows/on-pull-request.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,16 @@ jobs:

strategy:
matrix:
java-version: [ 17, 11, 8 ]
# ISSUE on JDK 21 --
# when running aggregated as
# mvnw install mode-build-full mode-build-nosign testing-tutorials
# while running qa-testing-example:
# Cannot define class using reflection: Unable to make protected java.lang.Package
# when running verify on qa-testing-example standalone:
# java.lang.RuntimeException: ProcessBuilder.start() debug
# (in addition)
# In summary, on JDK 21 the tests do not work.
java-version: [ 21, 17, 11 ]
os: [ ubuntu-latest, macos-latest, windows-latest ]

steps:
Expand All @@ -44,8 +53,8 @@ jobs:
mvnw install ^
-s %GITHUB_WORKSPACE%\settings.xml
verify-for-jdk8:
name: jdk-8 verify
verify-for-jdk11:
name: jdk-11 verify
# ISSUE ChromeDriver/GeckoDriver fail to initialize on Ubuntu
# see https://github.com/QA-Automation-Starter/qa-automation/actions/runs/3294884059/jobs/5432863668
# see https://docs.github.com/en/actions/using-github-hosted-runners/about-github-hosted-runners
Expand All @@ -58,7 +67,7 @@ jobs:
- uses: actions/checkout@v3
- uses: actions/setup-java@v3
with:
java-version: '8'
java-version: '11'
distribution: 'temurin'
cache: maven
settings-path: ${{ github.workspace }}
Expand Down Expand Up @@ -102,10 +111,15 @@ jobs:

generate-automation-project:
name: generate-build-run-standalone
needs: verify-for-jdk8
needs: verify-for-jdk11
runs-on: windows-latest

steps:
- uses: actions/setup-java@v3
with:
java-version: '11'
distribution: 'temurin'

- name: setup maven for sonatype snapshots
uses: whelk-io/maven-settings-xml-action@v20
with:
Expand Down Expand Up @@ -141,13 +155,14 @@ jobs:
shell: cmd
# ISSUE https://stackoverflow.com/questions/74581510/using-latest-for-archetypeversion-always-resolves-to-latest-release
# NOTE will use last released archetype
# TODO after 1.0.0 release should change DarchetypeVersion to LATEST
run: |
mvn archetype:generate ^
--batch-mode ^
-Dmaven.wagon.http.ssl.insecure=true ^
-DarchetypeGroupId=dev.aherscu.qa ^
-DarchetypeArtifactId=qa-testing-archetype ^
-DarchetypeVersion=LATEST ^
-DarchetypeVersion=1.0.0-SNAPSHOT ^
-DgroupId=com.acme ^
-DartifactId=testing ^
-Dversion=0.0.1-SNAPSHOT ^
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ jobs:
- uses: actions/checkout@v3
- uses: actions/setup-java@v3
with:
java-version: '8'
java-version: '11'
distribution: 'temurin'
cache: maven
settings-path: ${{ github.workspace }}
Expand Down
3 changes: 2 additions & 1 deletion .idea/misc.xml

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

10 changes: 8 additions & 2 deletions .idea/runConfigurations/rebuild_commons.xml

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion .mvn/wrapper/maven-wrapper.properties
Original file line number Diff line number Diff line change
Expand Up @@ -13,5 +13,5 @@
# See the License for the specific language governing permissions and
# limitations under the License.
#
distributionUrl=https://repo.maven.apache.org/maven2/org/apache/maven/apache-maven/3.9.4/apache-maven-3.9.4-bin.zip
distributionUrl=https://repo.maven.apache.org/maven2/org/apache/maven/apache-maven/3.9.5/apache-maven-3.9.5-bin.zip
wrapperUrl =https://repo.maven.apache.org/maven2/org/apache/maven/wrapper/maven-wrapper/3.2.0/maven-wrapper-3.2.0.jar
Loading

0 comments on commit ad6aac9

Please sign in to comment.