Azure.Provisioning: Avoid excess nulls in BicepStringBuilder (#46742)

Azure · Oct 22, 2024 · 8897cd4 · 8897cd4
1 parent 2deb63c
commit 8897cd4
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/sdk/provisioning/Azure.Provisioning.Storage/tests/BasicStorageTests.cs b/sdk/provisioning/Azure.Provisioning.Storage/tests/BasicStorageTests.cs
@@ -270,11 +270,12 @@ await test.Define(
                 infra.Add(new ProvisioningOutput("blobs_endpoint", typeof(string)) { Value = storage.PrimaryEndpoints.Value!.BlobUri });
 
                 // Manually compute the public Azure endpoint
+                string? nothing = null;
                 BicepValue<string> computed =
                     new BicepStringBuilder()
                     .Append("https://")
                     .Append($"{storage.Name}")
-                    .Append(".blob.core.windows.net");
+                    .Append($".blob.core.windows.net{nothing}");
                 infra.Add(new ProvisioningOutput("computed_endpoint", typeof(string)) { Value = computed });
 
                 return infra;

diff --git a/sdk/provisioning/Azure.Provisioning/src/Expressions/BicepStringBuilder.cs b/sdk/provisioning/Azure.Provisioning/src/Expressions/BicepStringBuilder.cs
@@ -102,10 +102,11 @@ public void AppendFormatted<T>(T t)
         }
         else
         {
-            _expressions.Add(
-                t is null ?
-                    BicepSyntax.Null() :
-                    BicepSyntax.Value(t.ToString() ?? ""));
+            string? s = t?.ToString();
+            if (s is not null)
+            {
+                _expressions.Add(BicepSyntax.Value(s));
+            }
         }
     }