try different test reporter

[michaelneale]

may be a bad mocha test reporter

[changeset-bot]

⚠️ No Changeset found

Latest commit: 5c0ca32

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[github-actions]

TBDocs Report

🛑 Errors: 0
⚠️ Warnings: 3

@tbdex/protocol

• Project entry file: packages/protocol/src/main.ts

@tbdex/http-client

• Project entry file: packages/http-client/src/main.ts

@tbdex/http-server

• Project entry file: packages/http-server/src/main.ts

📄 File: packages/http-server/src/http-server.ts
⚠️ extractor:ae-unresolved-link: The @link reference could not be resolved: The package "@tbdex/http-server" does not have an export "InMemoryOfferingsApi" #L49
⚠️ extractor:ae-unresolved-link: The @link reference could not be resolved: The package "@tbdex/http-server" does not have an export "InMemoryExchangesApi" #L49
⚠️ extractor:ae-unresolved-link: The @link reference could not be resolved: The package "@tbdex/http-server" does not have an export "InMemoryBalancesApi" #L49

---

TBDocs Report Updated at 2024-06-14T02:14:07Z 5c0ca32

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.71%. Comparing base (8d8f022) to head (5c0ca32).
Report is 4 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff             @@
##             main     #255       +/-   ##
===========================================
+ Coverage   83.50%   95.71%   +12.21%     
===========================================
  Files          42       42               
  Lines        3547     3528       -19     
  Branches      289      387       +98     
===========================================
+ Hits         2962     3377      +415     
+ Misses        585      151      -434     

Components	Coverage Δ
protocol	94.90% <0.00%> (ø)
http-client	100.00% <ø> (ø)
http-server	94.65% <100.00%> (+46.25%)	⬆️

[michaelneale]

fix for: #252 and fixes the build

[jiyoonie9]

what are the consequences if we get rid of this step? i feel like either @leordev or @finn-tbd set this up for some dashboard view

[jiyoonie9]

i want to say it's related to this page https://tbd54566975.github.io/sdk-report-runner/ but could be totally wrong

[michaelneale]

yeah - would like their input, as not sure. Could convert it to XML another way?

[michaelneale]

I tried updating to latest mocha and reporter, same issue.

[jiyoonie9]

It could be due to something other than the test runner. The test that flaky fails on CI and local seems to always be the one that parses in the test vector as JSON due to timeout.

[michaelneale]

yeah it is likely uncovering some other problem.

[michaelneale]

Ah just saw this locally:

So it isn't really a fix, just hiding some timeout.

[finn-block]

Consequences of getting rid of this step are indeed related to that page. It will no longer be able to display information about the tbdex-js, and just fully removing these steps may fully break the compliance report runner until it's updated to stop checking tbdex-js.

[michaelneale]

I don't think this fixes it yet BTW< saw this locally:

[michaelneale]

closing this as it didn't really solve it. Trying this approach:

#256