fix(suggestion): Remove 80001 from cjs/cts files.

[sidharthv96]

Removes the "File is a CommonJS module; it may be converted to an ES6 module. ts(80001)" hint in CommonJS Modules, cjs, cts Files.

Fixes #46240

[DanielRosenwasser]

Looks good; I might try consolidating these all into a single test file though.

[sidharthv96]

@DanielRosenwasser I did try to consolidate the tests before raising the PR.
When I added some tests with multiple files inside them, they weren't working as I expected.

Without the changes is suggestionDiagnostics.ts,

Passes (unexpected)

/// <reference path='fourslash.ts' />

// @allowJs: true
// @target: esnext

// @Filename: /a.ts
////const a = 0;

// @Filename: /b.js
////module.exports = 0;

verify.codeFixAvailable([]);

Fails (as Expected)

/// <reference path='fourslash.ts' />

// @allowJs: true
// @target: esnext

// @Filename: /b.js
////module.exports = 0;

verify.codeFixAvailable([]);

That is why I wrote 4 separate test files, If someone familiar with the system can point to how I should include tests in a single file, I'll gladly make the changes.

[andrewbranch]

I think you would need to go to each file with goTo.file or similar and call verify.codeFixAvailable for each.

[sidharthv96]

Thanks @andrewbranch !
I've modified the tests.
The tests with different @module: still had to be separate files as that's a global option.

[sandersn]

aside: Some time we should change errors to use the term "ES module" not "ES6 module".