Skip to content

Commit a2f055a

Browse files
committed
add constant for easier maintenance
1 parent cb08f3b commit a2f055a

3 files changed

Lines changed: 9 additions & 10 deletions

File tree

LabApiUtilities/src/main/com/microsoft/identity/labapi/utilities/constants/LabConstants.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@ public class LabConstants {
2626
private static final String NONE = "none";
2727

2828
public static final String DEFAULT_LAB_CLIENT_ID = "f62c5ae3-bf3a-4af5-afa8-a68b800396e9";
29+
public static final String DEFAULT_ID4SLAB2_CLIENT_ID = "c6bb302a-1e38-408e-9754-87c18fe81c80";
2930
public static final String DEFAULT_LAB_SCOPE = "https://request.msidlab.com/.default";
3031
public static final String KEYVAULT_SCOPE = "https://vault.azure.net/.default";
3132
public static final String DEFAULT_LAB_CERT_ALIAS = "LabAuth.MSIDLab.com";

common/src/test/java/com/microsoft/identity/common/internal/controllers/LocalMsalControllerTest.java

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,7 @@
4545
import com.microsoft.identity.labapi.utilities.client.ILabAccount;
4646
import com.microsoft.identity.labapi.utilities.client.ILabClient;
4747
import com.microsoft.identity.labapi.utilities.client.LabClient;
48+
import com.microsoft.identity.labapi.utilities.constants.LabConstants;
4849
import com.microsoft.identity.labapi.utilities.constants.UserType;
4950
import com.microsoft.identity.labapi.utilities.exception.LabApiException;
5051
import com.microsoft.identity.labapi.utilities.jwt.IJWTParser;
@@ -77,8 +78,6 @@ public class LocalMsalControllerTest {
7778

7879
private static final String AUTHORITY_URL = "https://login.microsoftonline.com/organizations";
7980

80-
private static final String CLIENT_ID = "c6bb302a-1e38-408e-9754-87c18fe81c80";
81-
8281
private static final String REDIRECT_URI = "msauth://com.msft.identity.client.sample.local/1wIqXSqBj7w%2Bh11ZifsnqwgyKrY%3D";
8382

8483
private static final String SCOPE = "User.read";
@@ -155,7 +154,7 @@ private RopcTokenCommandParameters createRopcCommandParameters() {
155154
.username(sTestAccount.getUsername())
156155
.password(sTestAccount.getPassword())
157156
.authority(Authority.getAuthorityFromAuthorityUrl(AUTHORITY_URL))
158-
.clientId(CLIENT_ID)
157+
.clientId(LabConstants.DEFAULT_ID4SLAB2_CLIENT_ID)
159158
.correlationId(UUID.randomUUID().toString())
160159
.redirectUri(REDIRECT_URI)
161160
.platformComponents(mPlatformComponents)
@@ -170,7 +169,7 @@ private RopcTokenCommandParameters createRopcCommandParameters() {
170169
private SilentTokenCommandParameters createSilentTokenCommandParameters(@NonNull final IAccountRecord accountRecord) {
171170
return SilentTokenCommandParameters.builder()
172171
.authority(Authority.getAuthorityFromAuthorityUrl(AUTHORITY_URL))
173-
.clientId(CLIENT_ID)
172+
.clientId(LabConstants.DEFAULT_ID4SLAB2_CLIENT_ID)
174173
.correlationId(UUID.randomUUID().toString())
175174
.redirectUri(REDIRECT_URI)
176175
.platformComponents(mPlatformComponents)

common/src/test/java/com/microsoft/identity/common/internal/request/MsalBrokerRequestAdapterTests.java

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -54,6 +54,7 @@
5454
import com.microsoft.identity.common.java.ui.BrowserDescriptor;
5555
import com.microsoft.identity.common.java.util.RequestHeaderSerializationUtil;
5656
import com.microsoft.identity.common.java.util.StringUtil;
57+
import com.microsoft.identity.labapi.utilities.constants.LabConstants;
5758

5859
import org.junit.Test;
5960
import org.junit.runner.RunWith;
@@ -68,8 +69,6 @@
6869
@RunWith(RobolectricTestRunner.class)
6970
public class MsalBrokerRequestAdapterTests {
7071

71-
final String CLIENT_ID = "c6bb302a-1e38-408e-9754-87c18fe81c80";
72-
7372
@Test
7473
public void test_getRequestBundleForSsoToken() {
7574
final String anAccountName = "anAccountName";
@@ -121,7 +120,7 @@ public void test_getRequestBundleForAcquireTokenInteractive() {
121120
final InteractiveTokenCommandParameters params = InteractiveTokenCommandParameters.builder()
122121
.platformComponents(components)
123122
.correlationId(CORRELATION_ID)
124-
.clientId(CLIENT_ID)
123+
.clientId(LabConstants.DEFAULT_ID4SLAB2_CLIENT_ID)
125124
.applicationName(CALLER_PACKAGE_NAME)
126125
.applicationVersion(VERSION)
127126
.redirectUri(REDIRECT_URI)
@@ -156,7 +155,7 @@ public void test_getRequestBundleForAcquireTokenInteractive_forAccountTransfer()
156155
final InteractiveTokenCommandParameters params = InteractiveTokenCommandParameters.builder()
157156
.platformComponents(components)
158157
.correlationId(CORRELATION_ID)
159-
.clientId(CLIENT_ID)
158+
.clientId(LabConstants.DEFAULT_ID4SLAB2_CLIENT_ID)
160159
.applicationName(CALLER_PACKAGE_NAME)
161160
.applicationVersion(VERSION)
162161
.redirectUri(REDIRECT_URI)
@@ -195,7 +194,7 @@ public void test_BrokerRequestFromAcquireTokenParameters_SignInWithGoogleParamet
195194
final AndroidInteractiveTokenCommandParameters params = AndroidInteractiveTokenCommandParameters.builder()
196195
.platformComponents(components)
197196
.correlationId("987d8962-3f4d-4054-a852-ac0c4b6a602e")
198-
.clientId(CLIENT_ID)
197+
.clientId(LabConstants.DEFAULT_ID4SLAB2_CLIENT_ID)
199198
.redirectUri("msauth://com.microsoft.identity.client.sample.local/1wIqXSqBj7w%2Bh11ZifsnqwgyKrY%3D")
200199
.applicationName("com.microsoft.identity.client.sample.local")
201200
.applicationVersion("1.0.0")
@@ -297,7 +296,7 @@ private void test_BrokerRequestFromAcquireTokenParametersInternal(final boolean
297296
final InteractiveTokenCommandParameters params = InteractiveTokenCommandParameters.builder()
298297
.platformComponents(components)
299298
.correlationId("987d8962-3f4d-4054-a852-ac0c4b6a602e")
300-
.clientId(CLIENT_ID)
299+
.clientId(LabConstants.DEFAULT_ID4SLAB2_CLIENT_ID)
301300
.redirectUri("msauth://com.microsoft.identity.client.sample.local/1wIqXSqBj7w%2Bh11ZifsnqwgyKrY%3D")
302301
.applicationName("com.microsoft.identity.client.sample.local")
303302
.applicationVersion("1.0.0")

0 commit comments

Comments
 (0)