fix(test): Adapt tests to changes 5.9 -> 5.18

ref: issue #41

.idea/runConfigurations/Test.xml

@@ -3,9 +3,9 @@
     <deployment type="dockerfile">
       <settings>
         <option name="imageTag" value="sdkman-for-fish-tests" />
-        <option name="containerName" value="" />
+        <option name="containerName" value="sdkman-for-fish-tests" />
         <option name="contextFolderPath" value="." />
-        <option name="commandLineOptions" value="--rm" />
+        <option name="commandLineOptions" value="" />
         <option name="sourceFilePath" value="test/Dockerfile" />
       </settings>
     </deployment>

CHANGELOG.md

@@ -13,10 +13,11 @@ this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.htm
 
 ### Features
 
-- Compatibility with SDKMAN! 5.9.2
+- Compatibility with SDKMAN! 5.18.2
   - Completions for `env`, `home`, `flush` (issue #35)
   - Correct behaviour of `env clear`. 
   - Honor `sdkman_auto_env=true` (issue #38)
+  - `broadcast` removed
   - TODO: custom SDKMAN! install path (issue #34)
 - Compatibility with fisher 4 (PR #37, #39)
 

README.md

@@ -10,9 +10,9 @@ Also adds binaries from installed SDKs to the PATH.
 
 Version 2.0.0 has been tested with 
 
- - fish 3.1.2, and 
+ - fish 3.6.1, and 
- - SDKMAN! 5.11.0, on
+ - SDKMAN! 5.18.2, on
- - Ubuntu 20.04 LTS and macOS 10.15
+ - Ubuntu 22.04 LTS and macOS 12.6
 
 ## Install
 

test/features/autoenv.feature

@@ -52,8 +52,8 @@ Feature: Support autoenv setting
             1.10.1
             """
 
-    # TODO: This one doesn't work due to a bug in sdkman. Track: https://github.com/sdkman/sdkman-cli/pull/878
+    # TODO: But that PR had been merged back when -- re-investigate
-    @pending
+    @pending # This one doesn't work due to a bug in sdkman. Track: https://github.com/sdkman/sdkman-cli/pull/878
     Scenario: Switching between directories with .sdkmanrc
         Given SDKMAN! config sets sdkman_auto_env to true
         And   file /tmp/autoenv-test-1/.sdkmanrc exists with content

test/features/completions.feature

@@ -9,7 +9,7 @@ Feature: Shell Completion
 
     Scenario: Command list correct
         When the user enters " " into the prompt
-        Then completion should propose "b, broadcast, c, current, d, default, flush, h, help, i, install, list, ls, offline, rm, selfupdate, u, ug, uninstall, update, upgrade, use, v, version"
+        Then completion should propose "c, current, d, default, flush, h, help, i, install, list, ls, offline, rm, selfupdate, u, ug, uninstall, update, upgrade, use, v, version"
 
     Scenario Outline: Commands complete
         When the user enters "<cmd>" into the prompt
@@ -17,7 +17,6 @@ Feature: Shell Completion
         But  completion should not propose <exclusions>
         Examples:
             | cmd | completions                            | exclusions |
-            | b   | b, broadcast                           | /^[^b]+$/  |
             | c   | c, current                             | /^[^c]+$/  |
             | d   | d, default                             | /^[^d]+$/  |
             | e   | e, env                                 | /^[^e]+$/  |
@@ -39,6 +38,7 @@ Feature: Shell Completion
             # Currently uncovered (except by fuzzy matches);
             # include negatives to prevent accidents:
             | a   |                                        | /^a/       |
+            | b   |                                        | /^b/       |
             | g   |                                        | /^g/       |
             | j   |                                        | /^j/       |
             | k   |                                        | /^k/       |
@@ -207,11 +207,9 @@ Feature: Shell Completion
         But  completion should not propose <exclusions>
         Examples:
             | cmd    | completions   | exclusions                |
-            |         | archives, broadcast, tmp, version | /^(?!archives\|broadcast\|tmp\|version).*$/ |
+            |        | temp, version | /^(?!temp\|version).*$/   |
-            | b       | broadcast                         | /^(?!broadcast).*$/                         |
+            | t      | temp          | /^(?!temp).*$/           |
-            | a       | archives                          | /^(?!archives\|broadcast).*$/               |
+            | v      | version       | /^(?!version).*$/         |
-            | t       | tmp                               | /^(?!tmp\|broadcast).*$/                    |
-            | v       | version                           | /^(?!version\|archives).*$/                 |
             | x      |               | /.*/                      |
             | 'tmp ' |               | /.*/                      |
 
@@ -222,8 +220,6 @@ Feature: Shell Completion
             | cmd         |
             | 'version '  |
             | 'version a' |
-            | 'broadcast '  |
-            | 'broadcast a' |
             | 'help '     |
             | 'help a'    |
             | 'update '   |

test/features/corner_cases.feature

@@ -12,6 +12,7 @@ Feature: Corner Cases
 
     # TODO: add test that fails if `test` in conf.d/sdk.fish:80 errors (cf issue #28 et al.)
 
+    @pending # cf. issue #10
     Scenario: PATH should contain only valid paths
         Given candidate kscript is installed
         When  candidate kscript is uninstalled

test/features/wrapper.feature

@@ -31,7 +31,7 @@ Feature: Wrapping of Bash
             | sdk update                                          |
             | sdk use ant 1.9.9                                   |
             | sdk offline enable > /dev/null; sdk install ant foo |
-            | sdk use ant 1.9.9 > /dev/null; sdk broadcast        |
+            | sdk use ant 1.9.9 > /dev/null; sdk version          |
             | sdk home ant 1.9.9                                  |
             | cd /tmp/env-test; sdk env                           |
             | cd /tmp/env-test; sdk env; sdk env clear            |