JSON-Patch icon indicating copy to clipboard operation
JSON-Patch copied to clipboard

Specs/tests link in readme broken

Open vollmerr opened this issue 6 years ago • 5 comments

The 'Run in browser' under Specs/tests in the readme pointing to http://starcounter-jack.github.io/JSON-Patch/test/ gives a blank screen with a bunch of 404's

vollmerr avatar Aug 06 '19 22:08 vollmerr

Thanks for reporting @vollmerr. The issue was probably introduced by me, when I switched test runner to use Jasmine & co. from node_modules instead of local hard-copy.

Locally you can run it by:

npm install && npm run serve

Then visit http://localhost:5000/test/index

@warpech Do you think we should:

  1. Use Jasmine from CDN for HTML spec runner https://github.com/Starcounter-Jack/JSON-Patch/blob/master/test/index.html (will stop working offline, locally)
  2. Push Jasmine back to the repo
  3. Remove this link from README

tomalec avatar Aug 07 '19 10:08 tomalec

  1. Wait for https://github.com/Starcounter-Jack/JSON-Patch/pull/232, because the HTML spec runner works OK there for any browser that supports ES modules.

warpech avatar Aug 08 '19 08:08 warpech

How would that help? it still uses ../node_modules/jasmine-core... https://github.com/Starcounter-Jack/JSON-Patch/blob/use-ES6-modules/test/index.html#L10

tomalec avatar Aug 08 '19 12:08 tomalec

How would that help? it still uses ../node_modules/jasmine-core... /test/index.html@use-ES6-modules#L10

Sorry, my bad. I tested locally with npm run serve and it worked well because I have node_modules locally.

Then I vote for 3. Maybe we can replace the link to spec runner with a link to working example on JSFiddle?

warpech avatar Aug 08 '19 13:08 warpech

FYI, I see this error in the console proper to all the 404's

Error with Permissions-Policy header: Origin trial controlled feature not enabled: 'interest-cohort'.

Perhaps related:

https://developer.chrome.com/docs/privacy-sandbox/permissions-policy/

https://www.studytonight.com/post/solved-error-with-permissionspolicy-header-parse-of-permission-policy-failed

grempe avatar Feb 22 '23 20:02 grempe