Skip to content

Commit 7aadc1f

Browse files
authored
Remove ADFS usage where not needed (#5444)
test fixes
1 parent c21e39b commit 7aadc1f

File tree

5 files changed

+20
-26
lines changed

5 files changed

+20
-26
lines changed

tests/Microsoft.Identity.Test.Integration.netcore/HeadlessTests/OnBehalfOfTests.cs

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -226,20 +226,17 @@ public async Task OboAndClientCredentials_WithRegional_ReturnsCorrectTokens_Test
226226
}
227227

228228
[TestMethod]
229-
#if IGNORE_FEDERATED
230-
[Ignore]
231-
#endif
232229
public async Task WithMultipleUsers_TestAsync()
233230
{
234231
var aadUser1 = (await LabUserHelper.GetDefaultUserAsync().ConfigureAwait(false)).User;
235232
var aadUser2 = (await LabUserHelper.GetDefaultUser2Async().ConfigureAwait(false)).User;
236-
var adfsUser = (await LabUserHelper.GetAdfsUserAsync(FederationProvider.ADFSv2019).ConfigureAwait(false)).User;
233+
var aadUser3 = (await LabUserHelper.GetDefaultUser3Async().ConfigureAwait(false)).User;
237234

238-
await RunOnBehalfOfTestAsync(adfsUser, false).ConfigureAwait(false);
235+
await RunOnBehalfOfTestAsync(aadUser3, false).ConfigureAwait(false);
239236
await RunOnBehalfOfTestAsync(aadUser1, false).ConfigureAwait(false);
240237
await RunOnBehalfOfTestAsync(aadUser1, true).ConfigureAwait(false);
241238
await RunOnBehalfOfTestAsync(aadUser2, false).ConfigureAwait(false);
242-
await RunOnBehalfOfTestAsync(adfsUser, true).ConfigureAwait(false);
239+
await RunOnBehalfOfTestAsync(aadUser3, true).ConfigureAwait(false);
243240
await RunOnBehalfOfTestAsync(aadUser2, true).ConfigureAwait(false);
244241
await RunOnBehalfOfTestAsync(aadUser2, false, true).ConfigureAwait(false);
245242
}

tests/Microsoft.Identity.Test.Integration.netcore/HeadlessTests/UsernamePasswordIntegrationTests.NetFwk.cs

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -81,14 +81,10 @@ public async Task ARLINGTON_ROPC_AAD_CCA_Async()
8181
}
8282

8383
[RunOn(TargetFrameworks.NetCore)]
84-
[TestCategory(TestCategories.Arlington)]
85-
#if IGNORE_FEDERATED
86-
[Ignore]
87-
#endif
88-
public async Task ARLINGTON_ROPC_ADFS_Async()
84+
public async Task Public_ROPC_Cloud_Async()
8985
{
90-
var labResponse = await LabUserHelper.GetArlingtonADFSUserAsync().ConfigureAwait(false);
91-
await RunHappyPathTestAsync(labResponse).ConfigureAwait(false);
86+
var aadUser = await LabUserHelper.GetDefaultUserAsync().ConfigureAwait(false);
87+
await RunHappyPathTestAsync(aadUser).ConfigureAwait(false);
9288
}
9389

9490
[RunOn(TargetFrameworks.NetCore)]

tests/Microsoft.Identity.Test.Integration.netcore/SeleniumTests/DeviceCodeFlowIntegrationTest.cs

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -62,19 +62,6 @@ public async Task ArlingtonDeviceCodeFlowTestAsync()
6262
await AcquireTokenWithDeviceCodeFlowAsync(labResponse, "aad user").ConfigureAwait(false);
6363
}
6464

65-
[TestMethod]
66-
[Timeout(2 * 60 * 1000)] // 2 min timeout
67-
[TestCategory(TestCategories.ADFS)]
68-
#if IGNORE_FEDERATED
69-
[Ignore]
70-
#endif
71-
public async Task DeviceCodeFlowAdfsTestAsync()
72-
{
73-
LabResponse labResponse = await LabUserHelper.GetAdfsUserAsync(FederationProvider.ADFSv2019, true).ConfigureAwait(false);
74-
75-
await AcquireTokenWithDeviceCodeFlowAsync(labResponse, "adfs user").ConfigureAwait(false);
76-
}
77-
7865
[TestMethod]
7966
[Timeout(2 * 60 * 1000)] // 2 min timeout
8067
[TestCategory(TestCategories.Arlington)]

tests/Microsoft.Identity.Test.LabInfrastructure/LabUserHelper.cs

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -75,6 +75,15 @@ public static Task<LabResponse> GetDefaultUser2Async()
7575
return GetLabUserDataAsync(UserQuery.PublicAadUser2Query);
7676
}
7777

78+
/// <summary>
79+
/// Returns the AAD cloud user [email protected]
80+
/// </summary>
81+
/// <returns></returns>
82+
public static Task<LabResponse> GetDefaultUser3Async()
83+
{
84+
return GetLabUserDataAsync(UserQuery.PublicAadUser3Query);
85+
}
86+
7887
public static Task<LabResponse> GetMsaUserAsync()
7988
{
8089
return GetLabUserDataAsync(UserQuery.MsaUserQuery);

tests/Microsoft.Identity.Test.LabInfrastructure/UserQueryParameters.cs

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,11 @@ public struct UserQuery
3535
3636
};
3737

38+
public static UserQuery PublicAadUser3Query => new UserQuery()
39+
{
40+
41+
};
42+
3843
public static UserQuery MsaUserQuery => new UserQuery
3944
{
4045
UserType = LabInfrastructure.UserType.MSA

0 commit comments

Comments
 (0)