Change Details

``` Preparation to deprecate ParseNonRFCJSONValue() but keep test coverage on the underlying UniValue::read() unaffected. The test coverage on AmountFromValue is no longer included, since that is already tested in the rpc_parse_monetary_values test case. Fuzzing coverage on ParseNonRFCJSONValue() was duplicated between string.cpp and parse_univalue.cpp, only the one in parse_univalue.cpp is kept. ``` Backport of [[https://github.com/bitcoin/bitcoin/pull/27256 | core#27256]]. Depends on D16293.

``` ParseNonRFCJSONValue() is no longer necessary and we can use UniValue::read() directly [...] To avoid code duplication, we keep the function to throw on invalid input data but rename it to Parse() and remove it from the header. The existing test coverage we had on ParseNonRFCJSONValue() is moved over to UniValue::read(). ``` Backport of core#27256.

``` Preparation to deprecate ParseNonRFCJSONValue() but keep test coverage on the underlying UniValue::read() unaffected. The test coverage on AmountFromValue is no longer included, since that is already tested in the rpc_parse_monetary_values test case.arseNonRFCJSONValue() is no longer necessary and we can use UniValue::read() directly Fuzzing coverage on ParseNonRFCJSONValue() was duplicated between string.cpp and parse_univalue.cpp, only the one in parse_univalue.cpp is kept. ```[...] To avoid code duplication, we keep the function to throw on invalid input data but rename it to Parse() and remove it from the header. Backport of [[https://github.com/bitcoin/bitcoin/pull/27256 | core#27256]].The existing test coverage we had on ParseNonRFCJSONValue() is moved over to UniValue::read(). ``` Depends on D16293Backport of core#27256.