Looks good! these tests need a lot of setup to run locally so let me run a quick test to see if everything keeps working fine.

This breaks Darc-Command-Impl :(

@riarenas Tests are flaky but I'm able to get all of them to pass locally. You good with merging this?

I tried running channels.ps1, and got:

fail: Microsoft.DotNet.Darc.Operations.Operation[0]       Could not find channel with name '238743518'
WARNING: Failed to delete channel 238743518
WARNING: Darc command exited with exit code: 42

During the final teardown function.
This means we are leaking test channels every time we run the test. More single quote issues 😢

EDIT: aaaand that's totally expected because the channel tests delete the channels they create as part of the test. Nothing to see here.

Hey @jonfortescue, the tests are failing consistently after this PR merged. It's not flakiness. (also, you didn't squash merge :( )

The azdoflow-nonbatched-all-checks-successful test is broken because the merge policy is getting generated incorrectly.

"mergePolicies": [
      {
        "name": "AllChecksSuccessful",
        "properties": {
          "ignoreChecks": [
            "'Work",
            "item",
            "linking'"
          ]
        }

should look like this:

"mergePolicies": [
      {
        "name": "AllChecksSuccessful",
        "properties": {
          "ignoreChecks": [
            "Work item linking"
          ]
        }

The args are getting split by spaces. This is the one place where we don't want that. This is causing the PRs opened by the test to not be merged because the IgnoreChecks are not correct. The Github allChecksSuccessful is working because the ignoreCheck for that one does not include spaces.