add disable discover enumeration = true for theory tests (#3042)

kllysng · web-flow · commit a89fc5d81985 · 2024-09-26T16:34:47.000-07:00
diff --git a/tests/Microsoft.Identity.Web.Test/AuthorityHelpersTests.cs b/tests/Microsoft.Identity.Web.Test/AuthorityHelpersTests.cs
@@ -120,9 +120,9 @@ public void EnsureAuthorityIsV2(string initialAuthority, string expectedAuthorit
         }
 
         [Theory]
-        [MemberData(nameof(GetAuthorityWithoutQueryIfNeededTheoryData))]
-        [MemberData(nameof(GetAuthorityWithoutQueryIfNeededExistingValuesTheoryData))]
-        [MemberData(nameof(GetAuthorityWithoutQueryIfNeededOverlappingExistingValuesTheoryData))]
+        [MemberData(nameof(GetAuthorityWithoutQueryIfNeededTheoryData), DisableDiscoveryEnumeration = true)]
+        [MemberData(nameof(GetAuthorityWithoutQueryIfNeededExistingValuesTheoryData), DisableDiscoveryEnumeration = true)]
+        [MemberData(nameof(GetAuthorityWithoutQueryIfNeededOverlappingExistingValuesTheoryData), DisableDiscoveryEnumeration = true)]
         public void GetAuthorityWithoutQueryIfNeeded(AuthorityHelpersTheoryData theoryData)
         {
             // arrange
diff --git a/tests/Microsoft.Identity.Web.Test/MsalTokenCacheProviderTests.cs b/tests/Microsoft.Identity.Web.Test/MsalTokenCacheProviderTests.cs
@@ -22,7 +22,7 @@ public void CreateTokenCacheSerializerTest()
             Assert.NotNull(tokenCacheProvider);
         }
 
-        [Theory, MemberData(nameof(CacheExpiryData))]
+        [Theory, MemberData(nameof(CacheExpiryData), DisableDiscoveryEnumeration = true)]
         public void CacheEntryExpiry_SetCorrectly_Test(TimeSpan? optionsCacheExpiry, DateTimeOffset? suggestedCacheExpiry, TimeSpan expectedCacheExpiry)
         {
             // Arrange

Original file line number	Diff line number	Diff line change
`@@ -120,9 +120,9 @@ public void EnsureAuthorityIsV2(string initialAuthority, string expectedAuthorit`
`120`	`120`	`}`
`121`	`121`
`122`	`122`	`[Theory]`
`123`		`- [MemberData(nameof(GetAuthorityWithoutQueryIfNeededTheoryData))]`
`124`		`- [MemberData(nameof(GetAuthorityWithoutQueryIfNeededExistingValuesTheoryData))]`
`125`		`- [MemberData(nameof(GetAuthorityWithoutQueryIfNeededOverlappingExistingValuesTheoryData))]`
	`123`	`+ [MemberData(nameof(GetAuthorityWithoutQueryIfNeededTheoryData), DisableDiscoveryEnumeration = true)]`
	`124`	`+ [MemberData(nameof(GetAuthorityWithoutQueryIfNeededExistingValuesTheoryData), DisableDiscoveryEnumeration = true)]`
	`125`	`+ [MemberData(nameof(GetAuthorityWithoutQueryIfNeededOverlappingExistingValuesTheoryData), DisableDiscoveryEnumeration = true)]`
`126`	`126`	`public void GetAuthorityWithoutQueryIfNeeded(AuthorityHelpersTheoryData theoryData)`
`127`	`127`	`{`
`128`	`128`	`// arrange`
Original file line number	Diff line number	Diff line change
`@@ -22,7 +22,7 @@ public void CreateTokenCacheSerializerTest()`
`22`	`22`	`Assert.NotNull(tokenCacheProvider);`
`23`	`23`	`}`
`24`	`24`
`25`		`- [Theory, MemberData(nameof(CacheExpiryData))]`
	`25`	`+ [Theory, MemberData(nameof(CacheExpiryData), DisableDiscoveryEnumeration = true)]`
`26`	`26`	`public void CacheEntryExpiry_SetCorrectly_Test(TimeSpan? optionsCacheExpiry, DateTimeOffset? suggestedCacheExpiry, TimeSpan expectedCacheExpiry)`
`27`	`27`	`{`
`28`	`28`	`// Arrange`