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