Move all Clipboard related tests to a new class and remove SkipOnArchitecture label

[LeafShi1]

Related #11227

Proposed changes

• Move all Clipboard related tests to a new class and remove SkipOnArchitecture label

Microsoft Reviewers: Open in CodeFlow

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 74.56461%. Comparing base (b3b877e) to head (c78800e).
Report is 13 commits behind head on main.

Additional details and impacted files

@@                 Coverage Diff                 @@
##                main      #11606         +/-   ##
===================================================
+ Coverage   74.56266%   74.56461%   +0.00195%     
===================================================
  Files           3040        3041          +1     
  Lines         629551      629493         -58     
  Branches       46854       46835         -19     
===================================================
- Hits          469410      469379         -31     
+ Misses        156784      156750         -34     
- Partials        3357        3364          +7     

Flag	Coverage Δ
Debug	74.56461% <100.00000%> (+0.00195%)	⬆️
integration	17.97987% <ø> (-0.01427%)	⬇️
production	47.49228% <ø> (-0.02751%)	⬇️
test	96.97886% <100.00000%> (+0.02177%)	⬆️
unit	44.52792% <ø> (-0.02846%)	⬇️

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

[paul1956]

As part of the VB Runtime there are clipboard access functions and a ClipboardProxy class. There are very few existing tests and they have the some issue you are trying to resolve. As part of my Code Cleanup PR, I added 100% code coverage for the VB Runtime but that will conflict/clash with this PR. Sent from my iPhoneI apologize for any typos Siri might have made.(503) 803-6077On Jun 27, 2024, at 11:58 PM, codecov[bot] ***@***.***> wrote: Codecov Report All modified and coverable lines are covered by tests ✅ Project coverage is 96.97886%. Comparing base (b3b877e) to head (c78800e). Additional details and impacted files @@ Coverage Diff @@ ## main #11606 +/- ## ==================================================== + Coverage 74.56266% 96.97886% +22.41620% ==================================================== Files 3040 1069 -1971 Lines 629551 344373 -285178 Branches 46854 4983 -41871 ==================================================== - Hits 469410 333969 -135441 + Misses 156784 9694 -147090 + Partials 3357 710 -2647 Flag Coverage Δ Debug 96.97886% <100.00000%> (+22.41620%) ⬆️ integration ? production ? test 96.97886% <100.00000%> (+0.02177%) ⬆️ unit ? Flags with carried forward coverage won't be shown. Click here to find out more. —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>

[Tanya-Solyanik]

Thank you!

[Tanya-Solyanik]

@paul1956 - could you please describe the conflict?

but that will conflict/clash with this PR.

[paul1956]

Sent from my iPhoneI apologize for any typos Siri might have made.(503) 803-6077On Jun 28, 2024, at 11:03 AM, Tanya Solyanik ***@***.***> wrote: @paul1956 - could you please describe the conflict? but that will conflict/clash with this PR.I will merge your changes and see what happens then I will know more.  What I know now is none of the new and existing Visual Basic Runtime Clipboard/ClopboardProxy tests are in your new project and most are written in VB. —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: ***@***.***>

[LeafShi1]

Sent from my iPhoneI apologize for any typos Siri might have made.(503) 803-6077On Jun 28, 2024, at 11:03 AM, Tanya Solyanik @.> wrote: @paul1956 - could you please describe the conflict? but that will conflict/clash with this PR.I will merge your changes and see what happens then I will know more.  What I know now is none of the new and existing Visual Basic Runtime Clipboard/ClopboardProxy tests are in your new project and most are written in VB. —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.>

Ok, I will keep this PR open.

[Tanya-Solyanik]

and existing Visual Basic Runtime Clipboard/ClopboardProxy tests are in your new project and most are written in VB.

@paul1956 - I don't think this is a problem, we are not synchronizing tests from different assemblies in this PR. This PR adds some clipboard related tests to the synchronized collection within a single test assembly. There might be some other tests that we hadn't added yet, for example in the datagridview area.

[paul1956]

@Tanya-Solyanik I looked into this more and I handle "synchronizing tests from different assemblies" for the VB tests. I will test to verify issue is fixed once you merge.
Thanks.