Skip to content

Conversation

@syeleti-msft
Copy link
Member

Type of Change

  • Bug fix
  • New feature
  • Code quality improvement
  • Other (describe): Add tests

Description

  • Feature / Bug Fix: (Brief description of the feature or issue being addressed)

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a comprehensive test suite to verify the data integrity of block cache functionality in Blobfuse2. The test suite includes various file operations and edge cases to ensure the block cache maintains data consistency across different scenarios.

  • Tests basic file operations (create, read, write, truncate) with integrity verification
  • Tests advanced scenarios like sparse writing, stripe operations, concurrent operations, and file descriptor duplication
  • Tests edge cases including O_TRUNC operations, unlink-on-open behavior, and parallel flush calls
Comments suppressed due to low confidence (2)

test/scenarios/blk_cache_integrity_test.go:1003

  • Function name has a typo: 'Parlrel' should be 'Parallel' to match the naming pattern of TestParallelFlushCalls.
func TestParllelFlushCallsByDuping(t *testing.T) {

test/scenarios/blk_cache_integrity_test.go:949

  • Function name has a typo: 'Parlrel' should be 'Parallel'.
func TestParllelFlushCalls(t *testing.T) {

@vibhansa-msft vibhansa-msft modified the milestones: v2-2.6.0, v2-2.5.1 Jul 29, 2025
@syeleti-msft syeleti-msft merged commit d47700b into main Aug 2, 2025
11 checks passed
@syeleti-msft syeleti-msft deleted the syeleti/bc-test-suite branch August 2, 2025 11:22
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