particle-os/debian-forge-composer
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

client: drop client timeout in tests

Browse source 
A timeout doesn't make sense on this level, because it is very difficult
to estimate how long downloading rpm metadata takes. Drop it completely
in favor of higher-level timeouts in the test runner.

Fixes #601
This commit is contained in:
Lars Karlitski 2020-05-12 20:08:17 +02:00
parent d49f0fef44
commit 2ecdee6814
3 changed files with 3 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

3
internal/client/integration_test.go
View file

@ -9,7 +9,6 @@ import (
"fmt"
"os"
"testing"
"time"
"github.com/osbuild/osbuild-composer/internal/test"
)
@ -21,7 +20,7 @@ var testState *TestState
// Also makes sure there is a running server to test against
func executeTests(m *testing.M) int {
var err error
testState, err = setUpTestState("/run/weldr/api.socket", 60*time.Second, false)
testState, err = setUpTestState("/run/weldr/api.socket", false)
if err != nil {
fmt.Printf("ERROR: Test setup failed: %s\n", err)
panic(err)

3
internal/client/unit_test.go
View file

@ -12,7 +12,6 @@ import (
"net/http"
"os"
"testing"
"time"
"github.com/osbuild/osbuild-composer/internal/distro/fedoratest"
rpmmd_mock "github.com/osbuild/osbuild-composer/internal/mocks/rpmmd"
@ -58,7 +57,7 @@ func executeTests(m *testing.M) int {
}
}()
testState, err = setUpTestState(socketPath, 60*time.Second, true)
testState, err = setUpTestState(socketPath, true)
if err != nil {
log.Fatalf("ERROR: Test setup failed: %s\n", err)
}

5
internal/client/utils.go
View file

@ -11,7 +11,6 @@ import (
"net/http"
"sort"
"strconv"
"time"
)
type TestState struct {
@ -32,12 +31,10 @@ func isStringInSlice(slice []string, s string) bool {
return false
}
func setUpTestState(socketPath string, timeout time.Duration, unitTest bool) (*TestState, error) {
func setUpTestState(socketPath string, unitTest bool) (*TestState, error) {
state := TestState{unitTest: unitTest}
state.socket = &http.Client{
// TODO This may be too short/simple for downloading images
Timeout: timeout,
Transport: &http.Transport{
DialContext: func(_ context.Context, _, _ string) (net.Conn, error) {
return net.Dial("unix", socketPath)