node-gradle/gradle-node-plugin

Add HTTPS support to integration tests

Open

#195 geöffnet am 7. Sept. 2021

Auf GitHub ansehen
 (6 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Groovy (122 Forks)batch import
good first issuehelp wanted

Repository-Metriken

Stars
 (674 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Überprüfen Sie die Integrationstest Konfiguration im Repository, wahrscheinlich in Dateien wie `src/test/groovy/...`. Der vorgeschlagene Fix besteht darin, npm audit Anfragen im Test Setup zu deaktivieren, möglicherweise durch Setzen von `npm config set audit false` oder durch Mocken des HTTPS Aufrufs. Prüfen Sie die Issue Kommentare auf zusätzliche Hinweise oder Teil Lösungen.
Tech Stack
javanodejs
Domain
testing
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Familiarity with GradleBasic Java/Groovy
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70