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

Possible issue with Batch query interface (this time with a failing test) #92

Merged
merged 2 commits into from
Apr 19, 2013

Conversation

ceterumnet
Copy link
Contributor

The last pull request wasn't failing the test correctly. I guess I got mixed up before committing. Now this example fails properly to demonstrate the issue.

I'm not sure if this is an issue or not. Ultimately, it seems that one bad statement in a batch query fails the whole batch - without an error.

@ceterumnet
Copy link
Contributor Author

I added additional code to connection.rb - this code fixes the issue with the batch query, but it introduces a failure on another example. I believe that the failure is correct, and exposes another issue.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 0eee207 on ceterumnet:master into aa6c1ae on maxdemarzi:master.

maxdemarzi added a commit that referenced this pull request Apr 19, 2013
Possible issue with Batch query interface (this time with a failing test)
@maxdemarzi maxdemarzi merged commit c6dad85 into maxdemarzi:master Apr 19, 2013
willkessler pushed a commit to willkessler/neography that referenced this pull request Apr 21, 2014
Updating Gemfiles per 2.0.0-p353 requirements and final build of AMI
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