Fix 124 identifiers tests #162
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Added ability to just generate a sha of the json. Was option, but not coded.
Added ability to just generate a sha of the json. Was option, but not coded.
valentinedwv
added a commit
that referenced
this pull request
Mar 3, 2023
* Cleaned up tests. Files now used. Added ability to just generate a sha of the json. Was option, but not coded.
valentinedwv
added a commit
that referenced
this pull request
Mar 6, 2023
* run configurations * adding a uuid command to get uuid for a file sent to stdin. updated cobra and viper package versions * Example JSON Paths and arrays of paths * glcon tool id --idtype indentifiersha --jsonld some file * test for #129. * Implemented changes to prevent context mangling * * Get things working and DRY code up a bit * Extract out a constant and a json processing function * Add test for json finding method * Test for the api config helper * Try to make this work for DataStream as well * Identifiers fix for @graph * fix wrap in quotes 'script[type="application/ld+json"]' * add an ideal example resource * IdentiferPath is a comma separated set of strings, or one string. * If you are trying not to recorder an array... and copy it, don't use the original later Clean up LdProc Normalize Tests. Empty triples will be empty triples * #150 #108 initialize gleaner and nabu configs idependently. * #145 * added test for a couple of problem children. * methods to allow dumping data before upload and glcon tools jsonld: jsonld > corrected jsonld rdf: jsonld > rdf * add a couple resources for cli testing * Headless rework to make more testable * Fix Minio connection check error, like was done on nabu * Implement headlessWait in the headless Javascript. Fixes #152 * Implement headlessWait in the headless Javascript. Fixes #152 add a test that is expected to fail, with 1 JSONld returned. * #155 enable --prefix flag for nabu * Using c.Page.LoadEventFired to wait for content. Works. leaving in the headlessWait incase another issue that requires its. * linux reading of config not working. Updating modules * Use the config class to read the gleanerConfig. * Use the config class to read the gleanerConfig. * Cleaned up tests. Files now used. (#161) Added ability to just generate a sha of the json. Was option, but not coded. * Fix 124 identifiers tests (#162) * Cleaned up tests. Files now used. Added ability to just generate a sha of the json. Was option, but not coded. * put in approvals into jsonutils * use log.info not println so this does not show up in console output. * test for #129. * Implemented changes to prevent context mangling * Example JSON Paths and arrays of paths * clean up some logging CLI. * glcon tool id --idtype indentifiersha --jsonld some file * If it's an array, we standardized. Tests to be sure arrays are not mangled by string and that strings are not mangled by array. * IdentiferPath is a comma separated set of strings, or one string. * methods to allow dumping data before upload glcon tools jsonld: jsonld > corrected jsonld rdf: jsonld > rdf Added ability to just generate a sha of the json. Was option, but not coded. * Fix 124 identifiers tests (#162) * put in approvals into jsonutils * add approval test .approved * Sort the jsonpath results because UUID generation depends on order of results. * One more fix for tool id. If no config is passed, just use a generic one. We need is the contexts, for the triples if it falls through to a normalizedsha makefile. copy binaries up to that assets are in correct location relative to binaries --------- Co-authored-by: melinda <melindaminch@oceannetworks.ca> Co-authored-by: Douglas Fils <drfils@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.