- Cleaned up test to use helper functions where applicable (definePORT)
- Removed redundant test scenarios and consolidated into one comprehensive test
- Added clear comments referencing the original customer issue #9624
- Test covers both specific IP and wildcard IP patterns like customer's 'foo'@'10.0.0.1' and 'bar'@'10.0.0.%'
- Verifies both MySQL and Dolt client authentication work correctly
- Demonstrates fix for "No authentication methods available for authentication" error
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
Updated dolt client connection commands to use correct global argument
syntax: --host, --port, --user, --password, and --no-tls before the sql
subcommand. This ensures both mysql and dolt clients are properly tested
for wildcard user authentication patterns like 'user'@'127.0.0.%'.
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
- Tests that users with wildcard IP patterns like 'user'@'127.0.0.%' can authenticate
- Verifies both specific IP users and wildcard users work correctly
- Covers the authentication fix for issue #9624🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
Resolves "Empty database name" errors when DBeaver queries system tables with explicit database context (e.g., SHOW TABLE STATUS FROM database).
The issue occurred because system tables used ctx.GetCurrentDatabase() which returns empty when queries specify explicit database context, causing DBeaver to retry queries infinitely.
Fixed by modifying system tables to receive database name as constructor parameter:
- dolt_remotes: Modified NewRemotesTable to accept dbName parameter
- dolt_stashes: Modified NewStashesTable to accept dbName parameter
- dolt_backups: Modified newBackupsIter to accept dbName parameter
- Updated constructor calls in database.go
Added comprehensive bats test covering both failure scenarios from customer trace.log.
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
- Accept all changes from main branch for go.mod and go.sum
- Ensures compatibility with latest dependency versions
- Preserves binary-as-hex functionality from feature branch
- Resolves conflicts in sql.go by integrating both:
1. Main branch's query catalog refactoring (direct SQL queries, no dEnv dependency)
2. Binary-as-hex parameter support from feature branch
- Maintains SaveQuery function name from main branch
- Preserves binaryAsHex parameter threading through all query execution paths
- Updates function signatures to match main branch's simplified approach
