glTF-Asset-Generator icon indicating copy to clipboard operation
glTF-Asset-Generator copied to clipboard

Published 20 hours ago • verified publisher icon KhronosGroup

Add some tests for uris

Open bghgary opened this issue 5 years ago • 2 comments

See https://github.com/KhronosGroup/glTF/issues/1449

Might consider adding other uri encoding edge cases as well for completeness.

bghgary avatar Sep 19 '18 16:09 bghgary

My suggestion:

  • [ ] Test a URI with an encoded %20 space
  • [ ] Test a URI with an un-encoded ' ' space
  • [ ] Test various other encoded characters that are allowable as filename characters on most filesystems (maybe $ might be a good test?)

No need to test encoded characters that are illegal in popular filesystems.

emackey avatar Sep 19 '18 16:09 emackey

Some more for images:

  • [ ] Test a URI with an embedded base64 texture
  • [ ] Test a URI with an embedded base64 buffer

kcoley avatar Jan 09 '19 00:01 kcoley