Address other migration from XmlElementWrapper

This was not broken, but seems better to change this as well, at least
for consistency, given the issue with Jmx configuration after migrating
from JAXB's XmlElementWrapper.

This covers all places that used to use JAXB's XmlElementWrapper before
migration to XStream.

Author: trask

core/src/main/java/com/microsoft/applicationinsights/internal/config/TelemetryProcessorXmlElement.java

@@ -30,7 +30,6 @@
 /**
  * Created by gupele on 7/26/2016.
  */
-@XStreamAlias("Processor")
 public class TelemetryProcessorXmlElement {
 
     @XStreamAsAttribute

core/src/main/java/com/microsoft/applicationinsights/internal/config/TelemetryProcessorsXmlElement.java

@@ -24,31 +24,38 @@
 import java.util.ArrayList;
 
 import com.thoughtworks.xstream.annotations.XStreamAlias;
+import com.thoughtworks.xstream.annotations.XStreamImplicit;
 
 /**
  * Created by gupele on 7/26/2016.
  */
 public class TelemetryProcessorsXmlElement {
 
     @XStreamAlias("CustomProcessors")
-    private ArrayList<TelemetryProcessorXmlElement> custom = new ArrayList<>();
+    private TelemetryProcessorWrapperXmlElement customWrapper = new TelemetryProcessorWrapperXmlElement();
 
     @XStreamAlias("BuiltInProcessors")
-    private ArrayList<TelemetryProcessorXmlElement> builtIn = new ArrayList<>();
+    private TelemetryProcessorWrapperXmlElement builtInWrapper = new TelemetryProcessorWrapperXmlElement();
 
     public ArrayList<TelemetryProcessorXmlElement> getBuiltInTelemetryProcessors() {
-        return builtIn;
+        return builtInWrapper.processors;
     }
 
     public void setBuiltInTelemetryProcessors(ArrayList<TelemetryProcessorXmlElement> builtIn) {
-        this.builtIn = builtIn;
+        builtInWrapper.processors = builtIn;
     }
 
     public ArrayList<TelemetryProcessorXmlElement> getCustomTelemetryProcessors() {
-        return custom;
+        return customWrapper.processors;
     }
 
     public void setCustomTelemetryProcessors(ArrayList<TelemetryProcessorXmlElement> custom) {
-        this.custom = custom;
+        customWrapper.processors = custom;
+    }
+
+    public static class TelemetryProcessorWrapperXmlElement {
+
+        @XStreamImplicit(itemFieldName = "Processor")
+        private ArrayList<TelemetryProcessorXmlElement> processors = new ArrayList<>();
     }
 }

core/src/test/java/com/microsoft/applicationinsights/internal/config/JaxbAppInsightsConfigurationBuilderTest.java

@@ -62,5 +62,11 @@ public void testBuilderProducesCorrectConfig() throws IOException {
         Assert.assertEquals("System Threads", windowsPC.getDisplayName());
         JmxXmlElement jmxXmlElement = config.getPerformance().getJmxXmlElements().get(0);
         Assert.assertEquals("Thread Count", jmxXmlElement.getDisplayName());
+        TelemetryProcessorXmlElement builtInTelemetryProcessor =
+                config.getTelemetryProcessors().getBuiltInTelemetryProcessors().get(0);
+        Assert.assertEquals("FixedRateSamplingTelemetryProcessor", builtInTelemetryProcessor.getType());
+        TelemetryProcessorXmlElement customTelemetryProcessor =
+                config.getTelemetryProcessors().getCustomTelemetryProcessors().get(0);
+        Assert.assertEquals("Test", customTelemetryProcessor.getType());
     }
 }