node-gradle/gradle-node-plugin

Add HTTPS support to integration tests

Open

#195 opened on Sep 7, 2021

View on GitHub
 (6 comments) (0 reactions) (0 assignees)Groovy (122 forks)batch import
good first issuehelp wanted

Repository metrics

Stars
 (674 stars)
PR merge metrics
 (No merged PRs in 30d)

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.

Contributor guide

Research direction
Review the integration test configuration in the repository, likely in files like `src/test/groovy/...`. The suggested fix is to disable npm audit requests in the test setup, possibly by setting `npm config set audit false` or mocking the HTTPS call. Check the issue comments for any additional guidance or partial solutions.
Tech stack
javanodejs
Domain
testing
Issue type
Bug
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
Under 1 hour
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
Familiarity with GradleBasic Java/Groovy
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
70