search

Agoric / agoric-sdk

monorepo for the Agoric Javascript smart contract platform
Apache License 2.0
328 stars 207 forks source link

coverage report untested files (and misc cleanup) #10556

Closed turadg closed 1 week ago

turadg commented 1 week ago

incidental

Description

The main change in this PR is to make the code coverage reports include files that aren't tested at all (or apparently aren't due to the bundling problem #1817)

It also cleans up a bunch of obsolete stuff about NESM and adds some more usage docs.

Security Considerations

none

Scaling Considerations

n/a

Documentation Considerations

Improves

Testing Considerations

I ran the commands locally to verify the reports. Once this lands they'll appear at https://agoric-sdk-coverage.netlify.app/

Upgrade Considerations

n/a

cloudflare-workers-and-pages[bot] commented 1 week ago

Deploying agoric-sdk with  Cloudflare Pages  Cloudflare Pages

Latest commit: 8efce30
Status: ✅  Deploy successful!
Preview URL: https://b676ca46.agoric-sdk.pages.dev
Branch Preview URL: https://ta-codecov-cleanup.agoric-sdk.pages.dev

View logs

turadg commented 6 days ago

Turns out the master CI job that calls test:c8-all depended on C8_OPTIONS. Fix in the works.