Move coverage upload to separate step

[Czaki]

This PR move coverage to separate steep nad add coverage summary information in action summary

[codecov]

Codecov Report

Merging #323 (39eef8d) into main (9d29e4d) will increase coverage by 0.24%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##             main      #323      +/-   ##
===========================================
+ Coverage   99.75%   100.00%   +0.24%     
===========================================
  Files          38        37       -1     
  Lines        2814      2774      -40     
===========================================
- Hits         2807      2774      -33     
+ Misses          7         0       -7     

Files	Coverage Δ
src/npe2/_inspection/_fetch.py	100.00% <100.00%> (+3.57%)	⬆️

[Czaki]

Ok. Windows tests are broken from a long time, but this reveals a much deeper problem as inspecting is using imports and imported packages are present in sys.modules after virtualenv removal.

[Czaki]

Codecov will be fixed once the new test plugin is released.

cc @DragaDoncila

[nclack]

Can you give me a bit more context around this removal? Was it always xfail-ing?

[Czaki]

there is no CI job with TEST_PACKAGE_NAME defined so it land in xfail branch.

[DragaDoncila]

Hmm smh now this test is failing because afaict the npe2 adapter doesn't convert npe1 themes? I don't really understand what this test is testing. I get that this is testing that fetching the manifest of an npe1 plugin works as expected, and includes its conversion to npe2, but it's odd because both the sample dict of dicts error and the theme error don't occur when launching the plugin in napari with the npe2 adapter enabled - which means it should be converting fine, no?

[DragaDoncila]

@Czaki I ended up just removing the theme from the npe1 plugin as well

[Czaki]

So it looks ready for review :)

[nclack]

lgtm! thanks!