Skip to content

api: add test — GET /assets/:id returns 404 for an unknown asset id - #332

Open
gregthelma8-eng wants to merge 2 commits into
RWA-ToolKit:mainfrom
gregthelma8-eng:main
Open

api: add test — GET /assets/:id returns 404 for an unknown asset id#332
gregthelma8-eng wants to merge 2 commits into
RWA-ToolKit:mainfrom
gregthelma8-eng:main

Conversation

@gregthelma8-eng

Copy link
Copy Markdown

closes #190
closes #191
closes #192
closes #193

RWA-ToolKit#190: add boundary-id tests (0 and u64::MAX) that assert 404 status
and the documented ApiErrorBody shape using test_support::state_with

RWA-ToolKit#191: add list field-set test for GET /assets — serializes a single
item and diffs its keys against the OpenAPI Asset required list so
schema drift is caught in CI

RWA-ToolKit#192: covered by existing list_assets_returns_empty_array_not_null_when_no_assets
RWA-ToolKit#193: covered by existing get_asset_by_unknown_id_returns_404
@drips-wave

drips-wave Bot commented Sep 1, 2026

Copy link
Copy Markdown

@gregthelma8-eng Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment