Added TCK for python #25
Conversation
test/tck/configure_feature_files.py
Outdated
There was a problem hiding this comment.
Sorry about the random comment here, I just noted that this line will fail on Python3. urllib2 has been renamed urllib (and slightly restructured) there.
There was a problem hiding this comment.
You are probably right. This has currently been build only focusing on python 2.7 at the moment.
|
If I understand your description of this PR correctly, I would argue that the Java driver should not permit Node/Rel/Path objects as parameters. Doing so is not supported by the current remoting interface (HTTP) and the behaviour for this is AFAIK not well-defined by Cypher, despite being permitted by the embedded API. @jakewins: we had conversations about this earlier last year - has this changed? |
…or sending path/node/relationship as parameters since this is not supported.
test/tck/steps/bolt_type_steps.py
Outdated
There was a problem hiding this comment.
Any reason these arguments need to be TitleCased?
There was a problem hiding this comment.
Nope. They just have that format in the feature file
TCK tests are set to run with runtests.sh
Python-drivers currently do not accept sending Path/Node/Relationship while Java-drivers do. So those tests are currently failing.