Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix typings issues with TestInstantiationService #164297

Closed
14 of 19 tasks
mjbvz opened this issue Oct 21, 2022 · 0 comments · Fixed by #166320
Closed
14 of 19 tasks

Fix typings issues with TestInstantiationService #164297

mjbvz opened this issue Oct 21, 2022 · 0 comments · Fixed by #166320
Assignees
Labels
debt Code quality issues insiders-released Patch has been released in VS Code Insiders
Milestone

Comments

@mjbvz
Copy link
Collaborator

mjbvz commented Oct 21, 2022

Due to an oversight, TestInstantiationService.createInstance() does not enforce arguments or its return type. Fixing this exposes around 80 typings errors in our codebase that should be addressed

To fix this issue in your code:

  1. In instantionService, comment out the first signature for createInstance. (Do not check in this change. I will check in the change to the signature once all error have been fixed)

  2. Now look at the file assigned to you and fix the type error (you may need to restart the compile since we are changing such a core type)

Affected files

  • src/vs/workbench/contrib/terminal/test/browser/xterm/xtermTerminal.test.ts @meganrogge @Tyriar
  • src/vs/workbench/contrib/terminal/test/browser/links/terminalUriLinkDetector.test.ts @meganrogge @Tyriar
  • src/vs/workbench/contrib/terminal/test/browser/links/terminalLocalLinkDetector.test.ts @meganrogge @Tyriar
  • src/vs/workbench/contrib/terminal/test/browser/links/terminalLinkManager.test.ts @meganrogge @Tyriar
  • src/vs/workbench/contrib/notebook/test/browser/contrib/notebookOutline.test.ts @rebornix
  • src/vs/workbench/services/dialogs/test/electron-sandbox/fileDialogService.test.ts @alexr00
  • src/vs/workbench/contrib/terminal/test/common/history.test.ts @meganrogge
  • src/vs/workbench/contrib/tasks/test/browser/taskTerminalStatus.test.ts @meganrogge
  • src/vs/workbench/contrib/search/test/common/searchResult.test.ts @andreamah
  • src/vs/workbench/contrib/search/test/browser/searchViewlet.test.ts @andreamah
  • src/vs/workbench/contrib/search/test/browser/searchActions.test.ts @andreamah
  • src/vs/workbench/contrib/notebook/test/browser/notebookKernelService.test.ts @mjbvz
  • src/vs/workbench/contrib/notebook/test/browser/notebookExecutionService.test.ts @roblourens
  • src/vs/workbench/contrib/notebook/test/browser/notebookEditorModel.test.ts @mjbvz
  • src/vs/workbench/contrib/extensions/test/electron-browser/extensionsViews.test.ts @sandy081
  • src/vs/workbench/contrib/extensions/test/electron-browser/extensionsActions.test.ts @sandy081
  • src/vs/editor/contrib/suggest/test/browser/suggestInlineCompletions.test.ts @hediet
  • src/vs/workbench/api/test/browser/mainThreadWorkspace.test.ts @jrieken
  • src/vs/platform/terminal/test/common/requestStore.test.ts @Tyriar
@mjbvz mjbvz added the debt Code quality issues label Oct 21, 2022
@mjbvz mjbvz added this to the November 2022 milestone Oct 21, 2022
@mjbvz mjbvz self-assigned this Oct 21, 2022
@alexr00 alexr00 self-assigned this Oct 21, 2022
@rebornix rebornix self-assigned this Oct 21, 2022
@sandy081 sandy081 self-assigned this Oct 24, 2022
@meganrogge meganrogge self-assigned this Oct 24, 2022
@alexr00 alexr00 removed their assignment Nov 2, 2022
@sandy081 sandy081 removed their assignment Nov 7, 2022
mjbvz added a commit to mjbvz/vscode that referenced this issue Nov 11, 2022
@roblourens roblourens removed their assignment Nov 11, 2022
@rebornix rebornix removed their assignment Nov 11, 2022
mjbvz added a commit to mjbvz/vscode that referenced this issue Nov 14, 2022
@VSCodeTriageBot VSCodeTriageBot added the unreleased Patch has not yet been released in VS Code Insiders label Nov 14, 2022
@VSCodeTriageBot VSCodeTriageBot added insiders-released Patch has been released in VS Code Insiders and removed unreleased Patch has not yet been released in VS Code Insiders labels Nov 15, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Dec 29, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
debt Code quality issues insiders-released Patch has been released in VS Code Insiders
Projects
None yet
Development

Successfully merging a pull request may close this issue.

10 participants