fix(skip,limit): use int object for parameter

Uses a neo4j integer object to pass the skip and limit parameter to neo4j. It was complaining about
the old version because it interpreted the number as a floating value.

fix #159

Author: jamesfer

src/clauses/limit.spec.ts

@@ -1,3 +1,4 @@
+import neo4jDriver from 'neo4j-driver';
 import { expect } from 'chai';
 import { Limit } from './limit';
 
@@ -6,7 +7,7 @@ describe('Limit', () => {
     it('should add a produce a limit clause', () => {
       const query = new Limit(10);
       expect(query.build()).to.equal('LIMIT $limitCount');
-      expect(query.getParams()).to.eql({ limitCount: 10 });
+      expect(query.getParams()).to.eql({ limitCount: neo4jDriver.int(10) });
     });
   });
 });

src/clauses/limit.ts

@@ -1,3 +1,4 @@
+import neo4jDriver from 'neo4j-driver';
 import { Clause } from '../clause';
 import { Parameter } from '../parameter-bag';
 
@@ -6,7 +7,7 @@ export class Limit extends Clause {
 
   constructor(public amount: number) {
     super();
-    this.amountParam = this.addParam(amount, 'limitCount');
+    this.amountParam = this.addParam(neo4jDriver.int(amount), 'limitCount');
   }
 
   build() {

src/clauses/skip.spec.ts

@@ -1,12 +1,13 @@
 import { expect } from 'chai';
+import neo4jDriver from 'neo4j-driver';
 import { Skip } from './skip';
 
 describe('Skip', () => {
   describe('#build', () => {
     it('should add a produce a skip clause', () => {
       const query = new Skip(10);
       expect(query.build()).to.equal('SKIP $skipCount');
-      expect(query.getParams()).to.eql({ skipCount: 10 });
+      expect(query.getParams()).to.eql({ skipCount: neo4jDriver.int(10) });
     });
   });
 });

src/clauses/skip.ts

@@ -1,3 +1,4 @@
+import neo4jDriver from 'neo4j-driver';
 import { Clause } from '../clause';
 import { Parameter } from '../parameter-bag';
 
@@ -6,7 +7,7 @@ export class Skip extends Clause {
 
   constructor(public amount: number) {
     super();
-    this.amountParam = this.addParam(amount, 'skipCount');
+    this.amountParam = this.addParam(neo4jDriver.int(amount), 'skipCount');
   }
 
   build() {

tests/scenarios.test.ts

@@ -1,8 +1,8 @@
-import { neo4jCredentials, neo4jUrl, waitForNeo } from './utils';
-import { Connection } from '../src';
-import { expect } from '../test-setup';
 import { Dictionary, isNil } from 'lodash';
+import { Connection } from '../src';
 import { node, relation } from '../src/clauses';
+import { expect } from '../test-setup';
+import { neo4jCredentials, neo4jUrl, waitForNeo } from './utils';
 
 function expectResults(
   results: any[],
@@ -108,6 +108,16 @@ describe('scenarios', () => {
       });
     });
 
+    it('should fetch a single node using limit', async () => {
+      const results = await db.matchNode('person', 'Person')
+        .return('person')
+        .skip(1)
+        .limit(1)
+        .run();
+
+      expectResults(results, 1, ['person']);
+    });
+
     it('should fetch a property of a set of nodes', async () => {
       const results = await db.matchNode('person', 'Person')
         .return({ 'person.age': 'yearsOld' })