Adding MacOS E2E Testing to CI Pipeline - #1903
Samuel Freiberg (samuelfreiberg) wants to merge 156 commits into
Conversation
|
Would like to hold this one till #1535 merges. Just had our sync and I'll try to get that merged ASAP, aka this week. |
Definitely, was planning on waiting. Just wanted to test the CI to make sure we only run this for MacOS and not iOS. The matrix part of the pipeline is a bit confusing to me, but I'm working on it :) |
| parameters: | ||
| relative_directory: $(relative_directory) | ||
|
|
||
| # - ${{ if eq(variables[relative_directory], 'macos')}}: |
There was a problem hiding this comment.
YML is hard to verify. locally so most likely you'll see a lot of small commits to try and debug the yml till it works
There was a problem hiding this comment.
Testing. Will mark this PR as a draft until I get it down
| - bash: echo Test 2 | ||
| condition: and(succeeded(), eq($(relative_directory), macos)) | ||
|
|
||
| - bash: echo Test 3 |
There was a problem hiding this comment.
what are we testing here?
There was a problem hiding this comment.
I'm going to remove this code, I just needed to test what syntax would work for certain conditions. I'm going to mark this PR as a draft until I figure it out :)
…tive into macosE2ECIPipeline # Conflicts: # .ado/azure-pipelines.yml # apps/fluent-tester/wdio.conf.macos.js # apps/macos/package.json
…he macos e2e testing
3 participants
Platforms Impacted
Description of changes
Adding MacOS E2E Testing to CI Pipeline
Verification
(how the change was tested, including both manual and automated tests)
Pull request checklist
This PR has considered (when applicable):