Skip to content

update documentation #152

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 10 commits into from
Apr 27, 2017
Merged

update documentation #152

merged 10 commits into from
Apr 27, 2017

Conversation

srbaker
Copy link
Contributor

@srbaker srbaker commented Apr 12, 2017

This updates the documentation to match, as closely as possible, what's available in the Java docs. There are some comments inline with some issues that I need assistance with.


# def test_config_trust_example(self):
# from config_trust_example import ConfigTrustExample
# pass
Copy link
Contributor Author

Choose a reason for hiding this comment

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

These commented out ones, I'm not entirely sure they have a mapping to the Python driver.

# def test_custom_auth_example(self):
# from custom_auth_example import CustomAuthExample
# pass

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Also not sure if there is a python equivalent of this.


# FIXME: also check the error output like in the Java example
self.assertEqual(employee_number, -1)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This one is disabled because the behaviour is different on Python than in Java, and I'm not sure what to do. In Java, the equivalent test passes, but when this test is enabled, close() throws an exception because there was an error.

# def test_service_unavailable_example(self):
# from service_unavailable_example import ServiceUnavailableExample
# pass

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I am not sure how to do the equivalent of neo4j.stop() that we do in the Java version.

@jjaderberg
Copy link
Contributor

I pushed this change to a new branch to make it easier to pull it into the docs build as WIP also while the PR is pending: https://github.com/neo4j/neo4j-python-driver/tree/1.2-examples-deletewhen152ismerged Feel free to delete that once this PR is merged, and please ping me so I can point docs build back to 1.2.

}
try:
with self._driver.session() as session:
session.write_transaction(lambda tx: tx.run("CREATE (a:Item)"))
Copy link
Contributor Author

Choose a reason for hiding this comment

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

When running, the test hangs here indefinitely. :/

@zhenlineo zhenlineo merged commit 6574439 into neo4j:1.2 Apr 27, 2017
zhenlineo pushed a commit to zhenlineo/neo4j-python-driver that referenced this pull request Apr 27, 2017
…ation"

This reverts commit 6574439, reversing
changes made to 370d60b.
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.

3 participants