[flashrom-gerrit] [S] Change in flashrom[master]: tests: Use MOCK_FD instead of NON_ZERO for file operations

[Anastasia Klimchuk (Code Review)]

Anastasia Klimchuk has uploaded this change for review. ( https://review.coreboot.org/c/flashrom/+/67310 )

Change subject: tests: Use MOCK_FD instead of NON_ZERO for file operations
......................................................................

tests: Use MOCK_FD instead of NON_ZERO for file operations

NON_ZERO can be a negative number, so MOCK_FD is safer option to
use as a mock file descriptor. Also it is more readable.

Change-Id: I097dd59f69c3fb532ac136796fcf5cae8839af7b
Signed-off-by: Anastasia Klimchuk 
---
M tests/tests.c
1 file changed, 15 insertions(+), 2 deletions(-)

git pull ssh://review.coreboot.org:29418/flashrom refs/changes/10/67310/1

diff --git a/tests/tests.c b/tests/tests.c
index 2994c02..f6b4df1 100644
--- a/tests/tests.c
+++ b/tests/tests.c
@@ -24,7 +24,7 @@

void *not_null(void)
 {
-	return (void *)NON_ZERO;
+	return (void *)MOCK_FD;
 }

/* Workaround for https://github.com/clibs/cmocka/issues/17 */
@@ -253,7 +253,7 @@
 int __wrap_fileno(FILE *fp)
 {
    LOG_ME;
-	return NON_ZERO;
+	return MOCK_FD;
 }

int __wrap_fsync(int fd)