Closed farost closed 4 months ago
Do not edit the content of this comment. The PR reviewer should simply update this comment by ticking each review item below, as they get completed.
Trivial Change
Code
Architecture
@dmitrii-ubskii
Do we not need to remove / replace the flags anywhere in this repo?
Studio only has unit and integration tests.
Unit tests do not use any server, and integration tests use TypeDB**Runner
, their construction can be found here.
There are no server runs in .circleci as well.
The previous version of TypeDB
used here was 2.26.3
, and diagnostics
have been introduced in 2.27.0
. So we didn't send any data from these tests, but we also don't want to miss anything with this dependency update, so I decided to be explicit and update + retest everything now.
Usage and product changes
We update
typedb
dependencies to run integration tests in the newly introduceddevelopment-mode
that disables external reporting for CI and other test builds. We also fixmaven
artifacts after multiplesync-dependencies
to fix the CI builds.