member-access rule should trigger with readonly parameter property

[cyrilgandon]

Bug Report

• TSLint version: 5.7.0
• TypeScript version: 2.5.3
• Running TSLint via: VSCode

TypeScript code being linted

class Point {
    constructor(
        readonly x: number,
        readonly y: number
    ) { }
}

with tslint.json configuration:

{
  "rules": {
    "member-access": [true, "check-accessor"]
  }
}

Actual behavior

No Error

Expected behavior

It should trigger the member-access rule:

The class property 'x' must be marked either 'private', 'public', or 'protected' (member-access)

The same code, written differently, triggers the error:

class Point {
    readonly x: number; // <-- Error
    readonly y: number; // <-- Error
    constructor(x: number, y: number) {
        this.x = x;
        this.y = y;
    }
}

Maybe there is a need for a new configuration of this rule, like check-parameter-property, because I'm not sure that it make sense to have it under check-accessor.

[ajafff]

Fixed by #3325