Add some tests for invalid GUIDs in JSON #2958

Closed
opened 2026-01-30 23:09:44 +00:00 by claunia · 2 comments
Owner

Originally created by @zadjii-msft on GitHub (Jul 26, 2019).

Should we add tests for the following?

* [ ]  GUID in different GUID format

* [ ]  completely invalid GUID (i.e.: guid = )

* [ ]  GUID with letters in caps

Or would the first two just be examples of invalid settings values anyways so there's no point?

From @carlos-zamora in #2117

Originally created by @zadjii-msft on GitHub (Jul 26, 2019). > > > Should we add tests for the following? > > * [ ] GUID in different GUID format > > * [ ] completely invalid GUID (i.e.: guid = ) > > * [ ] GUID with letters in caps > > > Or would the first two just be examples of invalid settings values anyways so there's no point? From @carlos-zamora in #2117
Author
Owner

@marcelwgn commented on GitHub (Apr 7, 2024):

I would like to take my shot at this

@marcelwgn commented on GitHub (Apr 7, 2024): I would like to take my shot at this
Author
Owner

@DHowett commented on GitHub (Apr 8, 2024):

Go for it!

@DHowett commented on GitHub (Apr 8, 2024): Go for it!
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#2958