[SECURITY-1390]

Author: gavinfish

pom.xml

@@ -132,6 +132,12 @@
             <version>4.5.7</version>
         </dependency>
 
+        <dependency>
+            <groupId>junit</groupId>
+            <artifactId>junit</artifactId>
+            <version>4.12</version>
+            <scope>test</scope>
+        </dependency>
         <!--fix the requireUpperBounds errors-->
         <dependency>
             <groupId>commons-codec</groupId>

src/main/java/com/microsoft/jenkins/azuread/AzureSecurityRealm.java

@@ -305,7 +305,7 @@ public void marshal(Object source, HierarchicalStreamWriter writer, MarshallingC
             writer.endNode();
 
             writer.startNode("clientsecret");
-            writer.setValue(realm.getClientSecret());
+            writer.setValue(realm.getClientSecretSecret());
             writer.endNode();
 
             writer.startNode("tenant");

src/test/java/com/microsoft/jenkins/azuread/AzureSecurityRealmTest.java

@@ -1,4 +1,71 @@
 package com.microsoft.jenkins.azuread;
 
+import com.thoughtworks.xstream.io.binary.BinaryStreamReader;
+import com.thoughtworks.xstream.io.binary.BinaryStreamWriter;
+import org.apache.commons.io.output.ByteArrayOutputStream;
+import org.junit.Assert;
+import org.junit.Before;
+import org.junit.Rule;
+import org.junit.Test;
+import org.jvnet.hudson.test.JenkinsRule;
+
+import java.io.ByteArrayInputStream;
+import java.io.IOException;
+import java.util.concurrent.ExecutionException;
+
 public class AzureSecurityRealmTest {
+    @Rule
+    public JenkinsRule j = new JenkinsRule();
+
+    @Before
+    public void init() throws Exception {
+        j.recipe();
+    }
+
+    @Test
+    public void testConverter() throws InterruptedException, ExecutionException, IOException {
+        BinaryStreamWriter writer = null;
+        BinaryStreamReader reader = null;
+        try {
+            AzureSecurityRealm securityRealm = new AzureSecurityRealm("tenant", "clientId", "secret");
+            AzureSecurityRealm.ConverterImpl converter = new AzureSecurityRealm.ConverterImpl();
+            ByteArrayOutputStream outputStream = new ByteArrayOutputStream();
+            writer = new BinaryStreamWriter(outputStream);
+            writer.startNode("parentNode");
+            converter.marshal(securityRealm, writer, null);
+            writer.endNode();
+            byte[] bytes = outputStream.toByteArray();
+            reader = new BinaryStreamReader(new ByteArrayInputStream(bytes));
+            AzureSecurityRealm result = (AzureSecurityRealm) converter.unmarshal(reader, null);
+
+            Assert.assertEquals(securityRealm.getTenant(), result.getTenant());
+            Assert.assertEquals(securityRealm.getClientId(), result.getClientId());
+            Assert.assertEquals(securityRealm.getClientSecret(), result.getClientSecret());
+        } finally {
+            if (writer != null) {
+                writer.close();
+            }
+            if (reader != null) {
+                reader.close();
+            }
+        }
+    }
+
+    @Test
+    public void testSavedConfig() throws InterruptedException, ExecutionException, IOException {
+        BinaryStreamWriter writer = null;
+        try {
+            String secretString = "thisIsSpecialSecret";
+            AzureSecurityRealm securityRealm = new AzureSecurityRealm("tenant", "clientId", secretString);
+            AzureSecurityRealm.ConverterImpl converter = new AzureSecurityRealm.ConverterImpl();
+            ByteArrayOutputStream outputStream = new ByteArrayOutputStream();
+            writer = new BinaryStreamWriter(outputStream);
+            converter.marshal(securityRealm, writer, null);
+            Assert.assertFalse(outputStream.toString().contains(secretString));
+        } finally {
+            if (writer != null) {
+                writer.close();
+            }
+        }
+    }
 }