Fix: Incorrect insertion location in fix-it for )

[vinu-vanjari]

Resolves #2431

---

Changes Summary:

1. Added shouldBeInsertedBeforePreviousTokenTrivia

Extracted this from existing code from DiagnosticExtensions.swift
Inserted this just after existing property shouldBeInsertedAfterNextTokenTrivia
This extraction is needed as we want to reuse this while deciding correct insertion location of fix-it

2. Correct insertion location for fix-it in MissingNodesError.swift

This should correct ), :, , syntax positions which should be inserted before previous token trivia. i.e. should not have whitespace before them. (check unit tests corrections for examples)

3. Correction of Unit Tests

Corrected the existing unit tests where missing ')' token error was expected.
Notice that we only change location markers and not other things in the unit tests.
None of the fixedSource in the examples in unit tests are changed

4. Added couple of Unit Tests

• ParserDiagnosticsFormatterIntegrationTests.swift->testRighParenLocation()
• TypeExprTests.swift->testTypeExpr31()

[ahoppen]

Very nice and simple fix 😍

[ahoppen]

@swift-ci Please test