Skip to content

Supporting browser, integration and stress tests on testkit #641

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

Merged
merged 1 commit into from
Dec 1, 2020

Conversation

bigmontz
Copy link
Contributor

@bigmontz bigmontz commented Nov 24, 2020

Teskit doesn't run Neo4j on the localhost and with the same credentials of boltkit, but it exposes
the host and credentials as env variables.

So all this configuration being loaded to karma on the config files, parsed in shared-neo4j with fallback to
the process env (case it's running the usual integration test) and to the default boltkit values.

All the integration tests were changed to use the values from the shareNeo4j object.

Disabled tests:

  • IPV6 tests are skipped on the testkit because of the lack of support for this
  • TLS tests are disabled since it's already tested on the testkit suite
  • Tests which depends on the enterprise edition of Neo4j is running only on this scenario

@bigmontz bigmontz closed this Nov 25, 2020
@bigmontz bigmontz reopened this Nov 25, 2020
@bigmontz bigmontz force-pushed the 4.3-supporting-browser-tests-testkit branch from a5a9f2c to da70749 Compare November 25, 2020 14:17
Teskit doesn't run Neo4j on the localhost and with the same credentials of boltkit, but it exposes
the host and credentials as env variables.

So all this configuration being loaded to karma on the config files, parsed in `shared-neo4j` with fallback to
the process env (case it's running the usual integration test) and to the default boltkit values.

All the integration tests were changed to use the values from the shareNeo4j object.

Disabled tests:
* IPV6 tests are skipped on the testkit because of the lack of support for this
* TLS tests are disabled since it's already tested on the testkit suite
* Tests which depends on the enterprise edition of Neo4j is running only on this scenario
@bigmontz bigmontz force-pushed the 4.3-supporting-browser-tests-testkit branch from da70749 to 73b2336 Compare November 25, 2020 18:14
@bigmontz bigmontz changed the title Supporting browser tests on testkit Supporting browser, integration and stress tests on testkit Nov 25, 2020
@bigmontz bigmontz marked this pull request as ready for review November 27, 2020 09:09
@bigmontz bigmontz merged commit c6513c4 into neo4j:4.3 Dec 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant