chore: add some test additional checks

Signed-off-by: Valery Piashchynski <piashchynski.valery@gmail.com>
roadrunner-server · Sep 5, 2024 · 1ea2498 · 1ea2498
1 parent 5af38c0
commit 1ea2498
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 2 deletions.
diff --git a/tests/configs/.rr-broken-pipes.yaml b/tests/configs/.rr-broken-pipes.yaml
@@ -1,7 +1,7 @@
 version: '3'
 
 rpc:
-  listen: tcp://127.0.0.1:6002
+  listen: tcp://127.0.0.1:6666
 
 server:
   command: "php php_test_files/http/client.php broken pipes"

diff --git a/tests/http_plugin3_test.go b/tests/http_plugin3_test.go
@@ -696,6 +696,7 @@ func TestMTLS1(t *testing.T) {
 
 	r, err := client.Do(req)
 	assert.NoError(t, err)
+	require.NotNil(t, r)
 
 	b, err := io.ReadAll(r.Body)
 	assert.NoError(t, err)
@@ -792,6 +793,7 @@ func TestMTLS2(t *testing.T) {
 
 	r, err := client.Do(req)
 	assert.NoError(t, err)
+	require.NotNil(t, r)
 
 	b, err := io.ReadAll(r.Body)
 	assert.NoError(t, err)
@@ -884,6 +886,7 @@ func TestMTLS3(t *testing.T) {
 
 	r, err := client.Do(req)
 	assert.NoError(t, err)
+	require.NotNil(t, r)
 
 	b, err := io.ReadAll(r.Body)
 	assert.NoError(t, err)
@@ -976,6 +979,7 @@ func TestMTLS4(t *testing.T) {
 
 	r, err := client.Do(req)
 	assert.NoError(t, err)
+	require.NotNil(t, r)
 
 	b, err := io.ReadAll(r.Body)
 	assert.NoError(t, err)

diff --git a/tests/http_plugin4_test.go b/tests/http_plugin4_test.go
@@ -213,6 +213,7 @@ func http3ResponseMatcher(t *testing.T) {
 
 	r, err := client.Do(req)
 	assert.NoError(t, err)
+	require.NotNil(t, r)
 
 	b, err := io.ReadAll(r.Body)
 	assert.NoError(t, err)

diff --git a/tests/http_plugin_test.go b/tests/http_plugin_test.go
@@ -541,6 +541,7 @@ func sslEcho(t *testing.T) {
 
 	r, err := client.Do(req)
 	assert.NoError(t, err)
+	require.NotNil(t, r)
 
 	b, err := io.ReadAll(r.Body)
 	assert.NoError(t, err)
@@ -665,7 +666,8 @@ func sslRedirect(t *testing.T) {
 
 	r, err := client.Do(req)
 	assert.NoError(t, err)
-	assert.NotNil(t, r.TLS)
+	require.NotNil(t, r)
+	require.NotNil(t, r.TLS)
 
 	b, err := io.ReadAll(r.Body)
 	assert.NoError(t, err)
@@ -764,6 +766,7 @@ func sslPush(t *testing.T) {
 
 	r, err := client.Do(req)
 	assert.NoError(t, err)
+	require.NotNil(t, r)
 
 	assert.NotNil(t, r.TLS)