[flashrom-gerrit] Change in flashrom[master]: tests: test write protection

[Sergii Dmytruk (Code Review)]

Attention is currently required from: Nico Huber, Thomas Heijligen.
Sergii Dmytruk has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/59075 )

Change subject: tests: test write protection
......................................................................

Patch Set 39:

(2 comments)

Patchset:

PS39:
> Can you provide a description what each test is doing?

Wouldn't that just duplicate test name, code and comments in tests? Maybe provide an example of a description you'd like to see.

File tests/chip_wp.c:

https://review.coreboot.org/c/flashrom/+/59075/comment/d7c6ab35_eaef692a 
PS39, Line 63: static const struct flashchip chip_W25Q128_V = {
> Why copying the chip?

It's copied in `chip.c`, so I did the same. I think the rational is that tests don't operate on real chips, so they should use their own definition with real chip as a reference.