[patch][ccla] JSON parsing tests should use strict parsing

[andybalaam]

Some of the tests in dojo/tests/json.js don't pass the "strict" flag to the JSON.parse function.

This flag has no effect in environments which provide a built-in JSON object.

In environments that fall back to the Dojo JSON parsing, this flag enables the safety check regular expression.

To ensure there are no bugs in this regular expression, we should pass strict=true to JSON.parse in all tests.

Andy Balaam (IBM, CCLA)

[andybalaam]

[patch][ccla] always-use-strict-mode-in-json-parsing-tests Andy Balaam (IBM, CCLA)

[bill]

The TestFramework category is for bugs in DOH itself, not for tests written using DOH.

[andybalaam]

OK, sorry about that bill, thanks.

[dylan]

Kris, please review this one as well for 1.9.

[Kris Zyp]

We do have strict tests to that exercise the regular expressions (in older IE). If we made all the tests strict than we wouldn't be verifying the non-script code path would we?

[trac-o-bot]

Because we get so many tickets, we often need to return them to the initial reporter for more information. If that person does not reply within 14 days, the ticket will automatically be closed, and that has happened in this case. If you still are interested in pursuing this issue, feel free to add a comment with the requested information and we will be happy to reopen the ticket if it is still valid. Thanks!