diff options
author | Mike Blumenkrantz <zmike@samsung.com> | 2018-06-25 15:18:23 -0400 |
---|---|---|
committer | Mike Blumenkrantz <zmike@samsung.com> | 2018-06-25 15:20:37 -0400 |
commit | 32cebe903d39ef212b19f040e68cd35bcfe2b21b (patch) | |
tree | ed73a785c4ac3f7f8fe781d56329e5760ec30c6e | |
parent | 83bab7cab40099d0e46a78bf33dcad4c732f7ea5 (diff) | |
download | efl-32cebe903d39ef212b19f040e68cd35bcfe2b21b.tar.gz |
tests: split a large failure case for eina_file tests to provide more info
Summary:
each failure case should always be separate in order to provide the highest
degree of detail available if a test fails
Reviewers: devilhorns, ManMower
Reviewed By: ManMower
Subscribers: ManMower, cedric, #committers
Tags: #efl
Differential Revision: https://phab.enlightenment.org/D6373
-rw-r--r-- | src/tests/eina/eina_test_file.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/tests/eina/eina_test_file.c b/src/tests/eina/eina_test_file.c index 5810d850c8..242922083f 100644 --- a/src/tests/eina/eina_test_file.c +++ b/src/tests/eina/eina_test_file.c @@ -774,7 +774,9 @@ EFL_START_TEST(eina_test_file_mktemp) errno = 0; tmpfile = NULL; fd = eina_file_mkstemp(patterns[k], &tmpfile); - fail_if((fd < 0) || !tmpfile || errno); + ck_assert(fd >= 0); + ck_assert(!!tmpfile); + ck_assert_msg(!errno, "ERROR(%s): %s\n", patterns[k], strerror(errno)); file = eina_file_open(tmpfile, EINA_FALSE); fail_if(!file); eina_file_close(file); |