Skip to content

HDDS-9795. [Recon][UI][Test] Add tests for Disk Usage page - #11513

Merged
chihsuan merged 3 commits into
apache:masterfrom
sravani-revuri:HDDS-9795
Oct 16, 2026
Merged

chihsuan merged 3 commits into
apache:masterfrom
sravani-revuri:HDDS-9795

Conversation

@sravani-revuri

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

This task adds test for the Namespace/Disk usage page on recon UI

What is the link to the Apache JIRA

https://issues.apache.org/jira/browse/HDDS-9795

How was this patch tested?

Ran tests.

 ✓ src/__tests__/namespaceUsage/NamespaceUsage.test.tsx > Namespace Usage page (Disk Usage) > does not fetch root usage until the user confirms load 192ms
 ✓ src/__tests__/namespaceUsage/NamespaceUsage.test.tsx > Namespace Usage page (Disk Usage) > navigates to a child path via breadcrumb submenu 115ms
 ✓ src/__tests__/namespaceUsage/NamespaceUsage.test.tsx > Namespace Usage page (Disk Usage) > navigates to a key path via breadcrumb search 199ms
 ✓ src/__tests__/namespaceUsage/NamespaceUsage.test.tsx > Namespace Usage page (Disk Usage) > shows error when usage API returns PATH_NOT_FOUND 98ms
 ✓ src/__tests__/namespaceUsage/NamespaceUsage.test.tsx > Namespace Usage page (Disk Usage) > notifies when namespace summary is still initializing 41ms

 Test Files  1 passed (1)
      Tests  5 passed (5)
   Start at  19:18:43
   Duration  3.23s (transform 123ms, setup 65ms, import 1.96s, tests 647ms, environment 430ms)

@sravani-revuri

sravani-revuri commented Oct 13, 2026 •

Copy link
Copy Markdown
Contributor Author

@chihsuan , @smengcl would you like to review this?

@chihsuan

Copy link
Copy Markdown
Member

Sure, I'll take a look. 🙂

@sravani-revuri
sravani-revuri marked this pull request as ready for review October 14, 2026 05:55

@smengcl smengcl left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@chihsuan chihsuan left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for working on this! @sravani-revuri Looks good overall. I left a few inline comments, mostly about the mocks.

@chihsuan chihsuan left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the patch! @sravani-revuri LGTM +1

@chihsuan
chihsuan merged commit 3acb980 into apache:master Oct 16, 2026
50 of 51 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants