Merge pull request #193 from cnpm/issue189-search-api

support json result in search, fixed #189
cnpm · Feb 19, 2014 · 90959ba · 90959ba
2 parents 666d98d + 0f6b6a2
commit 90959ba
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 1 deletion.
diff --git a/controllers/web/package.js b/controllers/web/package.js
@@ -120,7 +120,14 @@ exports.search = function (req, res, next) {
     if (err) {
       return next(err);
     }
-
+    // return a json result
+    if (req.query && req.query.type === 'json') {
+      return res.json({
+        keyword: word,
+        packages: result.searchMatchs,
+        keywords: result.keywordMatchs
+      });
+    }
     res.render('search', {
       title: 'Keyword - ' + word,
       keyword: word,

diff --git a/test/controllers/web/package.test.js b/test/controllers/web/package.test.js
@@ -129,6 +129,13 @@ describe('controllers/web/package.test.js', function () {
       .expect(/Packages match/, done);
     });
 
+    it('should list by keyword with json ok', function (done) {
+      request(app)
+      .get('/browse/keyword/cnpm?type=json')
+      .expect(200)
+      .expect('content-type', 'application/json; charset=utf-8', done);
+    });
+
     it('should list no match ok', function (done) {
       request(app)
       .get('/browse/keyword/notexistpackage')