Add test for remove_item triggering needs_rebuild

[cardmagic]

Summary

Add the final missing test for issue #51 - verifying that LSI#remove_item triggers needs_rebuild?.

Context

Most tests for remove_item and items were added in PR #58. This PR adds the one remaining test:

• remove_item basic functionality (added in Add edge case tests for text handling #58)
• remove_item on non-existent item (added in Add edge case tests for text handling #58)
• remove_item triggers needs_rebuild? (this PR)
• items returns list of indexed items (added in Add edge case tests for text handling #58)

Test Added

def test_remove_item_triggers_needs_rebuild
  lsi = Classifier::LSI.new auto_rebuild: false
  lsi.add_item @str1, 'Dog'
  lsi.add_item @str2, 'Dog'
  lsi.add_item @str3, 'Cat'
  lsi.build_index

  refute lsi.needs_rebuild?, 'Index should be current after build'

  lsi.remove_item @str1

  assert lsi.needs_rebuild?, 'Index should need rebuild after removing item'
end

Fixes #51

[Copilot]

Pull request overview

This PR adds a test to verify that LSI#remove_item triggers the needs_rebuild? flag, completing the test coverage for issue #51.

• Adds test_remove_item_triggers_needs_rebuild to validate that removing an item sets the rebuild flag
• Complements existing tests for remove_item functionality added in PR #58
• Uses auto_rebuild: false to explicitly test the needs_rebuild? state changes

---

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