node icon indicating copy to clipboard operation
node copied to clipboard

Add test for #55913

Open gorankarlic opened this issue 1 year ago • 3 comments

Added test for #55913 that is expected to fail until issue is fixed. Please approve MacOS testing CI to verify test is failing as expected.

gorankarlic avatar Dec 01 '24 09:12 gorankarlic

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 87.99%. Comparing base (3f9c6c0) to head (03ae6ae).

Additional details and impacted files
@@           Coverage Diff           @@
##             main   #56096   +/-   ##
=======================================
  Coverage   87.99%   87.99%           
=======================================
  Files         656      656           
  Lines      189000   189000           
  Branches    35992    35991    -1     
=======================================
+ Hits       166308   166318   +10     
+ Misses      15851    15843    -8     
+ Partials     6841     6839    -2     

see 18 files with indirect coverage changes

codecov[bot] avatar Dec 01 '24 14:12 codecov[bot]

Can you fix the linter issue and move this to known_issues directory?

aduh95 avatar Dec 01 '24 18:12 aduh95

Can you fix the linter issue and move this to known_issues directory?

Fixed the linter issues.

@aduh95 I plan to fix the issue once I can confirm that the test is failing. Should I still move the issue to known_issues directory?

If yes, what is the recommended way to deal with the test scaffolding from test-watch-mode.mjs - should I copy it over or write a test from scratch that does not use the existing test scaffolding (either seems cumbersome)?

gorankarlic avatar Dec 02 '24 04:12 gorankarlic