hide slim symbol values prefixed with SECRET_ on the slim runner side

FitNesseRoot/FitNesse/SuiteAcceptanceTests/SuiteSlimTests/SlimSymbols/SlimSymbolsWithSecretPrefixAreNotExpandedInHtml.wiki

@@ -0,0 +1,22 @@
+---
+Test
+---
+!3 When a symbol is prefixed with SECRET, it's contents will not be shown upon accessing the symbol
+
+!| script                                |
+|given page|SecretSymbolTest|with content|!-!define TEST_SYSTEM {slim}
+
+|import            |
+|fitnesse.slim.test|
+|fitnesse.fixtures |
+
+|script         |echo fixture                           |
+|$SECRET_symbol=|echo|this is secret info               |
+|set name       |$SECRET_symbol                         |
+|check          |echo|$SECRET_symbol|this is secret info|
+-! |
+|check |request page    |SecretSymbolTest?test            |200           |
+|ensure|content contains|!-$SECRET_symbol->[*****]-!                     |
+|ensure|content contains|&lt;span class="pass">this is secret info</span>|
+|show  |content                                                          |
+

src/fitnesse/testsystems/slim/SlimTestContextImpl.java

@@ -2,6 +2,11 @@
 // Released under the terms of the CPL Common Public License version 1.0.
 package fitnesse.testsystems.slim;
 
+import fitnesse.testsystems.ExecutionResult;
+import fitnesse.testsystems.TestPage;
+import fitnesse.testsystems.TestSummary;
+import fitnesse.testsystems.slim.tables.ScenarioTable;
+import fitnesse.testsystems.slim.tables.ScriptTable;
 import fitnesse.util.TimeMeasurement;
 
 import java.util.ArrayList;
@@ -12,12 +17,6 @@
 import java.util.List;
 import java.util.Map;
 
-import fitnesse.testsystems.ExecutionResult;
-import fitnesse.testsystems.TestPage;
-import fitnesse.testsystems.TestSummary;
-import fitnesse.testsystems.slim.tables.ScenarioTable;
-import fitnesse.testsystems.slim.tables.ScriptTable;
-
 public class SlimTestContextImpl implements SlimTestContext {
   private final Map<String, String> symbols = new HashMap<>();
   private final Map<String, ScenarioTable> scenarios = new HashMap<>(512);
@@ -36,6 +35,9 @@ public SlimTestContextImpl(TestPage pageToTest) {
 
   @Override
   public String getSymbol(String symbolName) {
+    if (symbolName.startsWith("SECRET_")) {
+      return "*****";
+    }
     return symbols.get(symbolName);
   }
 

test/fitnesse/testsystems/slim/tables/SlimTableTest.java

@@ -48,6 +48,13 @@ public void replaceSymbolsShouldReplaceSimpleSymbol() throws Exception {
     assertEquals("this is a", table.replaceSymbols("this is $x"));
   }
 
+  @Test
+  public void replaceSymbolsShouldReplaceSecretSymbol() throws Exception {
+    SlimTable table = new MockTable();
+    table.setSymbol("SECRET_x", "a");
+    assertEquals("this is *****", table.replaceSymbols("this is $SECRET_x"));
+  }
+
   @Test
   public void replaceSymbolsShouldReplaceMoreThanOneSymbol() throws Exception {
     SlimTable table = new MockTable();
@@ -87,6 +94,13 @@ public void replaceSymbolsFullExpansion_ShouldReplaceSimpleSymbol() throws Excep
     assertEquals("this is $x->[a]", table.replaceSymbolsWithFullExpansion("this is $x"));
   }
 
+  @Test
+  public void replaceSymbolsFullExpansion_ShouldReplaceSecretSymbol() throws Exception {
+    SlimTable table = new MockTable();
+    table.setSymbol("SECRET_x", "a");
+    assertEquals("this is $SECRET_x->[*****]", table.replaceSymbolsWithFullExpansion("this is $SECRET_x"));
+  }
+
   @Test
   public void replaceSymbolsFullExpansion_ShouldReplaceMoreThanOneSymbol() throws Exception {
     SlimTable table = new MockTable();