Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Skip to content

Commit 07efb17

Browse files
committed
Improved tests coverage
1 parent f2ff04f commit 07efb17

File tree

2 files changed

+15
-2
lines changed

2 files changed

+15
-2
lines changed

src/Static/index.js

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -41,8 +41,7 @@ class Static {
4141
return new Promise((resolve, reject) => {
4242
this.server.serve(request, response, function (err, good) {
4343
if (err) {
44-
err.message = err.message || 'Unknown error'
45-
err.message = `${err.message} while resolving ${request.url}`
44+
err.message = `Route ${err.message} while resolving ${request.url}`
4645
reject(err)
4746
return
4847
}

test/unit/response.spec.js

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -110,6 +110,20 @@ describe('Response', function () {
110110
done()
111111
})
112112

113+
it('should make jsonp response using jsonp default callback when callback is missing in query string', function * (done) {
114+
115+
const server = http.createServer((req, res) => {
116+
const request = new Request(req,res, Config)
117+
const response = new this.Response(request, res)
118+
response.jsonp({name:"foo"})
119+
})
120+
121+
const res = yield supertest(server).get('/').expect(200).expect('Content-type',/javascript/).end()
122+
expect(res.text).to.match(/typeof callback/)
123+
done()
124+
})
125+
126+
113127
it('should set request status', function * (done) {
114128
const server = http.createServer((req, res) => {
115129
const request = new Request(req,res, Config)

0 commit comments

Comments
 (0)