Merge pull request #494 from voxpupuli/debian12-node18

Fix pinning of NodeJS 18 on Debian 12 in tests
voxpupuli · Jan 3, 2024 · d42fbb1 · d42fbb1
2 parents f570e7c + 22af21b
commit d42fbb1
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/spec/acceptance/class_spec.rb b/spec/acceptance/class_spec.rb
@@ -34,9 +34,10 @@
 
     include_examples 'cleanup'
 
-    # Debian 12 contains NodeJS 18, when we test 16, we need to force the nodesource version
+    # Debian 12 contains NodeJS 18, when we test 16 and 18, we need to force the nodesource version
+    # as Debians versions *can* be newer
     repo_pin =
-      if nodejs_version == '16' && fact('os.family') == 'Debian' && fact('os.release.major') == '12'
+      if %w[16 18].include?(nodejs_version) && fact('os.family') == 'Debian' && fact('os.release.major') == '12'
         '1000'
       else
         'undef'