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

Improved e2e tests #1724

Merged
merged 4 commits into from
Sep 24, 2024
Merged

Improved e2e tests #1724

merged 4 commits into from
Sep 24, 2024

Conversation

mrhardikjoshi
Copy link
Member

This pull introduces/changes:

  • enhanced e2e tests to be compatible with jruby

@@ -96,7 +96,7 @@ def handle_incomplete_states!
return unless incomplete_states.any?
incomplete_versions = incomplete_states.map(&:migration_id)
fail MigrationError, <<-MSG
There are migrations struck in an incomplete states, that could not be fixed automatically:
There are migrations stuck in an incomplete states, that could not be fixed automatically:
Copy link
Member Author

Choose a reason for hiding this comment

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

typo?

@@ -19,7 +19,6 @@ jobs:
ruby: [ jruby, ruby-3.2.5 ]
neo4j: [ 5.22.0 ]
active_model: [ 7.1.3 ]
# jruby will fail till bug https://github.com/jruby/jruby-openssl/issues/290 is fixed
Copy link
Member Author

Choose a reason for hiding this comment

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

This is resolved now.

@klobuczek klobuczek merged commit 681afe8 into 12 Sep 24, 2024
4 of 10 checks passed
@klobuczek klobuczek deleted the e2e_test_improvement branch September 24, 2024 12:20
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