Skip to content

Commit 97f482d

Browse files
committed
Fixed couple tests to run with neo4j 3.4
Two tests failed when executed against snapshot version of neo4j 3.4. The did not expect returned Cypher runtime name to be "SLOTTED", which is the new default starting from 3.4. This commit fixes tests to only verify that some defined string is returned as runtime name.
1 parent b011bc1 commit 97f482d

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

test/v1/session.test.js

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@ import FakeConnection from '../internal/fake-connection';
2626
import sharedNeo4j from '../internal/shared-neo4j';
2727
import _ from 'lodash';
2828
import {ServerVersion, VERSION_3_1_0} from '../../src/v1/internal/server-version';
29+
import {isString} from '../../src/v1/internal/util';
2930

3031
describe('session', () => {
3132

@@ -291,7 +292,7 @@ describe('session', () => {
291292
expect(sum.hasPlan()).toBe(true);
292293
expect(sum.hasProfile()).toBe(false);
293294
expect(sum.plan.operatorType).toBe('ProduceResults');
294-
expect(sum.plan.arguments.runtime).toBe('INTERPRETED');
295+
expect(isString(sum.plan.arguments.runtime)).toBeTruthy();
295296
expect(sum.plan.identifiers[0]).toBe('n');
296297
expect(sum.plan.children[0].operatorType).toBe('CreateNode');
297298
done();
@@ -310,7 +311,7 @@ describe('session', () => {
310311
expect(sum.hasPlan()).toBe(true); //When there's a profile, there's a plan
311312
expect(sum.hasProfile()).toBe(true);
312313
expect(sum.profile.operatorType).toBe('ProduceResults');
313-
expect(['INTERPRETED', 'COMPILED']).toContain(sum.profile.arguments.runtime);
314+
expect(isString(sum.profile.arguments.runtime)).toBeTruthy();
314315
expect(sum.profile.identifiers[0]).toBe('n');
315316
expect(sum.profile.children[0].operatorType).toBe('Filter');
316317
expect(sum.profile.rows).toBe(0);

0 commit comments

Comments
 (0)