Fix PopTokenRequestContext ctor (Azure#43789)

Author: christothes

sdk/core/Azure.Core.Experimental/CHANGELOG.md

@@ -8,6 +8,8 @@
 
 ### Bugs Fixed
 
+- Fixed a bug in the `PopTokenRequestContext` constructor that caused the `IsProofOfPossessionEnabled` property to be ignored.
+
 ### Other Changes
 
 ## 0.1.0-preview.33 (2024-04-04)

sdk/core/Azure.Core.Experimental/src/PopTokenRequestContext.cs

@@ -30,6 +30,7 @@ public PopTokenRequestContext(string[] scopes, string? parentRequestId = default
             TenantId = tenantId;
             IsCaeEnabled = isCaeEnabled;
             ProofOfPossessionNonce = proofOfPossessionNonce;
+            IsProofOfPossessionEnabled = isProofOfPossessionEnabled;
             _request = request;
         }
 

sdk/core/Azure.Core.Experimental/tests/PopTokenRequestContextTests.cs

@@ -0,0 +1,41 @@
+// Copyright (c) Microsoft Corporation. All rights reserved.
+// Licensed under the MIT License.
+
+using System;
+using System.Net.Http;
+using Azure.Core;
+using Azure.Core.TestFramework;
+using NUnit.Framework;
+
+namespace Azure
+{
+    public class PopTokenRequestContextTests
+    {
+        [Test]
+        public void PopTokenRequestContextCtor()
+        {
+            var scopes = new string[] { "scope1", "scope2" };
+            var parentRequestId = Guid.NewGuid().ToString();
+            var claims = "claims";
+            var tenantId = Guid.NewGuid().ToString();
+            var isCaeEnabled = true;
+            var isProofOfPossessionEnabled = true;
+            var proofOfPossessionNonce = Guid.NewGuid().ToString();
+            var request = new MockRequest();
+            request.Method = RequestMethod.Get;
+            request.Uri.Reset(new Uri("http://example.com"));
+
+            var context = new PopTokenRequestContext(scopes, parentRequestId, claims, tenantId, isCaeEnabled, isProofOfPossessionEnabled, proofOfPossessionNonce, request);
+
+            Assert.AreEqual(scopes, context.Scopes);
+            Assert.AreEqual(parentRequestId, context.ParentRequestId);
+            Assert.AreEqual(claims, context.Claims);
+            Assert.AreEqual(tenantId, context.TenantId);
+            Assert.AreEqual(isCaeEnabled, context.IsCaeEnabled);
+            Assert.AreEqual(isProofOfPossessionEnabled, context.IsProofOfPossessionEnabled);
+            Assert.AreEqual(proofOfPossessionNonce, context.ProofOfPossessionNonce);
+            Assert.AreEqual(new HttpMethod(request.Method.ToString()), context.HttpMethod);
+            Assert.AreEqual(request.Uri.ToUri(), context.Uri);
+        }
+    }
+}