Enhancement: optimize tests of parameter type parser according to test coverage report

[JesseCodeBones]

Hi Guys,
I tried to enhance the test coverage, there is some missing part according to coverage report.
I might commit several commits or PRs under this issue, please let me know if it is not suitable.
Thank you.
Regards

[JesseCodeBones]

Phase: 1
File: src/parser.ts
Functions: [parseTypeParameter, parseParameters]
PR: #2297

[JesseCodeBones]

I found it could be a issue for test:

function extendsfunctiontype(this: ()=>void):void {}

the result is we get 1003 error as expected but we still can see it is compiled as below:
function extendsfunctiontype(): void {}
(that means it is included into AST now)

I am not sure if it is reasonable, because extends from anonymous function type is meaningless.
can you help to look at it? thank you.

[dcodeIO]

The parser/compiler try to recover from errors that will ultimately cause compilation to fail, so more than one error can be diagnosed. Here, the recognizable parts of the AST are still produced during parsing, but their compilation is not attempted.

[JesseCodeBones]

@dcodeIO
Hi DcodeIO,
I tried to fix the test coverage to 90.37%.

and I find a small issue here.
at parser.js line 2097, parseClassMember function.
the error message should be private modifier cannot be used here instead of protected modifier cannot be used here
Can you advice is we can fix it in this PR or we need another issue and another PR? thank you.

        } else if (flags & CommonFlags.PRIVATE) {
          this.error(
            DiagnosticCode._0_modifier_cannot_be_used_here,
            tn.range(accessStart, accessEnd), "protected"
          ); // recoverable
        }

[dcodeIO]

I guess this can be one PR, along the lines of "fix: X and add additional tests". Does this sound OK?

[JesseCodeBones]

@dcodeIO
OK, I will prepare it, thank you.

[github-actions]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in one week if no further activity occurs. Thank you for your contributions!

[JesseCodeBones]

@dcodeIO
Hi dcodeIO,
I noticed this issue has been closed, do we have plan to enhancement this part? Thank you.
Regards