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

fix: integration tests #1064

Merged
merged 4 commits into from
Nov 9, 2023
Merged

fix: integration tests #1064

merged 4 commits into from
Nov 9, 2023

Conversation

mbazhlekova
Copy link
Contributor

@mbazhlekova mbazhlekova commented Nov 8, 2023

This PR fixes the failing integration tests.

  1. Tests for deprecated APIs that always throw errors are skipped
  2. The Workflows API has some changes we needed to make for v2 users and it is now working again
  3. Synthetics tests need to be run sequentially else they throw errors
  4. Some tests for cloud and entity tags remain flakey and fail intermittently

@codecov-commenter
Copy link

codecov-commenter commented Nov 8, 2023

Codecov Report

Merging #1064 (262e576) into main (9883c83) will increase coverage by 16.75%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main    #1064       +/-   ##
===========================================
+ Coverage   41.30%   58.05%   +16.75%     
===========================================
  Files          79       82        +3     
  Lines        4995     5057       +62     
===========================================
+ Hits         2063     2936      +873     
+ Misses       2784     1873      -911     
- Partials      148      248      +100     
Flag Coverage Δ
integration 58.05% <ø> (?)
unit ?

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

see 79 files with indirect coverage changes

📣 Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!

@mbazhlekova mbazhlekova marked this pull request as draft November 9, 2023 15:04
@mbazhlekova mbazhlekova marked this pull request as ready for review November 9, 2023 16:50
rthorn-nr
rthorn-nr previously approved these changes Nov 9, 2023
@mbazhlekova mbazhlekova merged commit f452d6f into main Nov 9, 2023
14 checks passed
@mbazhlekova mbazhlekova deleted the maria/fix-integration-tests branch November 9, 2023 17: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.

4 participants