diff options
| author |   adrianvillin <avillin@cisco.com> | 2024-02-12 02:44:53 -0500 |
|---|---|---|
| committer |   Florin Coras <florin.coras@gmail.com> | 2024-02-15 02:38:10 +0000 |
| commit | 7c675471e87c72be272c078fd481844e4efa64d0 (patch) | |
| tree | 0fe20b6de72178124750a7a8f92155bef4a7c815 /extras/hs-test/suite_veth_test.go | |
| parent | 0215ef1010fbe41a72d57e7cddc4fb18dc3d53d2 (diff) | |
hs-test: improved logging
- improved readability of some error messages
- printing container logs to stdout on test failure (last 20 lines)
Type: test
Change-Id: Idbb358bdd89aa7b1a6bdc9d96bf029d4c299ce64
Signed-off-by: adrianvillin <avillin@cisco.com>
Diffstat (limited to 'extras/hs-test/suite_veth_test.go')
| -rw-r--r-- | extras/hs-test/suite_veth_test.go | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/extras/hs-test/suite_veth_test.go b/extras/hs-test/suite_veth_test.go index de4812d7f97..8593c88740a 100644 --- a/extras/hs-test/suite_veth_test.go +++ b/extras/hs-test/suite_veth_test.go @@ -36,8 +36,8 @@ func (s *VethsSuite) SetupTest() { serverContainer := s.getContainerByName("server-vpp") cpus := s.AllocateCpus() - serverVpp, _ := serverContainer.newVppInstance(cpus, sessionConfig) - s.assertNotNil(serverVpp) + serverVpp, err := serverContainer.newVppInstance(cpus, sessionConfig) + s.assertNotNil(serverVpp, err) s.setupServerVpp() @@ -45,8 +45,8 @@ func (s *VethsSuite) SetupTest() { clientContainer := s.getContainerByName("client-vpp") cpus = s.AllocateCpus() - clientVpp, _ := clientContainer.newVppInstance(cpus, sessionConfig) - s.assertNotNil(clientVpp) + clientVpp, err := clientContainer.newVppInstance(cpus, sessionConfig) + s.assertNotNil(clientVpp, err) s.setupClientVpp() } @@ -57,7 +57,7 @@ func (s *VethsSuite) setupServerVpp() { serverVeth := s.netInterfaces[serverInterfaceName] idx, err := serverVpp.createAfPacket(serverVeth) - s.assertNil(err) + s.assertNil(err, err) s.assertNotEqual(0, idx) } @@ -67,6 +67,6 @@ func (s *VethsSuite) setupClientVpp() { clientVeth := s.netInterfaces[clientInterfaceName] idx, err := clientVpp.createAfPacket(clientVeth) - s.assertNil(err) + s.assertNil(err, err) s.assertNotEqual(0, idx) } |