RCORE-2131: Don't throw immediately when convertion of string to number fails

[jedelbo]

The parser will throw later anyway if the comparison cannot be done.

What, How & Why?

Fixes #7714

☑️ ToDos

• 📝 Changelog update
• 🚦 Tests (or not relevant)
• C-API, if public C++ API changed
• bindgen/spec.yml, if public C++ API changed

[ironage]

LGTM once CI is passing.
See test: C API - queries::query::interpolate all types::type mismatch

[jedelbo]

@ironage damn this is a rabbit hole. We are not very consistent in when we throw InvalidQueryArgError and when it is just InvalidQueryError.

[coveralls-official]

Pull Request Test Coverage Report for Build jorgen.edelbo_263

Details

• 16 of 22 (72.73%) changed or added relevant lines in 2 files are covered.
• 51 unchanged lines in 19 files lost coverage.
• Overall coverage increased (+0.02%) to 90.808%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
src/realm/parser/driver.cpp	14	20	70.0%

Files with Coverage Reduction	New Missed Lines	%
src/realm/mixed.cpp	1	86.46%
src/realm/query_expression.hpp	1	93.81%
src/realm/sync/noinst/server/server.cpp	1	73.96%
src/realm/sync/noinst/server/server_history.cpp	1	63.7%
src/realm/util/serializer.cpp	1	90.43%
src/realm/uuid.cpp	1	98.48%
test/fuzz_tester.hpp	1	57.73%
test/test_dictionary.cpp	1	99.83%
src/realm/array_blobs_big.cpp	2	98.58%
src/realm/cluster.cpp	2	75.6%

Totals
Change from base Build 2326:	0.02%
Covered Lines:	214763
Relevant Lines:	236503

---

💛 - Coveralls