decentraland / lamb2 / 27842549301

83%

main: 83%

Build Type

Pull #466

github

Committed by LautaroPetaccio

Commit Message

chore: replace thegraph-component and migrate to native fetch

Continue replacing WKC packages with their core-components equivalents and
adopt the native-fetch stack now that dcl-catalyst-client supports it.

thegraph:
- @well-known-components/thegraph-component -> @dcl/thegraph-component

fetch / catalyst:
- bump dcl-catalyst-client to ^21.11.0, whose IFetchComponent accepts both
  the node-fetch and native (@dcl/core-commons) fetchers
- replace the custom node-fetch port (src/ports/fetch.ts) with
  @dcl/fetch-component (native, core-commons-typed)
- type the app-wide `fetch` as the @dcl/core-commons IFetchComponent, which
  lets @dcl/thegraph-component receive a native fetch directly (drops the
  earlier `as unknown as IFetchComponent` bridge in the-graph.ts)

Two third-party boundaries still need a localized, documented adaptation:
- eth-connect's HTTPProvider expects a loosely-typed FetchFunction
  (mode?: string, Promise<any>) that is looser than the native fetch
- the WKC test-helpers createLocalFetchCompoment has no native equivalent

Tests retype their fetch mocks to @dcl/core-commons and use the global
native Response.

Pull Request Pull Request #466: chore: replace WKC packages with core-components equivalents + native fetch

Coverage Stats

720 of 962 branches covered (74.84%)

Branch coverage included in aggregate %.

10 of 10 new or added lines in 4 files covered. (100.0%)

6 existing lines in 1 file now uncovered.

2028 of 2342 relevant lines covered (86.59%)

47.72 hits per line