Cherrypick, change boolean to Boolean, Fixes AB#3261502 (#2635)

Made a change to support work profile, but a refactor introduced
possibility of null values for work profile is available field, which is
causing exceptions in automation as boolean type cannot handle null
value. Changing to Boolean.


[AB#3261502](https://identitydivision.visualstudio.com/fac9d424-53d2-45c0-91b5-ef6ba7a6bf26/_workitems/edit/3261502)

Author: fadidurah

common4j/src/main/com/microsoft/identity/common/java/flighting/CommonFlight.java

@@ -109,7 +109,7 @@ public enum CommonFlight implements IFlightConfig {
     /**
      * Flight to enable adding x-client-MN and x-client-WPAvailable extra query parameters
      */
-    ENABLE_AM_API_WORKPROFILE_EXTRA_QUERY_PARAMETERS("EnableAmApiWorkProfileExtraQueryParameters", false),
+    ENABLE_AM_API_WORKPROFILE_EXTRA_QUERY_PARAMETERS("EnableAmApiWorkProfileExtraQueryParameters", true),
 
     /** Flight to enable the new key generation without PURPOSE_WRAP_KEY. Default is true.
      * This is applicable for API >= 23

common4j/src/main/com/microsoft/identity/common/java/platform/Device.java

@@ -79,7 +79,7 @@ public static void clearDeviceMetadata(){
     }
 
     @GuardedBy("sLock")
-    public static void setIsInPersonalProfileButClouddpcWorkProfileAvailable(final boolean isWorkProfileAvailable) {
+    public static void setIsInPersonalProfileButClouddpcWorkProfileAvailable(final Boolean isWorkProfileAvailable) {
         sLock.writeLock().lock();
         try {
             sIsInPersonalProfileButClouddpcWorkProfileAvailable = isWorkProfileAvailable;
@@ -89,7 +89,7 @@ public static void setIsInPersonalProfileButClouddpcWorkProfileAvailable(final b
     }
 
     @GuardedBy("sLock")
-    public static boolean isInPersonalProfileButClouddpcWorkProfileAvailable() {
+    public static Boolean isInPersonalProfileButClouddpcWorkProfileAvailable() {
         sLock.readLock().lock();
         try {
             return sIsInPersonalProfileButClouddpcWorkProfileAvailable;

common4j/src/test/com/microsoft/identity/common/java/platform/DeviceTest.java

@@ -45,6 +45,7 @@ public class DeviceTest {
     @After
     public void tearDown() {
         Device.clearDeviceMetadata();
+        Device.setIsInPersonalProfileButClouddpcWorkProfileAvailable(null);
     }
 
     @Test
@@ -135,5 +136,19 @@ public void testGetAllMetadata(){
                 MockDeviceMetadata.TEST_OS_DRS;
         Assert.assertEquals(expectedResult, deviceMetadata.getAllMetadata());
     }
+
+    @Test
+    public void testGetWorkProfileField(){
+        Assert.assertNull(Device.isInPersonalProfileButClouddpcWorkProfileAvailable());
+
+        Device.setIsInPersonalProfileButClouddpcWorkProfileAvailable(false);
+        Assert.assertFalse(Device.isInPersonalProfileButClouddpcWorkProfileAvailable());
+
+        Device.setIsInPersonalProfileButClouddpcWorkProfileAvailable(true);
+        Assert.assertTrue(Device.isInPersonalProfileButClouddpcWorkProfileAvailable());
+
+        Device.setIsInPersonalProfileButClouddpcWorkProfileAvailable(null);
+        Assert.assertNull(Device.isInPersonalProfileButClouddpcWorkProfileAvailable());
+    }
 }