Skip to content

Commit a53b93f

Browse files
committed
Update tests to support empty files
1 parent d8c00c5 commit a53b93f

File tree

4 files changed

+2
-10
lines changed

4 files changed

+2
-10
lines changed

test/fileFactory.spec.js

-8
Original file line numberDiff line numberDiff line change
@@ -26,14 +26,6 @@ describe('Test of the fileFactory factory', function() {
2626
beforeEach(() => server.clearUploadsDir());
2727

2828
it('return a file object', () => assert.ok(fileFactory(mockFileOpts)));
29-
it('return void if buffer is empty and useTempFiles is false.', () => {
30-
assert.equal(fileFactory({
31-
name: mockFileName,
32-
buffer: Buffer.concat([])
33-
}, {
34-
useTempFiles: false
35-
}), null);
36-
});
3729

3830
describe('Properties', function() {
3931
it('contains the name property', () => {

test/files/emptyfile.txt

Whitespace-only changes.

test/multipartUploads.spec.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ const uploadDir = server.uploadDir;
1212
const clearTempDir = server.clearTempDir;
1313
const clearUploadsDir = server.clearUploadsDir;
1414

15-
const mockFiles = ['car.png', 'tree.png', 'basketball.png'];
15+
const mockFiles = ['car.png', 'tree.png', 'basketball.png', 'emptyfile.txt'];
1616

1717
const mockUser = {
1818
firstName: 'Joe',

test/server.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ const setup = (fileUploadOptions) => {
4040
const testFile = req.files.testFile;
4141
const fileData = getUploadedFileData(testFile);
4242

43-
testFile.mv(fileData.uploadPath, function(err) {
43+
testFile.mv(fileData.uploadPath, (err) => {
4444
if (err) {
4545
console.log('ERR', err); // eslint-disable-line
4646
return res.status(500).send(err);

0 commit comments

Comments
 (0)