decentraland / builder-server / 39875428942

77%

master: 77%

Build Type

Pull #825

github

Committed by RocioCM

Commit Message

fix: address decentraland-bot review findings

- review_type compared a NULL contract address (third-party, unpublished) to the on-chain sets and got NULL, so those rows matched neither bucket, or all matched new when the sets were empty; COALESCE both sides to false, same as getStatusStatement, so they read as new
- the status conditions get the same COALESCE on their on-chain checks
- to_review's third-party branch now needs a pending request or none, as the docs say, instead of any unassigned curation
- /addresses gets the multi-value filter pass-through and the 400 cases; the review_type specs pin the COALESCE

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

Pull Request Pull Request #825: feat: multi-value review filters, review type and request-date sort for the collections list

Coverage Stats

1933 of 2684 branches covered (72.02%)

Branch coverage included in aggregate %.

69 of 72 new or added lines in 5 files covered. (95.83%)

61 existing lines in 3 files now uncovered.

3866 of 4880 relevant lines covered (79.22%)

58.41 hits per line