Merge pull request #443 from koajs/fix-query-cache
return same object from request.query
This commit is contained in:
commit
8c32dd2540
2 changed files with 7 additions and 2 deletions
|
@ -131,8 +131,6 @@ module.exports = {
|
||||||
|
|
||||||
get query() {
|
get query() {
|
||||||
var str = this.querystring;
|
var str = this.querystring;
|
||||||
if (!str) return {};
|
|
||||||
|
|
||||||
var c = this._querycache = this._querycache || {};
|
var c = this._querycache = this._querycache || {};
|
||||||
return c[str] || (c[str] = qs.parse(str));
|
return c[str] || (c[str] = qs.parse(str));
|
||||||
},
|
},
|
||||||
|
|
|
@ -7,6 +7,13 @@ describe('ctx.query', function(){
|
||||||
var ctx = context({ url: '/' });
|
var ctx = context({ url: '/' });
|
||||||
ctx.query.should.eql({});
|
ctx.query.should.eql({});
|
||||||
})
|
})
|
||||||
|
|
||||||
|
it('should return the same object each time it\'s accessed', function(done) {
|
||||||
|
var ctx = context({ url: '/' });
|
||||||
|
ctx.query.a = '2';
|
||||||
|
ctx.query.a.should.equal('2');
|
||||||
|
done();
|
||||||
|
});
|
||||||
})
|
})
|
||||||
|
|
||||||
it('should return a parsed query-string', function(){
|
it('should return a parsed query-string', function(){
|
||||||
|
|
Loading…
Reference in a new issue