Add test coverage for move/swap operations with non-trivially copyable types

[Copilot]

The ring buffer implementation has separate code paths for trivially copyable types (using memcpy/std::swap) and non-trivially copyable types (using move construction), but existing tests only exercised the trivial path with int.

Changes

• Added tests for move construction, move assignment, and swap operations using std::string and std::vector<int>
• Tests verify the std::false_type specializations of move_impl and swap_impl work correctly

TEST(RingBufferTest, MoveConstructionWithNonTriviallyCopyableType) {
    ring_buffer<std::string, 4> b1;
    b1.push_back("hello");
    b1.push_back("world");
    // ... verify move semantics work correctly
}

• Updated .gitignore to exclude CodeQL build artifacts

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

---

Important

Add tests for move and swap operations with non-trivially copyable types in test_main.cpp and update .gitignore.

• Tests:
  • Add tests for move construction, move assignment, and swap operations with std::string and std::vector<int> in test_main.cpp.
  • Verify std::false_type specializations of move_impl and swap_impl.
• Misc:
  • Update .gitignore to exclude CodeQL build artifacts.

^{This description was created by for cf5f003. You can customize this summary. It will automatically update as commits are pushed.}

[coderabbitai]

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

---

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

[ellipsis-dev]

Important

Looks good to me! 👍

Reviewed everything up to cf5f003 in 33 seconds. Click for details.

• Reviewed 89 lines of code in 2 files
• Skipped 1 files when reviewing.
• Skipped posting 5 draft comments. View those below.
• Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.

1. _codeql_detected_source_root:1

• Draft comment:
  Add a trailing newline for consistency.
• Reason this comment was not posted:
  Confidence changes required: 0% <= threshold 50% None

2. test_main.cpp:339

• Draft comment:
  Comprehensive tests for non-trivially copyable types (move/swap) are added—good coverage.
• Reason this comment was not posted:
  Confidence changes required: 0% <= threshold 50% None

3. test_main.cpp:350

• Draft comment:
  Ensure move constructor leaves the source buffer in an empty state as expected by the test.
• Reason this comment was not posted:
  Confidence changes required: 0% <= threshold 50% None

4. test_main.cpp:365

• Draft comment:
  Verify that move assignment resets the source buffer; test correctly expects an empty source.
• Reason this comment was not posted:
  Confidence changes required: 0% <= threshold 50% None

5. test_main.cpp:399

• Draft comment:
  Nice use of ADL with 'using std::swap' for non-member swap in non-trivially copyable type test.
• Reason this comment was not posted:
  Confidence changes required: 0% <= threshold 50% None

Workflow ID: wflow_3geD5LMhX96z5LTd

^{You can customize by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.}

[Copilot]

Pull request overview

This PR adds comprehensive test coverage for move and swap operations with non-trivially copyable types to ensure the ring buffer implementation correctly handles types that require move construction rather than memcpy operations.

Changes:

• Added four new test cases using std::string and std::vector<int> to verify move construction, move assignment, member swap, and non-member swap operations work correctly with non-trivially copyable types
• Updated .gitignore to exclude CodeQL build artifacts

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File	Description
test_main.cpp	Added four test cases for move/swap operations with std::string and std::vector<int> to verify non-trivial type handling
.gitignore	Added _codeql_build_dir to ignore CodeQL build artifacts
_codeql_detected_source_root	CodeQL configuration file indicating source root directory

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.