fix path for $count request

pofider · Apr 2, 2018 · a420fb4 · a420fb4
1 parent 28cb0e3
commit a420fb4
Show file tree

Hide file tree

Showing 3 changed files with 20 additions and 2 deletions.
diff --git a/lib/odataServer.js b/lib/odataServer.js
@@ -71,7 +71,7 @@ ODataServer.prototype._initializeRoutes = function () {
     res.writeHead(200, self.cfg.addCorsToHeaders({'Content-Type': 'application/xml', 'DataServiceVersion': '4.0', 'OData-Version': '4.0'}))
     return res.end(result)
   })
-  this.router.get('/:collection/\$count/', function (req, res) {
+  this.router.get('/:collection/\$count', function (req, res) {
     req.params.$count = true
     query(self.cfg, req, res)
   })

diff --git a/package-lock.json b/package-lock.json
diff --git a/test/odataServerTest.js b/test/odataServerTest.js
@@ -104,6 +104,24 @@ describe('odata server', function () {
       })
   })
 
+  it('get $count should return number of entries', function (done) {
+    odataServer.query(function (col, query, req, cb) {
+      cb(null, 1)
+    })
+
+    odataServer.on('odata-error', done)
+
+    request(server)
+      .get('/users/$count')
+      .expect(200)
+      .expect(function (res) {
+        res.body.value.should.be.eql(1)
+      })
+      .end(function (err, res) {
+        done(err)
+      })
+  })
+
   it('get should have the selection fields in its @odata.context if $select is passed', function (done) {
     var selectedField1 = 'num'
     var selectedField2 = 'image'