10000 fix tests that didn't properly use env variable to look for test by jsteemann · Pull Request #9399 · arangodb/arangodb · GitHub
[go: up one dir, main page]

Skip to content

fix tests that didn't properly use env variable to look for test #9399

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

Conversation

jsteemann
Copy link
Contributor

Scope & Purpose

Fix arangosearch tests that did not use env variable IRESEARCH_TEST_RESOURCE_DIR to determine the location for test files.

  • Bug-Fix for devel-branch (i.e. no need for backports?)
  • The behaviour change can only be verified via automatic tests

Testing & Verification

This change is already covered by existing tests, such as arangosearch catch tests.

https://jenkins01.arangodb.biz/view/PR/job/arangodb-matrix-pr/5047/

@jsteemann jsteemann added this to the devel milestone Jul 3, 2019
@jsteemann jsteemann requested a review from KVS85 Jul 8000 y 3, 2019 13:58
@KVS85
Copy link
Contributor
KVS85 commented Jul 3, 2019

@jsteemann
Copy link
Contributor Author

PR tests blue

@KVS85 KVS85 merged commit 7ffb068 into devel Jul 4, 2019
ObiWahn added a commit that referenced this pull request Jul 4, 2019
…ture/one-shard-db

* 'devel' of https://github.com/arangodb/arangodb:
  Bug fix/internal issue #586 (#9401)
  fix tests that didn't properly use env variable to look for test (#9399)
ObiWahn added a commit that referenced this pull request Jul 4, 2019
…ture/mimalloc

* 'devel' of https://github.com/arangodb/arangodb: (37 commits)
  improve handling when procdump detects the process is dead (#9381)
  Bug fix/internal issue #586 (#9401)
  fix tests that didn't properly use env variable to look for test (#9399)
  Pregel additional test & TSan error fix (#9357)
  use a lock when calling unload (#9375)
  @maierlars 😍 (#9394)
  fix typo (#9400)
  add initializeCursor back to DistinctCollectExecutor (#9386)
  Feature/add tcpdump support (#9396)
  apply filters before starting the server, so we can detect whether no test would be executed (#9387)
  [Devel] Queue-Full-Logging (#9388)
  add VelocyPackHelper::equal method (#9389)
  update velocypack version (#9379)
  Consistent formatting of CHANGELOG. (#9392)
  fix JSON statistics (#9385)
  undo removal (#9391)
  make sure all error code names are prefixed with ERROR_ @fceller @KVS85 (#9384)
  remove catch, refresh versions (#9390)
  fix invalid logId
  Fix ArangoSearch documentation examples
  ...
ObiWahn added a commit that referenced this pull request Jul 4, 2019
…-fix/oasis-statistics

* 'devel' of https://github.com/arangodb/arangodb:
  Don't use non-existent variable (#9407)
  relax test condition for windows (#9393)
  improve handling when procdump detects the process is dead (#9381)
  Bug fix/internal issue #586 (#9401)
  fix tests that didn't properly use env variable to look for test (#9399)
  Pregel additional test & TSan error fix (#9357)
  use a lock when calling unload (#9375)
  @maierlars 😍 (#9394)
  fix typo (#9400)
  add initializeCursor back to DistinctCollectExecutor (#9386)
  Feature/add tcpdump support (#9396)
@fceller fceller deleted the bug-fix/fix-arangosearch-tests-not-obeying-env-variable branch July 30, 2019 12:55
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.

2 participants
0