New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix tests and code samples checker #1020
Fix tests and code samples checker #1020
Conversation
@@ -20,6 +20,7 @@ const DEV_CLUSTER_CONFIG = ` | |||
xsi:schemaLocation="http://www.hazelcast.com/schema/config | |||
http://www.hazelcast.com/schema/config/hazelcast-config-4.0.xsd"> | |||
<cluster-name>dev</cluster-name> | |||
<jet enabled="true"></jet> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
some code samples use sql, so we need to enable it in the cluster that code samples are run against
@@ -59,6 +61,7 @@ describe('Data type test', function () { | |||
let mapName; | |||
const clientVersionNewerThanFive = TestUtil.isClientVersionAtLeast('5.0'); | |||
const serverVersionNewerThanFive = TestUtil.isServerVersionAtLeast(client, '5.0'); | |||
const JET_ENABLED_CONFIG = fs.readFileSync(path.join(__dirname, 'jet_enabled.xml'), 'utf8'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
all the tests that require sql need jet as enabled
resultSpy.callCount.should.be.greaterThanOrEqual(4); | ||
serviceSpy.callCount.should.be.greaterThanOrEqual(4); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
after a change fetch may not get rows as full in non-last pages. see the issue for details: hazelcast/hazelcast#19313
resultSpy.callCount.should.be.greaterThanOrEqual(4); | ||
serviceSpy.callCount.should.be.greaterThanOrEqual(4); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same as above comment
fixes test failures due to jet being disabled.