Crash on invalid Elixir+OTP combo (#134)

* Prevent "future" errors when choosing incompatible Elixir v. OTP versions

* Fix tests as per GitHub Actions' output

* Make test elixir `master` strict

It has no -otp- in the name
This commit is contained in:
Paulo F. Oliveira
2022-09-13 08:21:46 +01:00
committed by GitHub
parent 7dd0d763a8
commit 9027a7c3da
4 changed files with 13 additions and 8 deletions
+1
View File
@@ -116,6 +116,7 @@ jobs:
- elixir-version: 'master'
otp-version: '25'
os: 'ubuntu-20.04'
version-type: 'strict'
- gleam-version: 'nightly'
otp-version: '24'
- gleam-version: '0.23'
+2 -2
View File
@@ -171,8 +171,8 @@ async function testElixirVersions() {
let otpVersion
spec = '1.1.x'
otpVersion = 'OTP-23'
expected = 'v1.1.1'
otpVersion = 'OTP-17'
expected = 'v1.1.1-otp-17'
got = await setupBeam.getElixirVersion(spec, otpVersion)
assert.deepStrictEqual(got, expected)
+5 -3
View File
@@ -7358,9 +7358,11 @@ async function getElixirVersion(exSpec0, otpVersion) {
`Using Elixir ${elixirVersion} (built for OTP ${otpVersionMajor})`,
)
} else {
// ... and it's not available: fallback to the 'generic' version (v1.4.5 only).
elixirVersionWithOTP = elixirVersion
core.info(`Using Elixir ${elixirVersion}`)
// ... and it's not available: exit with exception
throw new Error(
`Requested Elixir / Erlang/OTP version (${exSpec0} / ${otpVersion}) not ` +
'found in version list (did you check Compatibility between Elixir and Erlang/OTP?)',
)
}
} else {
throw new Error(
+5 -3
View File
@@ -188,9 +188,11 @@ async function getElixirVersion(exSpec0, otpVersion) {
`Using Elixir ${elixirVersion} (built for OTP ${otpVersionMajor})`,
)
} else {
// ... and it's not available: fallback to the 'generic' version (v1.4.5 only).
elixirVersionWithOTP = elixirVersion
core.info(`Using Elixir ${elixirVersion}`)
// ... and it's not available: exit with exception
throw new Error(
`Requested Elixir / Erlang/OTP version (${exSpec0} / ${otpVersion}) not ` +
'found in version list (did you check Compatibility between Elixir and Erlang/OTP?)',
)
}
} else {
throw new Error(