Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: Fix failing migration test for Windows #2347

Merged
merged 1 commit into from
Jan 7, 2025
Merged

Conversation

guredd
Copy link
Collaborator

@guredd guredd commented Jan 6, 2025

Description

Summary of the change: Fixed file separator in the test testMigrationScriptDirectoryAndContent to be system independent


Type of Change

Please mark the relevant options with an "X":

  • Bug fix
  • New feature
  • Documentation update

Updates/remove existing public API methods:

  • Is breaking change

Affected databases:

  • MariaDB
  • Mysql5
  • Mysql8
  • Oracle
  • Postgres
  • SqlServer
  • H2
  • SQLite

Checklist

  • Unit tests are in place
  • The build is green (including the Detekt check)
  • All public methods affected by my PR has up to date API docs
  • Documentation for my change is up to date

Related Issues

@bog-walk bog-walk self-assigned this Jan 7, 2025
@bog-walk bog-walk changed the title fix tests for Windows test: Fix failing migration test for Windows Jan 7, 2025
Copy link
Member

@bog-walk bog-walk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks very much for the fix @guredd 👍

@bog-walk bog-walk merged commit 4f594d4 into main Jan 7, 2025
5 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