change test.yml and makefile

[S1nus]

Overview

The Code Coverage action appears to be broken, this is an attempt to fix it.

Fix inspired by https://github.com/celestiaorg/celestia-node/pull/2249/files

Checklist

• New and updated code has appropriate documentation
• New and updated code has new and/or updated testing
• Required CI checks are passing
• Visual proof for any user facing features like CLI or documentation updates
• Linked issues closed with keywords

[codecov]

Codecov Report

Patch coverage has no change and project coverage change: -1.87 ⚠️

Comparison is base (8b1f9b4) 56.17% compared to head (d509177) 54.31%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #968      +/-   ##
==========================================
- Coverage   56.17%   54.31%   -1.87%     
==========================================
  Files          68       54      -14     
  Lines       10915     6032    -4883     
==========================================
- Hits         6131     3276    -2855     
+ Misses       3901     2408    -1493     
+ Partials      883      348     -535     

see 39 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

[MSevey]

LGTM, yes the failure is because it thinks the coverage went down.
I can admin merge this once other's approve and then update the required CI to not include the remove test step