Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
One Line Summary
IAMs will no longer be displayed when running for Mac Catalyst on an M1 Mac.
Details
Our logic to not display IAMs on mac catalyst was not working for M1 Macs, because it was directly looking at the architecture. This PR now properly checks the compile target and will not display IAMs for catalyst.
Motivation
bug
Scope
This affects the
getDeviceVariant
method, which was also incorrect for simulators on M1 Macs.Testing
Unit testing
I have fixed the unit tests for this, but they now need to be run on specific device targets to test every case since I could not mock the compile target.
Manual testing
I tested on the M1 Mac in the office as well as my intel mac for simulator, catalyst, and device.
Affected code checklist
Checklist
Overview
Testing
Final pass
This change is