Ignore CachedAuth if Broker is present in AuthMode on win 10 or 11 (#419)

* Ignore cached auth if broker is present on win 10 or 11

* Update changelog

Author: Haard30

CHANGELOG.md

@@ -5,6 +5,8 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
 ## [Unreleased]
+### Changed
+- Removed ChachedAuth mode if Broker is already present in auth modes on windows 10 or 11 since Broker already tries CachedAuth in a compliant way.
 
 ## [0.9.0] - 2024-11-07
 ### Removed

src/MSALWrapper.Test/AuthFlow/AuthFlowFactoryTest.cs

@@ -101,11 +101,11 @@ public void Broker_Only()
 
             IEnumerable<IAuthFlow> subject = this.Subject(AuthMode.Broker);
 
-            subject.Should().HaveCount(2);
+            subject.Should().HaveCount(1);
             subject
                 .Select(a => a.GetType())
                 .Should()
-                .ContainInOrder(typeof(CachedAuth), typeof(Broker));
+                .Contain(typeof(Broker));
         }
 
         [Test]
@@ -115,12 +115,11 @@ public void Windows10Or11_Defaults()
 
             IEnumerable<IAuthFlow> subject = this.Subject(AuthMode.Default);
 
-            subject.Should().HaveCount(3);
+            subject.Should().HaveCount(2);
             subject
                 .Select(a => a.GetType())
                 .Should()
                 .ContainInOrder(
-                    typeof(CachedAuth),
                     typeof(Broker),
                     typeof(Web));
         }
@@ -149,12 +148,11 @@ public void Windows10Or11_All()
 
             IEnumerable<IAuthFlow> subject = this.Subject(AuthMode.All);
 
-            subject.Should().HaveCount(5);
+            subject.Should().HaveCount(4);
             subject
                 .Select(a => a.GetType())
                 .Should()
                 .ContainInOrder(
-                    typeof(CachedAuth),
                     typeof(Broker),
                     typeof(Web),
                     typeof(DeviceCode));
@@ -228,13 +226,12 @@ public void AllModes_Windows10Or11()
             IEnumerable<IAuthFlow> subject = this.Subject(AuthMode.All);
 
             this.pcaWrapperMock.VerifyAll();
-            subject.Should().HaveCount(5);
+            subject.Should().HaveCount(4);
             subject
                 .Select(flow => flow.GetType())
                 .Should()
                 .BeEquivalentTo(new[]
                 {
-                    typeof(CachedAuth),
                     typeof(IntegratedWindowsAuthentication),
                     typeof(Broker),
                     typeof(Web),

src/MSALWrapper/AuthFlow/AuthFlowFactory.cs

@@ -37,11 +37,14 @@ public static IEnumerable<IAuthFlow> Create(
 
             // This is a list. The order in which flows get added is very important
             // as it sets the order in which auth flows will be attempted.
-            List<IAuthFlow> flows = new List<IAuthFlow>
+            List<IAuthFlow> flows = new List<IAuthFlow>();
+
+            // We skip CachedAuth if Broker is present in authMode on windows 10 or 11, since Broker 
+            // already tries CachedAuth with its PCAWrapper object built using withBroker(options).
+            if (!(authMode.IsBroker() && platformUtils.IsWindows10Or11()))
             {
-                // We always try cached auth first.
-                new CachedAuth(logger, authParams, preferredDomain, pcaWrapper),
-            };
+                flows.Add(new CachedAuth(logger, authParams, preferredDomain, pcaWrapper));
+            }
 
             // We try IWA as the first auth flow as it works for any Windows version
             // and tries to auth silently.