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

Test: Install a set of extensions #12647

Closed
2 of 3 tasks
chrmarti opened this issue Sep 26, 2016 · 7 comments
Closed
2 of 3 tasks

Test: Install a set of extensions #12647

chrmarti opened this issue Sep 26, 2016 · 7 comments
Labels
extensions Issues concerning extensions testplan-item

Comments

@chrmarti
Copy link
Collaborator

chrmarti commented Sep 26, 2016

Test for #11937

Complexity - 4

Set up

  • Make sure you can publish an extension. Ref - http://code.visualstudio.com/docs/tools/vscecli
  • Use the sample test extensions
  • This repo has 3 extension packs (1, 2, 3) which are already published
  • Each has some dependencies - Look at it's package.json for dependencies
  • Use these extensions for testing following scenarios (You can also come up with your own if you want to :) )
  • Also do play around with changing dependencies and bumping the versions to test different scenarios for installing and updating.

Note: Publishing the extension takes a while to get reflected in the market place.

Extensions with dependencies

  • Search for one of the above extensions and install
  • A prompt appears asking for confirmation to install its dependencies also
  • If the dependencies are already installed, then no prompt is shown
  • Yes action - Installs the extension and all its dependencies and sub dependencies (transitive closure).
  • No action - Install the extension alone.
  • Cancel action - Cancels the installation.

Installing with dependencies

  • It's status shows installing and is finished only after itself and all dependencies are installed
  • If any of the dependencies is visible in the Extensions view, then it's status also shows installing and is finished only after all are installed
  • If the installation fails in between, installed extensions are rolled back / uninstalled.

Updating an extension with dependencies

  • If it was installed with dependencies, then update should also install any newly added dependencies
  • Otherwise, update should only update itself
  • Make sure the install with dependencies prompt is not shown during update
  • Update All and Auto update should have the same behavior

Others

  • Uninstalling an extension (with dependencies) is atomic - uninstalls only itself.
  • Behavior of Install / update / uninstall an extension with out dependencies is not changed
@chrmarti chrmarti added this to the September 2016 milestone Sep 26, 2016
@sandy081 sandy081 changed the title Test: Publish and install a set of extensions Test: Install a set of extensions Sep 26, 2016
@sandy081 sandy081 added the extensions Issues concerning extensions label Sep 26, 2016
@joaomoreno
Copy link
Member

joaomoreno commented Sep 27, 2016

@chrmarti @sandy081 This issue is big, it definitely needs more than one tester and needs testing across platforms... I added two more checkboxes and split them according to OS.

@jrieken
Copy link
Member

jrieken commented Sep 28, 2016

@chrmarti I was unable to test this today. It would be good if redmond can jump in

@chrmarti
Copy link
Collaborator Author

@jrieken Will do, thanks.

@mjbvz
Copy link
Collaborator

mjbvz commented Sep 28, 2016

I see this error when trying to install the published test-extension-pack-2 on Windows and selecting "Yes":

image

When I hit this error, I'd already mucked around with these extensions quite a bit by installing them and uninstalling them, and doing the same for the individual extensions in the packs, so I'm not sure exactly what caused this. I'm also having trouble getting a repo.

Anyone else hitting an issue like this? I'll see if I can get consistent repo steps.

@Tyriar
Copy link
Member

Tyriar commented Sep 28, 2016

@mjbvz I'd guess it's probably a problem with that particular extension. Depends what touches/created .vscode/Icon though.

@roblourens
Copy link
Member

Does that file exist?

@Tyriar
Copy link
Member

Tyriar commented Sep 28, 2016

@mjbvz actually I'd file a bug, .vscode/Icon is probably part of the extension install code to cache the icon.

@Tyriar Tyriar removed their assignment Sep 28, 2016
@roblourens roblourens removed their assignment Sep 28, 2016
@mjbvz mjbvz removed their assignment Sep 28, 2016
@mjbvz mjbvz closed this as completed Sep 28, 2016
@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 18, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
extensions Issues concerning extensions testplan-item
Projects
None yet
Development

No branches or pull requests

7 participants