c/ooni-probe-cli
Watch
1
0
Fork
You've already forked ooni-probe-cli
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Revert "fix(riseupvpn): skip currently failing tests (#212)" (#225)

Browse source 
This reverts commit 43f95e2c9a.

We can now safely close https://github.com/ooni/probe/issues/1338,
since the upstream issue is gone. I think this issue we have seen is
also possibly related to https://github.com/ooni/probe/issues/1348.

Cc: @cyberta.
This commit is contained in:
Simone Basso 2021-02-11 10:10:52 +01:00 • committed by GitHub
commit c41114261c
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 0 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

2
internal/engine/experiment/riseupvpn/riseupvpn_test.go
View file

@ -33,7 +33,6 @@ func TestNewExperimentMeasurer(t *testing.T) {
} }
func TestGood(t *testing.T) { func TestGood(t *testing.T) {
t.Skip("broken test; see https://github.com/ooni/probe/issues/1338")
if testing.Short() { if testing.Short() {
t.Skip("skip test in short mode") t.Skip("skip test in short mode")
} }
@ -279,7 +278,6 @@ func TestFailureGeoIpServiceBlocked(t *testing.T) {
} }
func TestFailureGateway(t *testing.T) { func TestFailureGateway(t *testing.T) {
t.Skip("broken test; see https://github.com/ooni/probe/issues/1338")
var testCases = [...]string{"openvpn", "obfs4"} var testCases = [...]string{"openvpn", "obfs4"}
eipService, err := fetchEipService() eipService, err := fetchEipService()
if err != nil { if err != nil {