Azure/ng-deploy-azure

Use SDK for E2E tests

Ouverte

#50 ouverte le 18 août 2019

Voir sur GitHubGuide contributeur
 (0 commentaire) (0 réaction) (0 personne assignée)TypeScript (43 forks)github user discovery
bugenhancementhelp wanted

Métriques du dépôt

Stars
 (79 étoiles)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

Describe the bug When running the e2e test, the script uses az - the Azure CLI to clean up the resources. However, we can't expect all developers of ng deploy to have the Azure CLI installed. The cleanup should be done using the resources that are already available - the SDKs that are used in the project.

Error: ./scripts/test.sh: line 44: az.cmd: command not found To Reproduce Steps to reproduce the behavior:

  1. If you have Azure CLI, uninstall it. (Run az to see that it's not available)
  2. Run npm run test:e2e
  3. If the tests run as expected, after getting the link of the website, it crashes with the error message above.
  4. Click on the link - it opens the website. Meaning the resources were not cleared.

Expected behavior 3 - the test script should not crash. 4 - the link should not be available.

Guide contributeur

Direction de recherche
Remplacez les commandes de l'interface Azure CLI dans le script de test e2e par des appels SDK équivalents à l'aide des packages @azure/identity et @azure/arm resources.
Stack technique
typescriptangularazurenodejs
Domaine
testingdevops
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Node.jsAzure SDK
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75