decentraland / comms-gatekeeper / 25228208712

85%

main: 85%

Pull #261

github

test: add lands mock helper and rewrite the lands spec to convention

introduces test/mocks/lands-mock.ts (createLandsMockedComponent) so
consumers don't need to inline the full ILandComponent shape.
room-metadata-sync.spec.ts now uses it instead of the local literal.

rewrites test/unit/lands.spec.ts to match the dcl-testing convention:
- describe blocks lead with "when ..." for each context and "and ..."
  for nested cases; it() descriptions describe the expected behavior
  ("should return ...", "should throw ...") with no "when" leakage.
- variables, fixtures and mocks live in beforeEach inside the describe
  that uses them, not in it() blocks or top-level constants.
- afterEach resets mocks at the top level.
- one assertion per it() except where two assertions belong to the
  same observed behavior (e.g., concurrent dedup test asserts both the
  fetch count and the equal payload).

no behavioral coverage change — same 30 lands tests, same 30
room-metadata-sync tests.

1061 of 1369 branches covered (77.5%)

Branch coverage included in aggregate %.

169 of 175 new or added lines in 15 files covered. (96.57%)

2993 of 3386 relevant lines covered (88.39%)

75.1 hits per line