azp: iif allow complex objects / disable short circuit (#535)

Author: ChristopherHX

src/Sdk/DTExpressions2/Expressions2/Sdk/Functions/v1/Iif.cs

@@ -8,11 +8,11 @@ internal sealed class Iif : Function
 
         protected sealed override Object EvaluateCore(EvaluationContext context, out ResultMemory memory)
         {
-            // TODO learn the semantics of this function
             memory = null;
             bool cond = Parameters[0].EvaluateBoolean(context);
-            String body = Parameters[cond ? 1 : 2].EvaluateString(context) as String ?? String.Empty;
-            return body;
+            EvaluationResult then = Parameters[1].Evaluate(context);
+            EvaluationResult otherwise = Parameters[2].Evaluate(context);
+            return cond ? then.Raw ?? then.Value : otherwise.Raw ?? otherwise.Value;
         }
     }
 }

…ines/iff-functions-ado-2025/pipeline.yml …ines/iif-functions-ado-2025/pipeline.ymltestworkflows/azpipelines/iff-functions-ado-2025/pipeline.yml renamed to testworkflows/azpipelines/iif-functions-ado-2025/pipeline.yml testworkflows/azpipelines/iff-functions-ado-2025/pipeline.yml renamed to testworkflows/azpipelines/iif-functions-ado-2025/pipeline.yml

@@ -0,0 +1,28 @@
+# src: https://learn.microsoft.com/en-us/azure/devops/release-notes/2025/pipelines/sprint-248-update#new-pipeline-expression-functions
+parameters:
+  - name: test
+    type: object
+    default:
+      pool: ManagedDevOpsPool
+  - name: test2
+    type: object
+    default:
+      pool: Azure Devops
+variables:
+  poolToUse: ${{ iif(eq(variables['Build.Reason'], 'PullRequest'), parameters.test2, parameters.test).pool}}
+  poolToUse2: ${{ iif(ne(variables['Build.Reason'], 'PullRequest'), parameters.test2, parameters.test).pool}}
+
+stages:
+- stage: build
+  pool: ${{variables.poolToUse}}
+  jobs:
+  - job:
+    steps:   
+    - task: DotNetCoreCLI@2
+      inputs:
+        command: 'build'
+    - ${{ if ne(variables.poolToUse2, 'Azure Devops') }}:
+      - assert: failure
+    
+    - ${{ if ne(variables.poolToUse, 'ManagedDevOpsPool') }}:
+      - assert: failure

testworkflows/azpipelines/iif-functions-ado-complex-object-2025/pipeline.yml

@@ -0,0 +1,23 @@
+parameters:
+  - name: test
+    type: object
+    default:
+      pool: ManagedDevOpsPool
+  - name: test2
+    type: object
+    default:
+      pool: Azure Devops
+variables:
+  poolToUse: ${{ iif(eq(variables['Build.Reason'], 'PullRequest'), parameters.test2, parameters.test).pool}}
+
+stages:
+- stage: build
+  pool: ${{variables.poolToUse}}
+  jobs:
+  - job:
+    steps:   
+    - task: DotNetCoreCLI@2
+      inputs:
+        command: 'build'
+    - ${{ if ne(variables.poolToUse, 'ManagedDevOpsPool') }}:
+      - assert: failure

testworkflows/azpipelines/iif-functions-ado-no-short-circuit-2-2025/pipeline.yml

@@ -0,0 +1,25 @@
+# ExpectedException: TemplateValidationException
+# ExpectedMessage: parameters.test3
+parameters:
+  - name: test
+    type: object
+    default:
+      pool: ManagedDevOpsPool
+  - name: test2
+    type: object
+    default:
+      pool: Azure Devops
+variables:
+  poolToUse: ${{ iif(eq(variables['Build.Reason'], 'PullRequest'), parameters.test3, parameters.test).pool}}
+
+stages:
+- stage: build
+  pool: ${{variables.poolToUse}}
+  jobs:
+  - job:
+    steps:   
+    - task: DotNetCoreCLI@2
+      inputs:
+        command: 'build'
+    - ${{ if ne(variables.poolToUse, 'ManagedDevOpsPool') }}:
+      - assert: failure