[riot-notifications] [RIOT-OS/RIOT] pkg/fatfs/fatfs_vfs: fix flag translation in _open (#14175)

benpicco notifications at github.com
Fri May 29 19:34:44 CEST 2020


@benpicco commented on this pull request.



> +static void test_newlib(void)
+{
+    FILE* fl;
+    char buf[sizeof(test_txt) + sizeof(test_txt2)];
+    print_test_result("test_newlib__mount", vfs_mount(&_test_vfs_mount) == 0);
+
+    /* try to open file that doesn't exist */
+    fl = fopen(FULL_FNAME_NXIST, "r");
+    print_test_result("test_newlib__fopen", fl == NULL);
+    if (fl) {
+        fclose(fl);
+    }
+
+    /* open file RO */
+    fl = fopen(FULL_FNAME1, "r");
+    print_test_result("test_newlib__fopen_r", fl != NULL);

This will always fail as that file got renamed by `test_rename()` - and now `test_create()` fails.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/14175#pullrequestreview-421153779
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20200529/60144254/attachment.htm>


More information about the notifications mailing list