apa512 / clj-rethinkdb

Eclipse Public License 1.0
205 stars 42 forks source link

Add missing tests #117

Open danielcompton opened 8 years ago

danielcompton commented 8 years ago

There are a bunch of query terms that don't have tests. If you're looking to help contribute to the driver, adding tests for some of these would be a great help.

danielytics commented 8 years ago

@anthonygalea Thanks for your awesome work on this! 👍

bhurlow commented 8 years ago

woooo!

lukaszkorecki commented 8 years ago

@anthonygalea so much ❤️ for this

anthonygalea commented 8 years ago

my pleasure

ds2643 commented 7 years ago

Hi all, I'm interested in helping. Is the above checklist still up to date?

danielcompton commented 7 years ago

It should be close, though there might be some for joins.

DonyorM commented 7 years ago

I'm going to try and work on this a little, anything I need to know about?

danielcompton commented 7 years ago

@DonyorM should be pretty close to up to date, leave a comment here if there's anything missed though.

DonyorM commented 7 years ago

@danielcompton Ok thank you. I submitted a pull request adding some of the joins tests. I believe someone also added a test for the eq-joins tests.