Change in flashrom[master]: tests: Add tests to erase a chip

[Angel Pons (Code Review)]

Attention is currently required from: Nico Huber, Edward O'Callaghan, Anastasia Klimchuk.
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/56501 )

Change subject: tests: Add tests to erase a chip
......................................................................

Patch Set 2:

(2 comments)

File tests/chip.c:

https://review.coreboot.org/c/flashrom/+/56501/comment/93e4eb1d_ef70eeb1 
PS1, Line 42: printf("Unlock chip called\n");
> Currently unlock is optional, which is […]

If you're emulating a chip, I'd also mock and verify locking to increase test coverage

https://review.coreboot.org/c/flashrom/+/56501/comment/1f6947c7_471a372c 
PS1, Line 57: 	char *param_dup = strdup("");
> This only works for the case when param is empty string. […]

Hmmm, I guess the `memmove` inside that function changes the haystack. If so... Eww!