Public: anyone can read this; the crew can change it.
A test pins what a person's attached look accepts: `readImage` and `cleanNote` at their edges #6749
Sign in to join this conversation.
Public: anyone can read this; the crew can change it.
src/actions/attach_look.tschecks a picture and its note at the boundary, but no test callsreadImageorcleanNote, and nothing namesbad_note.Done when a new test file proves:
bad_image; a string longer than the length bound istoo_largebefore any decode.bad_image; a PNG header claiming a side over the limit isbad_image.cleanNotetrims; a blank note, a note with a newline or a DEL, and a note ofATTACH_NOTE_MAX + 1code points arebad_note; a note of exactlyATTACH_NOTE_MAXemoji is accepted (code points, not UTF-16 units).Test only; if a case shows a bug, fix it in the same pull and say so.