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

Self-provided test suite, skip two tests, and assert @version correctly when specified #179

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

aredridel
Copy link

@aredridel aredridel commented May 6, 2017

This should get CI working.

  • I published the json-ld.org test suite as an npm module. I'm happy to pass on the keys to that npm package for updates if that is a concern; Also happy to maintain that going forward, I don't anticipate a lot of work there, but won't be making it a priority for my work unless someone wants to poke me about it
  • I added code to conform to tp003 and tp006 tests, asserting that version is 1.1 if it is specified at all.
  • I skipped new tp002 and tp005 tests, as we don't currently handle processing mode vs @version checks.

@davidlehn
Copy link
Member

The test skipping part looks ok.

The @version check is maybe ok. @dlongley: do we want to add in small parts of 1.1 support like this to 0.4.x? Or should that just be in 0.5.x or is there some other plan? Do we want to keep 0.4.x as just a json-ld-1.0 lib? I do think that the test suite should be marked such that 1.0 libs would just skip tests that even needed this sort of patch. Though we'll certainly need it for 1.1 support.

I'm not sure we want to merge in that test suite package change at the moment. I kind of like the idea but I can see keeping that up-to-date will be an issue. It's nice to just point at the latest test suite checkout and get new tests. It's also a problem when the 1.1 tests get mixed in but we'll slowly deal with that. There's also the issue that we, in theory, try to keep js, php, and python libs synced and they are all do a similar thing looking at the json-ld.org repo. It's kind of nice to test them all with the same suite at the same time and it might be odd to use npm packages for python and php testing. ;-) I'd say wait on that for a bit until we've considered it more.

@davidlehn
Copy link
Member

Thinking a bit more... the current way 1.1 tests are marked and skipped is problematic. Opened a test suite issue on this. Perhaps we can better skip all 1.1 related tests w/o listing specific tests.

@davidlehn
Copy link
Member

The 1.1 skipping is improved now since the test suite marks 1.1 tests with a specVersion property.

@davidlehn
Copy link
Member

@version support added in 0.5.x here:
f6a91c0

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.

None yet

2 participants