Skip to content

Store driver features as enum entries #290

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

Merged
merged 4 commits into from
Nov 3, 2021

Conversation

robsdedude
Copy link
Contributor

This fixes an error in
tests.neo4j.test_summary.TestSummary.test_protocol_version_information where
computation of the expected negotiated bolt version relied on having enum
entires available.

This fixes an error in
`tests.neo4j.test_summary.TestSummary.test_protocol_version_information` where
computation of the expected negotiated bolt version relied on having enum
entires available.
Copy link
Contributor

@bigmontz bigmontz left a comment

Choose a reason for hiding this comment

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

We should fail when some invalid feature name is supplied.

@robsdedude robsdedude requested a review from bigmontz November 3, 2021 11:06
Copy link
Contributor

@bigmontz bigmontz left a comment

Choose a reason for hiding this comment

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

📚

@bigmontz
Copy link
Contributor

bigmontz commented Nov 3, 2021

Fixing JS driver feature list: neo4j/neo4j-javascript-driver#805

@robsdedude robsdedude merged commit efffd1a into 4.4 Nov 3, 2021
@bigmontz bigmontz deleted the turn-driver-features-into-enum-entries branch November 3, 2021 17:11
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