node-gradle/gradle-node-plugin

Add HTTPS support to integration tests

Open

#195 ouverte le 7 sept. 2021

Voir sur GitHub
 (6 commentaires) (0 réactions) (0 assignés)Groovy (122 forks)batch import
good first issuehelp wanted

Métriques du dépôt

Stars
 (674 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

With no changes in the code we've started getting failures like this:

java.lang.AssertionError: Request not found exactly once, expected:<{
          "method" : "GET",
          "path" : "/case",
          "headers" : {
            "Host" : [ "registry.npmjs.org:80" ]
          },
          "secure" : false
        }> but was:<{
          "method" : "POST",
          "path" : "/-/npm/v1/security/audits/quick",

We probably just need to disable audits in the integration tests since we're mocking for proxy testing purposes.

Guide contributeur

Direction de recherche
Examinez la configuration des tests d'intégration dans le dépôt, probablement dans des fichiers comme `src/test/groovy/...`. La correction suggérée est de désactiver les requêtes npm audit dans la configuration des tests, éventuellement en définissant `npm config set audit false` ou en simulant l'appel HTTPS. Consultez les commentaires de l'issue pour toute orientation supplémentaire ou solutions partielles.
Stack technique
javanodejs
Domaine
testing
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.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Familiarity with GradleBasic Java/Groovy
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70