Skip to content

#39#41

Merged
guycipher merged 2 commits intotidesdb:masterfrom
mukesh-2096:consolidate-tests
Feb 23, 2026
Merged

#39#41
guycipher merged 2 commits intotidesdb:masterfrom
mukesh-2096:consolidate-tests

Conversation

@mukesh-2096
Copy link
Copy Markdown
Contributor

@mukesh-2096 mukesh-2096 commented Feb 23, 2026

Summary

Moved max_memory_usage tests from the standalone test_max_memory.lua file into tests/test_tidesdb.lua to maintain consistency with the project's test structure.

Changes

  • Added test_max_memory_usage() function to tests/test_tidesdb.lua
  • Removed standalone test_max_memory.lua file
  • Followed existing code style and comment patterns

Testing

The test covers:

  • Default config includes max_memory_usage field
  • Opening database with default max_memory_usage (0 = unlimited)
  • Opening database with custom max_memory_usage value
  • Creating database with TidesDB.new() constructor with max_memory_usage

All tests use the existing test utilities (assert_true, cleanup_db, etc.) and follow the established pattern.

@guycipher guycipher changed the title Consolidate tests #39 Feb 23, 2026
@guycipher
Copy link
Copy Markdown
Member

@mukesh-2096 accept the DCO kindly! :D

@guycipher
Copy link
Copy Markdown
Member

@guycipher guycipher merged commit a991d9f into tidesdb:master Feb 23, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants