Merge pull request #187 from yoavgurevich/issue62v2

Fixes #62
This commit is contained in:
Alan K 2014-05-21 16:17:19 -04:00
commit 247ebd113a
3 changed files with 17 additions and 6 deletions

View File

@ -68,7 +68,7 @@ define(function(require) {
SUPER_NODE_ID: '00000000-0000-0000-0000-000000000000',
//Reserved FileDescriptors for streams
// Reserved File Descriptors for streams
STDIN: 0,
STDOUT: 1,
STDERR: 2,
@ -80,4 +80,4 @@ define(function(require) {
}
};
});
});

View File

@ -22,7 +22,7 @@ define(function(require) {
var STDIN = require('src/constants').STDIN;
var STDOUT = require('src/constants').STDOUT;
var STDERR = require('src/constants').STDERR;
var FD = require('src/constants').FIRST_DESCRIPTOR;
var FIRST_DESCRIPTOR = require('src/constants').FIRST_DESCRIPTOR;
// The core fs operations live on impl
var impl = require('src/filesystem/implementation');
@ -82,12 +82,11 @@ define(function(require) {
fs.stdin = STDIN;
fs.stdout = STDOUT;
fs.stderr = STDERR;
fs.firstFD = FD;
// Safely expose the list of open files and file
// descriptor management functions
var openFiles = {};
var nextDescriptor = 3;
var nextDescriptor = FIRST_DESCRIPTOR;
Object.defineProperty(this, "openFiles", {
get: function() { return openFiles; }
});

View File

@ -62,7 +62,7 @@ define(["Filer", "util"], function(Filer, util) {
it('should return a unique file descriptor', function(done) {
var fs = util.fs();
var fd1
var fd1;
fs.open('/file1', 'w+', function(error, fd) {
if(error) throw error;
@ -79,6 +79,18 @@ define(["Filer", "util"], function(Filer, util) {
});
});
it('should return the argument value of the file descriptor index matching the value set by the first useable file descriptor constant', function(done) {
var fs = util.fs();
var firstFD = require('src/constants').FIRST_DESCRIPTOR;
var fd1;
fs.open('/file1', 'w+', function(error, fd) {
if(error) throw error;
expect(fd).to.equal(firstFD);
done();
});
});
it('should create a new file when flagged for write', function(done) {
var fs = util.fs();