Skip to content

Commit b9bef6d

Browse files
committed
Adding instrumentation to testcase failing on Travis
1 parent 7607df5 commit b9bef6d

File tree

1 file changed

+13
-9
lines changed

1 file changed

+13
-9
lines changed

hdl_checker/tests/test_builders.py

Lines changed: 13 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -642,20 +642,22 @@ def test_GetRebuilds(self, rebuild_info, expected):
642642
with patch.object(
643643
self.builder, "_searchForRebuilds", return_value=[rebuild_info]
644644
):
645-
self.builder._database.getDependenciesByPath = MagicMock(
645+
with patch.object(
646+
self.builder._database,
647+
"getDependenciesByPath",
646648
return_value=[
647649
RequiredDesignUnit(
648650
owner=Path(""),
649651
name=Identifier("very_common_pkg"),
650652
library=Identifier("work"),
651653
)
652-
]
653-
)
654+
],
655+
):
654656

655-
self.assertCountEqual(
656-
self.builder._getRebuilds(_source("source.vhd"), "", library),
657-
{expected},
658-
)
657+
self.assertCountEqual(
658+
self.builder._getRebuilds(_source("source.vhd"), "", library),
659+
{expected},
660+
)
659661

660662

661663
class TestMiscCases(TestCase):
@@ -707,7 +709,9 @@ def requiredDesignUnit(name):
707709
included_results.put(None)
708710

709711
def resolveIncludedPath(*_):
710-
return included_results.get(block=False)
712+
result = included_results.get(block=False)
713+
_logger.info("Returning %s", result)
714+
return result
711715

712716
database.getDependenciesByPath.return_value = [
713717
includedPath(name="resolved/include"),
@@ -719,7 +723,7 @@ def resolveIncludedPath(*_):
719723

720724
calls = [] # type: List[List[str]]
721725

722-
def shell(cmd_with_args, *args, **kwargs):
726+
def shell(cmd_with_args, *_, **__):
723727
calls.append(cmd_with_args)
724728
_logger.debug("$ %s", cmd_with_args)
725729
if "-version" in cmd_with_args:

0 commit comments

Comments
 (0)