fix: Update tests to use new Error/Warning parameter order

Updated test calls to match the fixed parameter order:
- Error(text, condition, code) instead of Error(text, code, condition)
- Warning(text, condition, code) instead of Warning(text, code, condition)

This fixes the test failures caused by commit ed674db.

Author: JerrettDavis

test/JD.MSBuild.Fluent.Tests/CoverageTests.cs

@@ -56,8 +56,8 @@ public void Fluent_builders_and_renderer_cover_surface()
           .Comment("inside-target")
           .Message("hello", "Low", "'m'=='1'")
           .Exec("echo hi", "C:\\", "'e'=='1'")
-          .Error("err", "E1", "'er'=='1'")
-          .Warning("warn", "W1", "'w'=='1'")
+          .Error("err", "'er'=='1'", "E1")
+          .Warning("warn", "'w'=='1'", "W1")
           .PropertyGroup("'p'=='1'", g => g.Property<PropertyPG>("V"), label: "Tpg")
           .ItemGroup("'i'=='1'", g => g.Include<MsBuildItemTypes.None>("x.txt"), label: "Tig")
           .Task(new DoStuffTaskName(), task => task

test/JD.MSBuild.Fluent.Tests/EfcptCanonicalParityTests.cs

@@ -400,10 +400,10 @@ private static void RewriteTargetElement(TargetBuilder builder, MsBuildTargetEle
         builder.Exec(exec.Command, exec.WorkingDirectory, exec.Condition);
         break;
       case MsBuildErrorStep err:
-        builder.Error(err.Text, err.Code, err.Condition);
+        builder.Error(err.Text, err.Condition, err.Code);
         break;
       case MsBuildWarningStep warn:
-        builder.Warning(warn.Text, warn.Code, warn.Condition);
+        builder.Warning(warn.Text, warn.Condition, warn.Code);
         break;
       case MsBuildTaskStep task:
         builder.Task(new MsBuildTaskReference(task.TaskName), t => RewriteTask(t, task), task.Condition);