Skip to content

Commit d8c5ba2

Browse files
committed
Change test fixture from symlink to regular file
The only test using this is test/simple/test-fs-chmod.js, and it was treating a.js and a1.js as two separate files, resulting in a race condition. (Interestingly enough, it was *not* using the symlink file to test lchmod, which uses a different temp file.)
1 parent 3aa2fd3 commit d8c5ba2

File tree

1 file changed

+46
-1
lines changed

1 file changed

+46
-1
lines changed

test/fixtures/a1.js

-1
This file was deleted.

test/fixtures/a1.js

+46
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,46 @@
1+
// Copyright Joyent, Inc. and other Node contributors.
2+
//
3+
// Permission is hereby granted, free of charge, to any person obtaining a
4+
// copy of this software and associated documentation files (the
5+
// "Software"), to deal in the Software without restriction, including
6+
// without limitation the rights to use, copy, modify, merge, publish,
7+
// distribute, sublicense, and/or sell copies of the Software, and to permit
8+
// persons to whom the Software is furnished to do so, subject to the
9+
// following conditions:
10+
//
11+
// The above copyright notice and this permission notice shall be included
12+
// in all copies or substantial portions of the Software.
13+
//
14+
// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS
15+
// OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
16+
// MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN
17+
// NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM,
18+
// DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR
19+
// OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE
20+
// USE OR OTHER DEALINGS IN THE SOFTWARE.
21+
22+
var c = require('./b/c');
23+
24+
console.error('load fixtures/a.js');
25+
26+
var string = 'A';
27+
28+
exports.SomeClass = c.SomeClass;
29+
30+
exports.A = function() {
31+
return string;
32+
};
33+
34+
exports.C = function() {
35+
return c.C();
36+
};
37+
38+
exports.D = function() {
39+
return c.D();
40+
};
41+
42+
exports.number = 42;
43+
44+
process.on('exit', function() {
45+
string = 'A done';
46+
});

0 commit comments

Comments
 (0)