Skip to content

Commit c2c03cb

Browse files
authored
[test] Remove test/wasmfs/get_backend.h. NFC (#23828)
These days we have a better mechanism that uses `setup_nodefs.js` as a `--pre-js`. See `setup_nodefs_test` in `test/common.py` The advantage of this new mechanism is that it doesn't require the test itself to know about wasmfs or any specific backend.
1 parent 4c91962 commit c2c03cb

File tree

6 files changed

+45
-76
lines changed

6 files changed

+45
-76
lines changed

test/other/test_unlink.cpp

Lines changed: 1 addition & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -12,23 +12,9 @@
1212
#include <unistd.h>
1313
#include <assert.h>
1414

15-
#ifdef WASMFS
16-
#include "../wasmfs/get_backend.h"
17-
#include <emscripten/wasmfs.h>
18-
#endif
19-
2015
int main() {
2116
const char *filename = "test.dat";
22-
const char *dirname = "test";
23-
24-
#ifdef WASMFS
25-
if (wasmfs_create_directory("root", 0777, get_backend()) != 0) {
26-
return 1;
27-
}
28-
if (chdir("root") != 0) {
29-
return 1;
30-
}
31-
#endif
17+
const char *dirname = "test.dir";
3218

3319
// Create a file
3420
int fd = open(filename, O_RDWR | O_CREAT, 0777);
@@ -61,7 +47,6 @@ int main() {
6147
assert(access(dirname, F_OK) == -1);
6248

6349
// The rest of this test does not yet pass with the node backend!
64-
#ifndef WASMFS_NODE_BACKEND
6550

6651
// Check that we can still read the directory, but that it is empty.
6752
errno = 0;
@@ -92,8 +77,6 @@ int main() {
9277
close(parent);
9378
#endif
9479

95-
#endif // WASMFS_NODE_BACKEND
96-
9780
closedir(d);
9881

9982
printf("ok\n");

test/test_other.py

Lines changed: 14 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -118,11 +118,16 @@ def wasmfs_all_backends(f):
118118
@wraps(f)
119119
def metafunc(self, backend, *args, **kwargs):
120120
self.setup_wasmfs_test()
121+
if backend == 'node':
122+
self.setup_nodefs_test()
123+
elif backend == 'raw':
124+
self.setup_noderawfs_test()
121125
self.emcc_args.append(f'-D{backend}')
122126
f(self, *args, **kwargs)
123127

124-
parameterize(metafunc, {'': ('WASMFS_MEMORY_BACKEND',),
125-
'node': ('WASMFS_NODE_BACKEND',)})
128+
parameterize(metafunc, {'': ('memory',),
129+
'node': ('node',),
130+
'raw': ('raw',)})
126131
return metafunc
127132

128133

@@ -133,12 +138,17 @@ def also_with_wasmfs_all_backends(f):
133138
def metafunc(self, backend, *args, **kwargs):
134139
if backend:
135140
self.setup_wasmfs_test()
141+
if backend == 'node':
142+
self.setup_nodefs_test()
143+
elif backend == 'raw':
144+
self.setup_noderawfs_test()
136145
self.emcc_args.append(f'-D{backend}')
137146
f(self, *args, **kwargs)
138147

139148
parameterize(metafunc, {'': (None,),
140-
'wasmfs': ('WASMFS_MEMORY_BACKEND',),
141-
'wasmfs_node': ('WASMFS_NODE_BACKEND',)})
149+
'wasmfs': ('memory',),
150+
'wasmfs_node': ('node',),
151+
'wasmfs_raw': ('raw',)})
142152
return metafunc
143153

144154

test/wasmfs/get_backend.h

Lines changed: 0 additions & 26 deletions
This file was deleted.

test/wasmfs/wasmfs_getdents.c

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -20,8 +20,6 @@
2020
#include <sys/syscall.h>
2121
#include <unistd.h>
2222

23-
#include "get_backend.h"
24-
2523
void print_one(int fd) {
2624
struct dirent d;
2725
int nread = getdents(fd, &d, sizeof(d));
@@ -54,21 +52,23 @@ void print(const char* dir) {
5452
}
5553

5654
int main() {
57-
int err = wasmfs_create_directory("/root", 0777, get_backend());
55+
int err;
5856

5957
// Set up test directories.
60-
err = mkdir("/root/working", 0777);
58+
err = mkdir("root", 0777);
59+
assert(err != -1);
60+
err = mkdir("root/working", 0777);
6161
assert(err != -1);
62-
err = mkdir("/root/working/test", 0777);
62+
err = mkdir("root/working/test", 0777);
6363
assert(err != -1);
6464

6565
struct dirent d;
6666

6767
// Try opening the directory that was just created.
68-
printf("------------- Reading from /root/working Directory -------------\n");
69-
print("/root/working");
68+
printf("------------- Reading from root/working Directory -------------\n");
69+
print("root/working");
7070

71-
int fd = open("/root/working", O_RDONLY | O_DIRECTORY);
71+
int fd = open("root/working", O_RDONLY | O_DIRECTORY);
7272

7373
// Try reading an invalid fd.
7474
errno = 0;
@@ -109,17 +109,17 @@ int main() {
109109

110110
// Try to advance the offset of the directory.
111111
// Expect that '.' will be skipped.
112-
fd = open("/root/working", O_RDONLY | O_DIRECTORY);
113-
printf("/root/working file position is: %lli\n", lseek(fd, 1, SEEK_SET));
112+
fd = open("root/working", O_RDONLY | O_DIRECTORY);
113+
printf("root/working file position is: %lli\n", lseek(fd, 1, SEEK_SET));
114114
printf(
115-
"------------- Reading one from /root/working Directory -------------\n");
115+
"------------- Reading one from root/working Directory -------------\n");
116116
print_one(fd);
117117
close(fd);
118118

119119
// Try to add a file to the /working directory.
120-
fd = open("/root/working/foobar", O_CREAT, S_IRGRP);
120+
fd = open("root/working/foobar", O_CREAT, S_IRGRP);
121121
assert(fd != -1);
122122
close(fd);
123-
printf("------------- Reading from /root/working Directory -------------\n");
124-
print("/root/working");
123+
printf("------------- Reading from root/working Directory -------------\n");
124+
print("root/working");
125125
}

test/wasmfs/wasmfs_getdents.out

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
------------- Reading from /root/working Directory -------------
1+
------------- Reading from root/working Directory -------------
22
d.d_name = .
33
d.d_reclen = 280
44
d.d_type = directory
@@ -58,13 +58,13 @@ stdin
5858
stdout
5959
urandom
6060

61-
/root/working file position is: 1
62-
------------- Reading one from /root/working Directory -------------
61+
root/working file position is: 1
62+
------------- Reading one from root/working Directory -------------
6363
d.d_name = ..
6464
d.d_reclen = 280
6565
d.d_type = directory
6666

67-
------------- Reading from /root/working Directory -------------
67+
------------- Reading from root/working Directory -------------
6868
d.d_name = .
6969
d.d_reclen = 280
7070
d.d_type = directory

test/wasmfs/wasmfs_readfile.c

Lines changed: 12 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -6,21 +6,19 @@
66
*/
77

88
#include <assert.h>
9-
#include <emscripten.h>
10-
#include <emscripten/wasmfs.h>
9+
#include <emscripten/em_asm.h>
1110
#include <errno.h>
1211
#include <fcntl.h>
1312
#include <stdio.h>
1413
#include <stdlib.h>
1514
#include <string.h>
15+
#include <sys/stat.h>
1616
#include <unistd.h>
1717

18-
#include "get_backend.h"
19-
2018
int main() {
21-
int err = wasmfs_create_directory("/root", 0777, get_backend());
19+
int err = mkdir("root", 0777);
2220
assert(err == 0);
23-
int fd = open("/root/test", O_CREAT | O_WRONLY, S_IRUSR | S_IWUSR);
21+
int fd = open("root/test", O_CREAT | O_WRONLY, S_IRUSR | S_IWUSR);
2422
const char* msg = "Success\n";
2523

2624
errno = 0;
@@ -30,25 +28,29 @@ int main() {
3028
assert(err == 0);
3129

3230
EM_ASM({
33-
var output = FS.readFile("/root/test");
31+
var output = FS.readFile("root/test");
3432
out(UTF8ArrayToString(output));
3533
out("Length: " + output.byteLength);
36-
var err = FS.unlink("/root/test");
34+
var err = FS.unlink("root/test");
3735
out("FS.unlink: " + err);
3836
});
3937

4038
EM_ASM({
4139
try {
42-
var output = FS.readFile("/root/no-exist", {encoding : 'utf8'});
40+
var output = FS.readFile("root/no-exist", {encoding : 'utf8'});
4341
} catch (err) {
42+
return;
4443
}
44+
assert(false);
4545
});
4646

4747
EM_ASM({
4848
try {
4949
// Already unlinked above, file should not exist anymore
50-
var output = FS.readFile("/root/test", {encoding : 'utf8'});
50+
var output = FS.readFile("root/test", {encoding : 'utf8'});
5151
} catch (err) {
52+
return;
5253
}
54+
assert(false);
5355
});
5456
}

0 commit comments

Comments
 (0)