From 8f81047e0223e60d2f2d20500321afb34fffaad7 Mon Sep 17 00:00:00 2001 From: Henrik Hautakoski Date: Wed, 12 Oct 2022 12:24:52 +0200 Subject: [PATCH] src/parse_request_test.go: compare api's LogInfo() function as comparing the structs directly. Comparing the structs may failsly error because of internal variables (like pointer) not being equal. So just compare the data we actually care about, and those is returned by LogInfo(). --- src/parse_request_test.go | 19 ++++++++++--------- 1 file changed, 10 insertions(+), 9 deletions(-) diff --git a/src/parse_request_test.go b/src/parse_request_test.go index 85156e4..9732940 100644 --- a/src/parse_request_test.go +++ b/src/parse_request_test.go @@ -33,7 +33,7 @@ func TestParseEosioV1(t *testing.T) { api, err := ParseRequest("v1|http://api.example.com") assert.NoError(t, err) - assert.Equal(t, expected, api) + assert.Equal(t, expected.LogInfo(), api.LogInfo()) } func TestParseEosioV1WithBlockNumber(t *testing.T) { @@ -42,7 +42,7 @@ func TestParseEosioV1WithBlockNumber(t *testing.T) { api, err := ParseRequest("v1|http://api.example.com|2000") assert.NoError(t, err) - assert.Equal(t, expected, api) + assert.Equal(t, expected.LogInfo(), api.LogInfo()) } @@ -52,7 +52,7 @@ func TestParseEosioV1Full(t *testing.T) { api, err := ParseRequest("v1|http://api.example.com|1000|http://host.example.com") assert.NoError(t, err) - assert.Equal(t, expected, api) + assert.Equal(t, expected.LogInfo(), api.LogInfo()) } // EosioV2 @@ -64,7 +64,7 @@ func TestParseEosioV2(t *testing.T) { api, err := ParseRequest("v2|http://api.v2.example.com") assert.NoError(t, err) - assert.Equal(t, expected, api) + assert.Equal(t, expected.LogInfo(), api.LogInfo()) } func TestParseEosioV2WithOffset(t *testing.T) { @@ -73,7 +73,7 @@ func TestParseEosioV2WithOffset(t *testing.T) { api, err := ParseRequest("v2|http://api.v2.example.com|1000") assert.NoError(t, err) - assert.Equal(t, expected, api) + assert.Equal(t, expected.LogInfo(), api.LogInfo()) } func TestParseEosioV2Full(t *testing.T) { @@ -82,7 +82,8 @@ func TestParseEosioV2Full(t *testing.T) { api, err := ParseRequest("v2|http://api.v2.example.com|1000|http://host.example.com") assert.NoError(t, err) - assert.Equal(t, expected, api) + + assert.Equal(t, expected.LogInfo(), api.LogInfo()) } // EosioContract @@ -94,7 +95,7 @@ func TestParseEosioContract(t *testing.T) { api, err := ParseRequest("contract|http://api.contract.example.com") assert.NoError(t, err) - assert.Equal(t, expected, api) + assert.Equal(t, expected.LogInfo(), api.LogInfo()) } func TestParseEosioContractWithBlockTime(t *testing.T) { @@ -103,7 +104,7 @@ func TestParseEosioContractWithBlockTime(t *testing.T) { api, err := ParseRequest("contract|http://api.contract.example.com|512") assert.NoError(t, err) - assert.Equal(t, expected, api) + assert.Equal(t, expected.LogInfo(), api.LogInfo()) } func TestParseDebugApi(t *testing.T) { @@ -112,5 +113,5 @@ func TestParseDebugApi(t *testing.T) { api, err := ParseRequest("debug|some_api_call") assert.NoError(t, err) - assert.Equal(t, expected, api) + assert.Equal(t, expected.LogInfo(), api.LogInfo()) }