Skip to content

Commit

Permalink
fix(riseupvpn): make tests compile without warnings
Browse files Browse the repository at this point in the history
This change lands on top of @cyBerta changes to make sure the tests
compile after recent renamings and to avoid warnings.
  • Loading branch information
bassosimone authored and cyBerta committed Aug 4, 2023
1 parent 768102b commit 7aec078
Showing 1 changed file with 3 additions and 6 deletions.
9 changes: 3 additions & 6 deletions internal/experiment/riseupvpn/riseupvpn_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -14,9 +14,8 @@ import (
"github.com/google/go-cmp/cmp"
"github.com/ooni/probe-cli/v3/internal/experiment/riseupvpn"
"github.com/ooni/probe-cli/v3/internal/experiment/urlgetter"
"github.com/ooni/probe-cli/v3/internal/legacy/mockable"
"github.com/ooni/probe-cli/v3/internal/mocks"
"github.com/ooni/probe-cli/v3/internal/model"
"github.com/ooni/probe-cli/v3/internal/model/mocks"
"github.com/ooni/probe-cli/v3/internal/netxlite"
"github.com/ooni/probe-cli/v3/internal/tracex"
)
Expand Down Expand Up @@ -696,7 +695,7 @@ func TestMissingTransport(t *testing.T) {
}

ctx := context.Background()
sess := &mockable.Session{MockableLogger: log.Log}
sess := &mocks.Session{MockLogger: func() model.Logger { return log.Log }}
measurement := new(model.Measurement)
callbacks := model.NewPrinterCallbacks(log.Log)
args := &model.ExperimentArgs{
Expand Down Expand Up @@ -1100,9 +1099,7 @@ func runDefaultMockTest(t *testing.T, multiGetter urlgetter.MultiGetter) *model.
args := &model.ExperimentArgs{
Callbacks: model.NewPrinterCallbacks(log.Log),
Measurement: measurement,
Session: &mockable.Session{
MockableLogger: log.Log,
},
Session: &mocks.Session{MockLogger: func() model.Logger { return log.Log }},
}
err := measurer.Run(context.Background(), args)

Expand Down

0 comments on commit 7aec078

Please sign in to comment.