Skip to content

Conversation

Florian-Bloechinger
Copy link

Description

This PR introduces comprehensive testing infrastructure improvements and documentation updates for the SurrealORM project.

Key Changes

  • Database Testing Scripts

    • Added cross-platform npm scripts for SurrealDB initialization
    • Implemented environment-specific scripts (Windows/Unix)
  • Documentation Updates

    • Improved README with detailed testing prerequisites
    • Added step-by-step testing instructions
    • Documented environment variable configurations
  • Connection Reliability

    • Improved error handling in connection methods
    • Added connection state validation
    • Enhanced error messages for better debugging
    • Implemented proper cleanup on connection failures

- introduce npm scripts for starting SurrealDB in different environments
- enhance README with detailed testing requirements and instructions
- improve error handling in connection methods for better reliability
…elationships

- introduce new test suite for advanced scenarios including error handling, complex queries, and relationship management
- implement tests for invalid credentials, network issues, and required field validation
- cover complex nested queries and handling of special characters in field names
- ensure performance testing for large datasets and edge cases
@ookamidev
Copy link
Contributor

I will keep this open for now, however I cannot merge this PR. There are a few things that needs to be changed first.

  • You are mixing NPM and PNPM into a Bun project
  • There is no need for a .bat or .sh for spinning up a test SurrealDB server, this can be done using a GitHub Workflow which I find to be a more elegant solution
  • There is no support for relations yet, this feature I'm currently working on

@ookamidev ookamidev added documentation Improvements or additions to documentation enhancement New feature or request invalid This doesn't seem right labels Jun 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request invalid This doesn't seem right

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants