Code Coverage for ClipboardProxy from PR #11863

[paul1956]

Fixes #

Proposed changes

• Add Clipboard Proxy Tests from PR Add tests for VB runtime REVIEW 1ST DRAFT #11863

Customer Impact

• Improved code quality

Regression?

• No

Risk

• None add additional tests

Microsoft Reviewers: Open in CodeFlow

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 75.56431%. Comparing base (0469577) to head (946f18e).

Additional details and impacted files

@@                 Coverage Diff                 @@
##                main      #12227         +/-   ##
===================================================
+ Coverage   75.55840%   75.56431%   +0.00591%     
===================================================
  Files           3111        3112          +1     
  Lines         635122      635157         +35     
  Branches       46914       46917          +3     
===================================================
+ Hits          479888      479952         +64     
+ Misses        151775      151751         -24     
+ Partials        3459        3454          -5     

Flag	Coverage Δ
Debug	75.56431% <100.00000%> (+0.00591%)	⬆️
integration	18.29755% <ø> (+0.01411%)	⬆️
production	49.04142% <ø> (+0.01020%)	⬆️
test	97.03109% <100.00000%> (+0.00030%)	⬆️
unit	45.97334% <ø> (-0.01654%)	⬇️

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

[paul1956]

@Tanya-Solyanik based on all the work you are doing on Clipboard should I close this? If there are still uncover ClipboardProxy areas I can open a new PR after it's merged.

[paul1956]

@Tanya-Solyanik
The following ClipboardProxy functions are currently missing coverage and the ones I was testing with this PR.

Friend Sub New()
Public Function GetAudioStream() As Stream
Public Function ContainsText(format As TextDataFormat) As Boolean
Public Function GetAudioStream() As Stream
Public Function GetFileDropList() As StringCollection
Public Sub SetAudio(audioBytes As Byte())
Public Sub SetFileDropList(filePaths As StringCollection)

If you have these covered or don't think they need to be I will close.

[Tanya-Solyanik]

Suggested change

	''' These are just checking the Proxy functions, the underlying functions are tested elsewhere.
	''' These are just checking the <see cref="ClipboardProxy" /> functions, the underlying functions are tested elsewhere.

[Tanya-Solyanik]

This test is equivalent to

winforms/src/Microsoft.VisualBasic/tests/UnitTests/Microsoft/VisualBasic/MyServices/ClipboardProxyTests.cs

Line 84 in 3760721

public void Text()

[Tanya-Solyanik]

If you have these covered or don't think they need to be I will close.

These are covered as exposed from the Clipboard class, we still need tests on the Proxy level that establish that it invokes the CLipboard methods or at least gets the same results as the Clipboard does.