Fixing the misconception that there are no challengeas after challeng…

…e 6 ;-)

src/main/java/org/owasp/wrongsecrets/Challenges.java

@@ -88,6 +88,10 @@ public boolean isFirstChallenge(ChallengeDefinition challengeDefinition) {
     return challengeDefinition.equals(definitions.challenges().get(0));
   }
 
+  public boolean isLastChallenge(ChallengeDefinition challengeDefinition) {
+    return challengeDefinition.equals(definitions.challenges().getLast());
+  }
+
   public List<ChallengeDefinition> getChallengeDefinitions() {
     return definitions.challenges();
   }

src/main/java/org/owasp/wrongsecrets/challenges/ChallengesController.java

@@ -48,7 +48,7 @@ public class ChallengesController {
   @Value("${ctf_enabled}")
   private boolean ctfModeEnabled;
 
-  private boolean spoilingEnabled;
+  private final boolean spoilingEnabled;
 
   @Value("${ctf_key}")
   private String ctfKey;
@@ -93,7 +93,7 @@ public String spoiler(@PathVariable("short-name") String shortName, Model model)
       model.addAttribute("spoiler", new Spoiler("Spoils are disabled in the configuration"));
     } else {
       Optional<Spoiler> spoilerFromRuntimeEnvironment =
-          challenges.findChallenge(shortName, runtimeEnvironment).map(c -> c.spoiler());
+          challenges.findChallenge(shortName, runtimeEnvironment).map(Challenge::spoiler);
       Supplier<Spoiler> spoilerFromRandomChallenge =
           () -> {
             var challengeDefinition = findByShortName(shortName);
@@ -151,8 +151,13 @@ public String challenge(Model model, @PathVariable("short-name") String shortNam
     model.addAttribute("answerCorrect", null);
     model.addAttribute("answerIncorrect", null);
     model.addAttribute("solution", null);
+    String clickNext =
+        "This challenge has been disabled. Click \"next\" to go to the next challenge.";
+    if (challenges.isLastChallenge(challengeDefinition)) {
+      clickNext = "This challenge has been disabled";
+    }
     if (!isChallengeEnabled(challengeDefinition)) {
-      model.addAttribute("answerIncorrect", "This challenge has been disabled.");
+      model.addAttribute("answerIncorrect", "This challenge has been disabled." + clickNext);
     }
     if (ctfModeEnabled && challenges.isFirstChallenge(challengeDefinition)) {
       if (!Strings.isNullOrEmpty(ctfServerAddress) && !ctfServerAddress.equals("not_set")) {
@@ -331,7 +336,7 @@ private void enrichWithHintsAndReasons(Model model) {
   private void fireEnding(Model model) {
     var notCompleted =
         challenges.getDefinitions().challenges().stream()
-            .filter(def -> isChallengeEnabled(def))
+            .filter(this::isChallengeEnabled)
             .filter(this::challengeNotCompleted)
             .count();
     if (notCompleted == 0) {

src/main/resources/explanations/missing_cloud.adoc

@@ -1,2 +1,4 @@
 We are running outside a properly configured Cloud environment. Please run this in an AWS/Azure/GCP environment as
 explained in the https://github.com/OWASP/wrongsecrets#cloud-challenges[README.md]
+
+There are still supported challenges after this one. Please try another challenge instead!

src/main/resources/explanations/missing_k8s.adoc

@@ -1,3 +1,5 @@
 We are running outside a K8s cluster.
 Please run this in the K8s cluster as explained in the
 https://github.com/OWASP/wrongsecrets#basic-k8s-exercise[README.md]
+
+There are still supported challenges after this one. Please try another challenge instead!

src/main/resources/explanations/missing_vault.adoc

@@ -1,2 +1,4 @@
 We are running outside a K8s cluster with Vault. Please run this in the K8s cluster as explained in the
 https://github.com/OWASP/wrongsecrets#vault-exercises-with-minikube[README.md]
+
+There might still be supported challenges after this one. Please try another challenge instead!