Add boundary view const coverage

[psychocoderHPC]

Split from local commit fb3ef0d3d04c12b9aa2262f5850b6b04a8447c87.

Base branch: pr-add-boundaryiter-host-coverage.

Summary by CodeRabbit

• Tests
  • Added compile-time validation checks for const-correctness behavior in buffer subview operations across mutable, inner-const, and outer-const buffer configurations.

[coderabbitai]

Warning

Rate limit exceeded

@psychocoderHPC has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 8 minutes and 29 seconds before requesting another review.

Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 8 minutes and 29 seconds.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 65cbf538-b64d-4ce0-bb04-ca4b8a8877a8

📥 Commits

Reviewing files that changed from the base of the PR and between d4a9884 and 3a72e41.

📒 Files selected for processing (1)

• test/unit/mem/constCorrectness.cpp

📝 Walkthrough

Walkthrough

Added compile-time const-correctness checks to a unit test file validating that View::getSubView(boundaryDir) correctly preserves or enforces const-correctness for SharedBuffer objects across mutable, inner-const, and outer-const configurations.

Changes

Cohort / File(s)	Summary
Const-correctness test assertions test/unit/mem/constCorrectness.cpp	Added boundary direction construction and three new compile-time validation checks ensuring getSubView() boundary operations maintain proper const-correctness semantics for different SharedBuffer const qualifications.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 A boundary tested, a const declared true,
With three careful checks on what access can do,
Mutable flows free, while const stands its ground,
In the realm of the views, correctness is found! 🌟

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'Add boundary view const coverage' directly describes the main change: adding const-correctness tests for boundary view functionality in SharedBuffer.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch pr-add-boundary-view-const-coverage

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[psychocoderHPC]

@coderabbitai review

[coderabbitai]

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[psychocoderHPC]

@coderabbitai review

[coderabbitai]

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.