all relevant tests covered

.gitignore

@@ -1,2 +1,3 @@
 node_modules/
 *.DS_Store
+temp/

package.json

@@ -1,6 +1,6 @@
 {
   "name": "multer",
-  "version": "0.1.2",
+  "version": "0.1.3",
   "description": "Connect middleware for handling multipart/form-data",
   "main": "index.js",
   "repository": {
@@ -34,6 +34,7 @@
     "co": "^3.0.6",
     "express": "*",
     "mocha": "*",
+    "rimraf": "^2.2.8",
     "supertest": "^0.13.0"
   },
   "author": {

test/test.js → test/form.js

@@ -9,7 +9,7 @@ app.use(multer());
 app.post('/', handler);
 app.put('/', handler);
 
- function handler(req, res) {
+function handler(req, res) {
     var form = {
         body: req.body,
         files: req.files
@@ -27,7 +27,7 @@ app2.post('/', function (req, res) {
     res.send(form);
 });
 
-describe('Basic Functionality', function () {
+describe('Form', function () {
 
     it('should process multipart/form-data POST request', function (done) {
 
@@ -90,15 +90,12 @@ describe('Basic Functionality', function () {
     it('should not process non-multipart POST request', function (done) {
         request(app)
             .post('/')
-            .attach('small0', __dirname + '/files/small0.dat')
-            .field('name', 'Multer')
+            .send({name: 'Multer'})
             .expect(200)
             .end(function (err, res) {
                 var form = res.body;
                 expect(err).to.be.null;
-                expect(form.body).to.be.an('object');
                 expect(Object.keys(form.body).length).to.equal(0);
-                expect(form.files).to.be.an('object');
                 expect(Object.keys(form.files).length).to.equal(0);
                 done();
             })

test/functionality.js

@@ -0,0 +1,59 @@
+var fs = require('fs');
+var rimraf = require('rimraf');
+var expect = require('chai').expect
+var request = require('supertest');
+var express = require('express');
+var multer = require('../');
+
+describe('Functionality', function () {
+
+    // delete the temp dir after the tests are run
+    after(function (done) { rimraf('./temp', done); });
+
+    var app = express();
+    app.use(multer({
+        dest: './temp',
+        rename: function (fieldname, filename) {
+            return fieldname + filename;
+        }
+    }));
+    app.post('/', function (req, res) {
+        var form = {
+            body: req.body,
+            files: req.files
+        }
+        res.send(form);
+    });
+
+
+    it('should upload the file to the `dest` dir', function (done) {
+        request(app)
+            .post('/')
+            .type('form')
+            .attach('small0', __dirname + '/files/small0.dat')
+            .expect(200)
+            .end(function (err, res) {
+                var form = res.body;
+                expect(err).to.be.null;
+                expect(fs.existsSync('./temp/small0small0.dat')).to.equal(true);
+                done();
+            })
+    })
+
+
+    it('should rename the uploaded file', function (done) {
+        request(app)
+            .post('/')
+            .type('form')
+            .attach('small0', __dirname + '/files/small0.dat')
+            .expect(200)
+            .end(function (err, res) {
+                var form = res.body;
+                expect(err).to.be.null;
+                expect(form.files.small0.name).to.equal('small0small0.dat');
+                done();
+            })
+    })
+
+
+})