Skip to content

[Feature] Fix Mocha this typing in smoke test dispose hooks #2636

@EmmaYuan1015

Description

@EmmaYuan1015
  • Problem:
    • TS next reports TS2683: 'this' implicitly has type 'any' in smoke suite dispose functions.
  • Current locations:
    • test/smoke/suites/actionBar.test.ts
    • test/smoke/suites/commands.test.ts
    • test/smoke/suites/debugConfiguration.test.ts
    • test/smoke/suites/fileExplorer.test.ts
    • test/smoke/suites/packager.test.ts
  • Proposed change:
    • Type hook context explicitly, e.g. async function dispose(this: Mocha.Context) { ... }.
    • Ensure @types/mocha usage is consistent in smoke project.
  • Acceptance criteria:
    • TS2683 is removed for all dispose hooks.
    • Smoke test behavior is unchanged (screenshots still taken on failures).

Metadata

Metadata

Assignees

No one assigned

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions