express-mung - npm Package Compare versions

Comparing version 0.1.1 to 0.2.0

index.js

@@ -6,2 +6,5 @@ 'use strict';
 
+function isScalar(v) {
+    return typeof v !== 'object' && !Array.isArray(v);
+}
 
@@ -24,2 +27,8 @@ mung.json = function json (fn) {
 
+            // If scalar value, then text/plain
+            if (isScalar(json)) {
+                res.set('content-type', 'text/plain');
+                return res.send(json);
+            }
+
             return original.call(this, json);
@@ -44,3 +53,9 @@ }
 
-                    return original.call(this, json);
+                    // If scalar value, then text/plain
+                    if (isScalar(json)) {
+                        res.set('content-type', 'text/plain');
+                        return res.send(json);
+                    }
+
+                return original.call(this, json);
             });
@@ -47,0 +62,0 @@

package.json

 {
   "name": "express-mung",
-  "version": "0.1.1",
+  "version": "0.2.0",
   "description": "Transform an express response (or make until no good)",
@@ -5,0 +5,0 @@ "main": "index.js",

README.md

@@ -53,2 +53,4 @@ # express-mung [![Build Status](https://travis-ci.org/richardschneider/express-prefer.svg)](https://travis-ci.org/richardschneider/express-mung)
 
+**NOTE** when `mung.json*` receives a scalar value then the `content-type` is switched `text-plain`.
+
 # License
@@ -55,0 +57,0 @@ The MIT license

test/json.js

@@ -18,2 +18,6 @@ 'use strict';
 
+    function reduce (json, req, res) {
+        return json.a;
+    }
+
     it('should return the munged JSON result', done => {
@@ -59,2 +63,16 @@ let server = express()
 
+    it('should return a munged scalar result as text/plain', done => {
+        let server = express()
+            .use(mung.json(reduce))
+            .get('/', (req, res) => res.status(200).json({ a: 'a' }).end());
+        request(server)
+            .get('/')
+            .expect(200)
+            .expect(res => {
+                res.text.should.equal('a');
+                res.headers.should.have.property('content-type', 'text/plain; charset=utf-8');
+            })
+            .end(done);
+    });
+
 })

test/jsonAsync.js

@@ -24,2 +24,8 @@ 'use strict';
 
+    function reduce (json, req, res) {
+        return Promise.resolve(json)
+            .then(json => json.a);
+    }
+
+
     it('should return the munged JSON result', done => {
@@ -50,3 +56,16 @@ let server = express()
 
+    it('should return a munged scalar result as text/plain', done => {
+        let server = express()
+            .use(mung.jsonAsync(reduce))
+            .get('/', (req, res) => res.status(200).json({ a: 'alpha' }).end());
+        request(server)
+            .get('/')
+            .expect(200)
+            .expect(res => {
+                res.text.should.equal('alpha');
+                res.headers.should.have.property('content-type', 'text/plain; charset=utf-8');
+            })
+            .end(done);
+    });
 
 })

No alert changes

Improved metrics

Total package byte prevSize

13510

increased by15.6%

Lines of code

249

increased by22.66%

Number of lines in readme file

59

increased by3.51%

No dependency changes