Recached

decentraland / squid-management-server
54%
main: 37%

LAST BUILD BRANCH: feat/purge-old-schemas
DEFAULT BRANCH: main
Repo Added
Token oEexvDtXSSCCc25NgF88IrQRaATI8Tpwi regen
Build 126 Last
Files 13
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH feat/purge-old-schemas
branch: SELECT
CHANGE BRANCH
x
Sync Branches

coverage: 53.729% (+8.2%) from 45.54%
24802458721

Pull #75

github

LautaroPetaccio 
test: cover invalid-name, DROP failure, credits DB, and guard throws

Close the test coverage gaps flagged in review. All four cases now
have their own 'when/and' describe blocks following the project's
dcl-testing conventions.

- olderThanMs <= 0: asserts the guard throws before any ECS or
  database call.
- invalid-name: an 'squid_old-with-hyphen' fixture exercises the
  SAFE_SCHEMA_NAME defensive gate. Asserts the schema is reported
  under skipped with reason 'invalid-name' and no transaction opens.
- DROP failure: dappsMock.withTransaction.mockImplementationOnce
  throws for the one candidate. Asserts the failure is not listed
  under deleted, the transaction was attempted, and the skip
  reasons for the other candidates are still surfaced — which is
  the "one broken DROP doesn't abort the run" contract the per-
  schema try/catch added earlier.
- Credits DB: populates creditsMock with its own orphan schema,
  asserts result.deleted contains a `database: 'credits'` entry
  and the transaction opens on creditsMock, not dappsMock.

One additional small assertion in the existing dry-run case that
result.dryRun is `true` — the field is new and now covered.

Tests grow from 40 to 50 passing.
Pull Request #75: Purge old, unused squid schemas on a daily schedule

80 of 168 branches covered (47.62%)

Branch coverage included in aggregate %.

86 of 105 new or added lines in 3 files covered. (81.9%)

1 existing line in 1 file now uncovered.

237 of 422 relevant lines covered (56.16%)

7.32 hits per line

Relevant lines Covered
422 RELEVANT LINES 237 COVERED LINES
7.32 HITS PER LINE
Source Files on main

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
24802458721 feat/purge-old-schemas test: cover invalid-name, DROP failure, credits DB, and guard throws Close the test coverage gaps flagged in review. All four cases now have their own 'when/and' describe blocks following the project's dcl-testing conventions. - olderThanMs <= 0... Pull #75 LautaroPetaccio github
53.73
24801025979 feat/purge-old-schemas docs: document purge API and log every drop, would-drop, and skip Address two gaps reviewers flagged: 1. Public types and methods for the schema purge were thinly or inconsistently documented while the extracted queries in queries.ts alrea... Pull #75 LautaroPetaccio github
51.57
24799175807 feat/purge-old-schemas feat: add dry-run-purge script for manual validation Add scripts/dry-run-purge.ts so operators can see exactly what the daily schema purge would do against a live environment without executing anything destructive. Run with: SCHEMA_PURGE_MAX... Pull #75 LautaroPetaccio github
51.33
24798481661 feat/purge-old-schemas refactor: extract purge queries to queries.ts and reuse existing ones The queries used by purgeOldSchemas were inlined in component.ts; move them next to the other SQL builders and, while doing so, drop a duplicate of the already-existing getSche... Pull #75 LautaroPetaccio github
51.33
24796790408 feat/purge-old-schemas test: restructure specs per dcl-testing standards Reorganize both unit test files to match the project's testing conventions: describe blocks use "when" for scenarios and "and" for nested branches, each it() asserts one behavior with a specific d... Pull #75 LautaroPetaccio github
50.27
24796509831 feat/purge-old-schemas chore: make schema purge job interval configurable via env Add SCHEMA_PURGE_INTERVAL_HOURS env var so the cadence can be tuned without a code change. Defaults to 24 when unset or non-positive, so existing behaviour is preserved. Documented in .en... Pull #75 LautaroPetaccio github
50.27
24796413248 feat/purge-old-schemas chore: document SCHEMA_PURGE_MAX_AGE_DAYS and read it at startup Hoist the SCHEMA_PURGE_MAX_AGE_DAYS read out of the daily job callback and into initComponents, so the feature's enable/disable state is resolved once at boot and logged with a clea... Pull #75 LautaroPetaccio github
51.09
24795970841 feat/purge-old-schemas feat: purge old, unused squid schemas on a daily schedule Add a new ISquidComponent.purgeOldSchemas(options) method that walks the dapps and credits databases looking for squid_* schemas that are older than a configurable threshold and no longer ... push LautaroPetaccio github
51.65
24787181135 chore/dependency-overhaul refactor: drop slack wrapper, consume @dcl/slack-component directly The thin wrapper at src/ports/slack/ added one method (sendFormattedMessage) that simply mapped to @dcl/slack-component's sendMessage with a default channel. Every caller was the... Pull #74 LautaroPetaccio github
45.54
24786422727 chore/dependency-overhaul feat: migrate to @dcl/job-component and separate monitor from wiring Replace the hand-rolled generic job runner in src/ports/job/ with @dcl/job-component, and invert the monitor's shape: squid-monitor.ts now exports the monitoring function instea... push LautaroPetaccio github
44.44
See All Builds (126)

Badge your Repo: squid-management-server

We detected this repo isn’t badged! Grab the embed code to the right, add it to your repo to show off your code coverage, and when the badge is live hit the refresh button to remove this message.
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

Refresh