Skip to content

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