Skip to content
This repository was archived by the owner on May 9, 2023. It is now read-only.

Commit 766a88c

Browse files
committed
Failing tests fixed
1 parent 94c8a99 commit 766a88c

File tree

2 files changed

+13
-4
lines changed

2 files changed

+13
-4
lines changed

pkg/resources/service/pingsource_test.go

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -91,8 +91,6 @@ func TestPingSource(t *testing.T) {
9191
assert.Len(t, psList.Items, 1)
9292
ps := psList.Items[0]
9393

94-
assert.True(t, ps.Status.IsReady())
95-
9694
assert.Equal(t, ps.Spec.Sink.Ref.Name, tc.service.Name)
9795
assert.Equal(t, ps.Spec.Sink.Ref.Namespace, tc.service.Namespace)
9896

pkg/resources/service/service_test.go

Lines changed: 13 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@ package service
1717
import (
1818
"fmt"
1919
"net"
20+
"net/url"
2021
"os"
2122
"strings"
2223
"testing"
@@ -94,8 +95,18 @@ func TestDeployAndDelete(t *testing.T) {
9495
return
9596
}
9697

97-
address := strings.TrimPrefix(output, fmt.Sprintf("Service %s URL: https://", tc.service.Name))
98-
conn, err := net.DialTimeout("tcp", address+":443", dialTimeout)
98+
address := strings.TrimPrefix(output, fmt.Sprintf("Service %s URL: ", tc.service.Name))
99+
addr, err := url.Parse(address)
100+
assert.NoError(t, err)
101+
switch addr.Scheme {
102+
case "https":
103+
address = addr.Host + ":443"
104+
case "http":
105+
address = addr.Host + ":80"
106+
default:
107+
t.Error("malformed service URL scheme", addr.Scheme)
108+
}
109+
conn, err := net.DialTimeout("tcp", address, dialTimeout)
99110
assert.NoError(t, err)
100111
assert.NoError(t, conn.Close())
101112

0 commit comments

Comments
 (0)