Giters

KhronosGroup / glTF-Asset-Generator

Tool for generating various glTF assets for importer validation

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Add some tests for uris

bghgary opened this issue · comments

commented

See KhronosGroup/glTF#1449

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

commented

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.

commented

Some more for images:

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