diff options
author | Simon Glass <sjg@chromium.org> | 2023-01-23 11:29:41 -0700 |
---|---|---|
committer | Simon Glass <sjg@chromium.org> | 2023-01-26 10:47:45 -0700 |
commit | 060a65e899859dcbf42049a18be20ce7118e7c0e (patch) | |
tree | 6289299feae412a54dd6251985bb65bfe5dc9501 /tools/binman/entry_test.py | |
parent | c2e13aa9e1a99da9334f0a666783c8f7dd98016f (diff) |
binman: Fix a test-coverage regression
Unfortunately a recent patch snuck through without the require test
coverage. Fix it.
Signed-off-by: Simon Glass <sjg@chromium.org>
Fixes: 571bc4e67d3 ("binman: Support positioning an entry by and ELF symbol")
Diffstat (limited to 'tools/binman/entry_test.py')
-rw-r--r-- | tools/binman/entry_test.py | 19 |
1 files changed, 19 insertions, 0 deletions
diff --git a/tools/binman/entry_test.py b/tools/binman/entry_test.py index aa470c58163..a6fbf62731f 100644 --- a/tools/binman/entry_test.py +++ b/tools/binman/entry_test.py @@ -114,6 +114,25 @@ class TestEntry(unittest.TestCase): self.assertEquals(tools.get_bytes(0, 1024), base.CompressData(b'abc')) self.assertEquals(tools.get_bytes(0, 1024), base.DecompressData(b'abc')) + def testLookupOffset(self): + """Test the lookup_offset() method of the base class""" + def MyFindEntryByNode(node): + return self.found + + base = entry.Entry.Create(None, self.GetNode(), 'blob-dtb') + base.FindEntryByNode = MyFindEntryByNode + base.section = base + self.found = None + base.offset_from_elf = [self.GetNode(), 'start', 0] + with self.assertRaises(ValueError) as e: + base.lookup_offset() + self.assertIn("Cannot find entry for node 'u-boot'", str(e.exception)) + + self.found = base + with self.assertRaises(ValueError) as e: + base.lookup_offset() + self.assertIn("Need elf-fname property 'u-boot'", str(e.exception)) + if __name__ == "__main__": unittest.main() |