Remove dependency on RemoteExecutor on some files

[ricardobossan]

Fixes #

Proposed changes

Customer Impact

Regression?

• Yes / No

Risk

Screenshots

Before

After

Test methodology

Accessibility testing

Test environment(s)

Microsoft Reviewers: Open in CodeFlow

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 76.12353%. Comparing base (0ce1218) to head (4612a1a).
Report is 18 commits behind head on main.

Additional details and impacted files

@@                 Coverage Diff                 @@
##                main      #12379         +/-   ##
===================================================
+ Coverage   76.03858%   76.12353%   +0.08495%     
===================================================
  Files           3181        3187          +6     
  Lines         639670      639990        +320     
  Branches       47215       47229         +14     
===================================================
+ Hits          486396      487183        +787     
+ Misses        149756      149269        -487     
- Partials        3518        3538         +20     

Flag	Coverage Δ
Debug	76.12353% <100.00000%> (+0.08495%)	⬆️
integration	18.16929% <ø> (-0.00311%)	⬇️
production	49.99701% <ø> (+0.16343%)	⬆️
test	97.03652% <100.00000%> (+0.00454%)	⬆️
unit	47.25564% <ø> (+0.20114%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

[ricardobossan]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[Tanya-Solyanik]

This should be in a try/finally block

[ricardobossan]

@Tanya-Solyanik I have re-enabled all tests that were previously skipped with the explanation:
Crash with AbandonedMutexException. See: https://github.com/dotnet/arcade/issues/5325.

Additionally, I restored the correct state of the Application.ParkingWindowTests.cs method by setting:
Control.CheckForIllegalCrossThreadCalls = true;.

Summary of Attempts:

• Approach 1:
  Removed RemoteExecutor only from Application.ParkingWindowTests.cs while simply re-enabling the other tests.

  • CI Checks Results

• Approach 2:
  Removed RemoteExecutor from all tests.

  • CI Checks Results

Both approaches passed local tests but failed automated CI checks.

For now, I’m pausing work on this issue to focus on higher-priority tasks. Let me know if you have any suggestions or would like me to revisit this sooner!

[Tanya-Solyanik]

We'll re-open when we are working on this again.