percona / percona-everest-cli

Apache License 2.0
6 stars 5 forks source link

EVEREST-107 Fix tests run #185

Closed oksana-grishchenko closed 1 year ago

oksana-grishchenko commented 1 year ago

Fix integration tests failure

Problem: EVEREST-107

The integration tests were failing with

2023-10-20T15:27:22Z    error   install/operators.go:207    Could not connect to Everest. Make sure Everest is running and is accessible from this machine. {"component": "install/operators"}

Long story short - BE runs in background too slow and tests are running before it gets available


Investigation:

  1. Tried to move the BE run to the foreground to check if it gets available it all - it does.
  2. Noticed it takes long to run the BE (perhaps because of the vault has been added)
  3. Added the waiting step, so the CI would check if the BE is actually available before running the tests. It worked.
codecov-commenter commented 1 year ago

Codecov Report

Merging #185 (2edaef3) into main (f7ac65b) will not change coverage. Report is 1 commits behind head on main. The diff coverage is n/a.

@@          Coverage Diff          @@
##            main    #185   +/-   ##
=====================================
  Coverage   5.02%   5.02%           
=====================================
  Files         13      13           
  Lines       3005    3005           
=====================================
  Hits         151     151           
  Misses      2827    2827           
  Partials      27      27           

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more