Refactor test structure with AAA pattern and editor folds

[bernardladenthin]

Summary

This PR refactors the StreamBufferTest class to improve test readability and organization by implementing the Arrange-Act-Assert (AAA) pattern consistently across all tests and adding editor fold markers for logical test grouping.

Key Changes

• AAA Pattern Implementation: Added explicit // arrange, // act, and // assert comments to all test methods, making the test structure immediately clear and improving maintainability

• Editor Fold Organization: Grouped related tests using <editor-fold> tags with descriptive names:

  • roundtrip - Round-trip read/write tests
  • constructor - Constructor tests
  • getMaxBufferElements - Max buffer element getter tests
  • setMaxBufferElements - Max buffer element setter tests
  • isSafeWrite - Safe write flag tests
  • isClosed - Closed state tests
  • read_changeBufferFromOutside - External buffer modification tests
  • getBufferSize - Buffer size tests
  • read - Read operation tests
  • write - Write operation tests
  • available - Available bytes tests
  • blockDataAvailable - Blocking data availability tests
  • mark - Mark/reset tests
  • trim - Buffer trimming tests
  • close - Close operation tests
  • concurrentReadWrite - Concurrent read/write stress tests
  • getInputStream - Input stream getter tests
  • concurrent trim and write - Concurrent trim and write tests
  • signal/slot - Signal/slot mechanism tests

• Test Reorganization: Moved several tests to more logical locations within their respective fold sections for better organization

• Minor Formatting: Fixed trailing whitespace and improved code consistency throughout

Implementation Details

• No changes to test logic or assertions - all tests remain functionally identical
• The AAA pattern makes it easier to understand test intent at a glance
• Editor folds enable IDE-based collapsing of test groups for better navigation in large test files
• Tests are now organized by functionality rather than arbitrary order, improving discoverability

https://claude.ai/code/session_01HMQygoP7fJWayQi7wnLxRb

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.00%. Comparing base (757b5b7) to head (66b6e3f).
⚠️ Report is 2 commits behind head on master.
✅ All tests successful. No failed tests found.

Additional details and impacted files

@@            Coverage Diff            @@
##             master      #28   +/-   ##
=========================================
  Coverage     97.00%   97.00%           
  Complexity       47       47           
=========================================
  Files             1        1           
  Lines           167      167           
  Branches         25       25           
=========================================
  Hits            162      162           
  Partials          5        5           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.