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

Use common test suite? #113

Closed
maxlinc opened this issue Aug 1, 2014 · 7 comments
Closed

Use common test suite? #113

maxlinc opened this issue Aug 1, 2014 · 7 comments
Labels

Comments

@maxlinc
Copy link

maxlinc commented Aug 1, 2014

There is a common test suite for implementors of JSON Schema validation libraries. It's being used by at least 15 validators now, but it doesn't look like it's setup for this project.

Are you interested in hooking up this suite, or did you already look at it and decide not to use it for some reason?

@apsoto
Copy link
Contributor

apsoto commented Aug 1, 2014

+1

@pd
Copy link
Contributor

pd commented Aug 1, 2014

I can't speak for @hoxworth, but I doubt there's any reason not to use it. I'd assume it's mostly just that this library significantly predates the common test suite, combined with not being aware of it / never having gotten around to implementing it.

👍 if you're willing to try your hand at it!

@maxlinc
Copy link
Author

maxlinc commented Aug 1, 2014

I'm willing to help hook it up and fix any issues. Definitely don't want to
go rogue and do it on my own, though!
On Aug 1, 2014 6:41 PM, "Kyle Hargraves" notifications@github.com wrote:

I can't speak for @hoxworth https://github.com/hoxworth, but I doubt
there's any reason not to use it. I'd assume it's mostly just that this
library significantly predates the common test suite, combined with not
being aware of it / never having gotten around to implementing it.

[image: 👍] if you're willing to try your hand at it!


Reply to this email directly or view it on GitHub
#113 (comment)
.

@hoxworth hoxworth added the Todo label Aug 3, 2014
@hoxworth
Copy link
Contributor

hoxworth commented Aug 3, 2014

As @pd said, there is no opposition to integrating it at all. I do remember some discussion of the test suite a while back on the mailing list, but I tend to just casually subscribe to the mailing list due to the signal/noise ratio. Would love to have this in - if you want to do it, feel free, otherwise it will sit on the backlog with all the other things...

@mpalmer
Copy link
Contributor

mpalmer commented Sep 18, 2014

I got bored and put together some code for making this happen. PR #122 created.

@iainbeeston
Copy link
Contributor

Should this be closed now?

@pd pd closed this as completed Oct 26, 2014
@pd
Copy link
Contributor

pd commented Oct 26, 2014

Thanks to everyone that made this happen!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

6 participants