Attention is currently required from: Nico Huber, Edward O'Callaghan, Angel Pons. Anastasia Klimchuk has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/56413 )
Change subject: tests: Mock file i/o for linux_mtd and linux_spi tests ......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1: This patch has two tests (linux_spi existed before, linux_mtd is new), so I spent some time thinking to split or not to split. Both tests are using file i/o, but what's most important is that after adding a second test it became necessary/clear to add file i/o to the framework. The first test by itself was fine, adding the second one inspired some improvements. So in my mind these two tests are connected, but of course I can split if people think it should be split.
-
Anastasia Klimchuk (Code Review)