Skip to content

test(fuse): update INIT_EXT flag name expectation - #1450

Merged
lzjqsdd merged 1 commit into
CurvineIO:mainfrom
ligeng8848:test/fuse-init-ext-flag-name
Jul 31, 2026
Merged

test(fuse): update INIT_EXT flag name expectation#1450
lzjqsdd merged 1 commit into
CurvineIO:mainfrom
ligeng8848:test/fuse-init-ext-flag-name

Conversation

@ligeng8848

Copy link
Copy Markdown
Contributor

Summary

Update the FUSE INIT capability-name test to reflect that bit 30 is now registered as FUSE_INIT_EXT, while preserving coverage for the hexadecimal fallback of unmapped bits.

Issue Describe / Design

No related issue.

Root cause:
The test still treated bit 30 (0x40000000) as unknown, but the production capability-name map now registers that bit as INIT_EXT.

Reproduction:
cargo test -p curvine-fuse fuse_init_flag_names_maps_known_and_unknown --lib

Fix approach:
Assert that FUSE_INIT_EXT renders as INIT_EXT, and use bit 31 to verify the hexadecimal fallback for an unmapped bit.

Changes

Module / File Change Impact on existing behavior
curvine-fuse/src/fs/curvine_file_system.rs Update the INIT flag-name test data and expectations None; test-only change

Test verified

Test case Result Notes
cargo test -p curvine-fuse fuse_init_flag_names_maps_known_and_unknown --lib PASS 1 passed
cargo test -p curvine-fuse init_ --lib PASS 8 passed
cargo fmt --package curvine-fuse -- --check PASS
cargo test -p curvine-fuse --lib PARTIAL 333 passed; 5 unrelated permission/special-file tests failed independently
make format BLOCKED Full workspace clippy is blocked on local macOS by existing recursion-limit, RocksDB C++ header, and unrelated warning failures; the target file remains formatted

Dependencies

  • Related PRs: None
  • Related issues: None
  • Blocks / blocked by: None

@ligeng8848
ligeng8848 marked this pull request as ready for review July 31, 2026 10:00

@szbr9486 szbr9486 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.

Reviewed the test update against the current FUSE INIT capability map and the unknown-bit fallback. I found no actionable code issues in this change.

@lzjqsdd
lzjqsdd merged commit 4e6eb31 into CurvineIO:main Jul 31, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants