You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@plc4x.apache.org by cd...@apache.org on 2023/02/23 21:01:39 UTC

[plc4x] branch fix/cdutz/reenable-golang-driver-testsuites updated: fix(plc4go): Fixed one place where Sebastian's HexDiff wasn't used yet.

This is an automated email from the ASF dual-hosted git repository.

cdutz pushed a commit to branch fix/cdutz/reenable-golang-driver-testsuites
in repository https://gitbox.apache.org/repos/asf/plc4x.git


The following commit(s) were added to refs/heads/fix/cdutz/reenable-golang-driver-testsuites by this push:
     new 386cade444 fix(plc4go): Fixed one place where Sebastian's HexDiff wasn't used yet.
386cade444 is described below

commit 386cade4442a96b92cf8747ab379e447d872ee2d
Author: Christofer Dutz <ch...@c-ware.de>
AuthorDate: Thu Feb 23 22:01:31 2023 +0100

    fix(plc4go): Fixed one place where Sebastian's HexDiff wasn't used yet.
---
 plc4go/spi/testutils/DriverTestRunner.go | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/plc4go/spi/testutils/DriverTestRunner.go b/plc4go/spi/testutils/DriverTestRunner.go
index e2f29ddc96..ea2c286b3b 100644
--- a/plc4go/spi/testutils/DriverTestRunner.go
+++ b/plc4go/spi/testutils/DriverTestRunner.go
@@ -331,7 +331,7 @@ func (m DriverTestsuite) ExecuteStep(connection plc4go.PlcConnection, testcase *
 					actual, err := m.rootTypeParser(readBufferByteBased)
 					log.Error().Err(err).Msgf("A readable render of expectation:\n%v\nvs actual paket\n%v\n", expectedSerializable, actual)
 				}
-				return errors.Errorf("actual output doesn't match expected output:\nactual:\n%s\nexpected:\n%s", utils.Dump(actualRawOutput), utils.Dump(expectedRawOutput))
+				return errors.Errorf("actual output doesn't match expected output:\nactual:   0x%X\nexpected: 0x%X\nHexdumps:\n%s", actualRawOutput, expectedRawOutput, utils.DiffHex(actualRawOutput, expectedRawOutput))
 			}
 		}
 		// If there's a difference, parse the input and display it to simplify debugging