Skip to content
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

chore(functest): Install ggshield with all extras before functional tests #633

Merged
merged 1 commit into from
Jul 13, 2023

Conversation

Walz
Copy link
Collaborator

@Walz Walz commented Jul 13, 2023

Fix the functional tests in the CI. cryptography wasn't installed and the tests failed.

@codecov-commenter
Copy link

Codecov Report

Merging #633 (182b555) into main (9670a7b) will not change coverage.
The diff coverage is n/a.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@           Coverage Diff           @@
##             main     #633   +/-   ##
=======================================
  Coverage   94.03%   94.03%           
=======================================
  Files         103      103           
  Lines        4813     4813           
=======================================
  Hits         4526     4526           
  Misses        287      287           
Flag Coverage Δ
unittests 94.03% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

@Walz Walz merged commit 68a0030 into main Jul 13, 2023
21 of 22 checks passed
@Walz Walz deleted the sguillaume/fix-functest-ci branch July 13, 2023 15:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants