Skip to content

Commit

Permalink
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Relax requirement of mandrel versions starting with mandrel-
Browse files Browse the repository at this point in the history
Since we can now define the distribution there is no longer the need to
include the mandrel- prefix in Mandrel versions.
zakkak committed Oct 31, 2023
1 parent bbbd2bd commit 1ad351b
Showing 5 changed files with 155 additions and 134 deletions.
7 changes: 4 additions & 3 deletions __tests__/mandrel.test.ts
Original file line number Diff line number Diff line change
@@ -9,7 +9,7 @@ process.env['RUNNER_TEMP'] = path.join(__dirname, 'TEMP')
test('request invalid version/javaVersion combination', async () => {
for (var combination of [
['mandrel-23.1.1.0-Final', '17'],
['mandrel-23.0.2.1-Final', '21'],
['mandrel-23.0.2.1-Final', '21']
]) {
let error = new Error('unexpected')
try {
@@ -29,7 +29,7 @@ test('request invalid version/javaVersion combination', async () => {
test('request invalid version', async () => {
for (var combination of [
['mandrel-23.1.1.0', '21'],
['mandrel-23.0.2.1', '17'],
['mandrel-23.0.2.1', '17']
]) {
let error = new Error('unexpected')
try {
@@ -58,7 +58,8 @@ test('get latest Mandrel for specific JDK', async () => {
// Test deprecated versions that won't get updates anymore
for (var combination of [
['11', '22.2.0.0-Final'],
['20', '23.0.1.2-Final']]) {
['20', '23.0.1.2-Final']
]) {
const latest = await mandrel.getLatestMandrelReleaseUrl(combination[0])
expect(latest).toContain(`mandrel-java${combination[0]}`)
expect(latest).toContain(combination[1])
55 changes: 14 additions & 41 deletions dist/cleanup/index.js

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

Loading

0 comments on commit 1ad351b

Please sign in to comment.