Skip to content

Commit de5cb8f

Browse files
committed
Enable proxy tests again, they should work
1 parent 13c6b78 commit de5cb8f

File tree

4 files changed

+3
-8
lines changed

4 files changed

+3
-8
lines changed

.github/workflows/build.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,9 +6,9 @@ jobs:
66
strategy:
77
matrix:
88
config:
9-
- {os: ubuntu-latest, params: "-PtestAllSupportedGradleVersions=true -PtestProxyIntegrationTests=false" }
10-
- {os: windows-latest, params: "-PtestProxyIntegrationTests=false" }
11-
- {os: macos-latest, params: "-PtestProxyIntegrationTests=false" }
9+
- {os: ubuntu-latest, params: "-PtestAllSupportedGradleVersions=true" }
10+
- {os: windows-latest, params: "" }
11+
- {os: macos-latest, params: "" }
1212
steps:
1313
- name: Checkout Source Code
1414
uses: actions/checkout@v2

build.gradle.kts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,6 @@ tasks.test {
7474
)
7575
systemProperty("testMinimumCurrentGradleVersion", project.properties["testMinimumCurrentGradleVersion"] ?: "false")
7676
systemProperty("testCurrentGradleVersion", project.properties["testCurrentGradleVersion"] ?: "true")
77-
systemProperty("testProxyIntegrationTests", project.properties["testProxyIntegrationTests"] ?: "true")
7877

7978
val processorsCount = Runtime.getRuntime().availableProcessors()
8079
maxParallelForks = if (processorsCount > 2) processorsCount.div(2) else processorsCount

src/test/groovy/com/github/gradle/node/npm/task/NpmProxy_integTest.groovy

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,13 +5,11 @@ import com.github.gradle.node.ProxyTestHelper
55
import org.gradle.testkit.runner.TaskOutcome
66
import org.mockserver.integration.ClientAndServer
77
import org.mockserver.socket.PortFactory
8-
import spock.lang.Requires
98

109
import static org.mockserver.integration.ClientAndServer.startClientAndServer
1110
import static org.mockserver.model.HttpRequest.request
1211
import static org.mockserver.verify.VerificationTimes.exactly
1312

14-
@Requires({ System.getProperty("testProxyIntegrationTests").equals("true") })
1513
class NpmProxy_integTest extends AbstractIntegTest {
1614
private ClientAndServer proxyMockServer
1715

src/test/groovy/com/github/gradle/node/yarn/task/YarnProxy_integTest.groovy

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,15 +5,13 @@ import com.github.gradle.node.ProxyTestHelper
55
import org.gradle.testkit.runner.TaskOutcome
66
import org.mockserver.integration.ClientAndServer
77
import org.mockserver.socket.PortFactory
8-
import spock.lang.Requires
98

109
import static java.nio.charset.StandardCharsets.UTF_8
1110
import static org.mockserver.integration.ClientAndServer.startClientAndServer
1211
import static org.mockserver.model.HttpRequest.request
1312
import static org.mockserver.model.SocketAddress.Scheme.HTTPS
1413
import static org.mockserver.verify.VerificationTimes.exactly
1514

16-
@Requires({ System.getProperty("testProxyIntegrationTests").equals("true") })
1715
class YarnProxy_integTest extends AbstractIntegTest {
1816
private ClientAndServer proxyMockServer
1917
// We have to configure a second proxy otherwise Yarn does not want to use its own repository using HTTP,

0 commit comments

Comments
 (0)