Tests: rename InstantRunTest to FastDevTest and drop duplicate

[jonathanpeppers]

Why rename?

The "Instant Run" name in tests/MSBuildDeviceIntegration/Tests/InstantRunTest.cs is leftover Android Studio jargon from ~2016. We never implemented the Android Studio Instant Run protocol — these tests have always exercised our Fast Deployment pipeline (_BuildApkFastDev + _Upload), which pushes managed assemblies to the on-device override directory so an assembly-only edit can skip repackaging/reinstalling the APK.

To make matters worse, the older Enhanced Fast Dev path (which actually did slice/sideload dex on device) was removed. So the InstantRun name suggests behavior we never had and sits next to behavior we no longer have. Renaming to FastDev matches the targets the tests actually assert against (_BuildApkFastDev) and the rest of our docs/codebase.

Before	After
InstantRunTest.cs	FastDevTest.cs
class InstantRunTest	class FastDevTest
InstantRunSimpleBuild	FastDevSimpleBuild
InstantRunFastDevDexes	FastDevWithEmbeddedDex
temp/InstantRunTargetsSkipped_*	temp/FastDevTargetsSkipped_*

InstantRunFastDevDexes also said "FastDev" twice — the test isn't really about dexes, it's about Fast Dev install/launch interacting with the android:useEmbeddedDex manifest attribute, so FastDevWithEmbeddedDex describes it more honestly.

.github/skills/tests/references/test-catalog.md filter alias was updated to match.

Why remove InstantRunResourceChange?

It is fully duplicative of SkipFastDevAlreadyInstalledResources:

	InstantRunResourceChange (removed)	SkipFastDevAlreadyInstalledResources (kept)
Edit applied	LinearLayout → RelativeLayout in Main.axml	Same
Install + reinstall flow	Yes	Yes
Assertion strategy	Greps build log for _BuildApkFastDev / _Upload target names	Cracks open the packaged_resources zip and asserts the new layout bytes are actually present
Catches a real regression that the other doesn't	No	Yes — if the resources weren't actually re-packaged, the kept test fails; the removed test would still pass as long as the targets ran

The kept test is strictly stronger. Anything the removed test could detect (Fast Dev targets running on a resource edit) is implied by the kept test passing.

Test impact

No behavior change in product code — this is rename + delete-one-redundant-test in tests/.

[Copilot]

Pull request overview

This PR updates device integration tests to reflect the actual feature under test: Fast Deployment (via _BuildApkFastDev + _Upload), replacing the outdated “Instant Run” naming and removing a redundant resource-change test.

Changes:

• Renames the InstantRunTest fixture and key test methods to FastDev* to match the Fast Deployment pipeline being asserted.
• Removes InstantRunResourceChange as duplicative of the stronger SkipFastDevAlreadyInstalledResources test.
• Updates the test-catalog filter alias from InstantRunTest to FastDevTest.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
tests/MSBuildDeviceIntegration/Tests/FastDevTest.cs	Renames the test fixture/methods to FastDev* and deletes the redundant resource-change test.
.github/skills/tests/references/test-catalog.md	Updates the documented filter alias to FullyQualifiedName~FastDevTest and adjusts the description accordingly.