c/ooni-probe-cli
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix(ooapi): disable tests we're not using (#525)

Browse Source 
At the moment ooapi is not used. It will eventually be used since
it's a better way of accessing the OONI backend API.

To fix these tests, we need to fix the swagger emitted by the
backend API, which is not a priority at the moment, since we are
working instead to integrate websteps in miniooni.

Issue https://github.com/ooni/probe/issues/1790 tracks the work
required to re-enabled the tests I'm skipping with this diff.

This work is part of https://github.com/ooni/probe/issues/1733.
This commit is contained in:
Simone Basso 2021-09-30 00:04:11 +02:00 committed by GitHub
parent 26f84ccc20
commit 24a6ac9a5f
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
internal/ooapi/swaggerdiff_test.go
View File

@ -139,6 +139,7 @@ func compare(serverURL string) bool {
} }
func TestWithProductionAPI(t *testing.T) { func TestWithProductionAPI(t *testing.T) {
t.Skip("skip until we use this part of the codebase")
if testing.Short() { if testing.Short() {
t.Skip("skip test in short mode") t.Skip("skip test in short mode")
} }
@ -149,6 +150,7 @@ func TestWithProductionAPI(t *testing.T) {
} }
func TestWithTestingAPI(t *testing.T) { func TestWithTestingAPI(t *testing.T) {
t.Skip("skip until we use this part of the codebase")
if testing.Short() { if testing.Short() {
t.Skip("skip test in short mode") t.Skip("skip test in short mode")
} }