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

test: prepare run api tests on ci #127

Merged
merged 3 commits into from
Dec 19, 2023

Conversation

pcheremu
Copy link
Contributor

What ❔

PR with CI workflow and fixed\disabled unstable tests

Why ❔

run integration api tests on CI

Checklist

  • [ +] PR title corresponds to the body of PR (we generate changelog entries from PRs).
  • [ -] Tests for the changes have been added / updated.
  • [ -] Documentation comments have been added / updated.

Copy link

github-actions bot commented Dec 15, 2023

API E2E Test Results

206 tests  ±0   206 ✔️ ±0   28s ⏱️ ±0s
  14 suites ±0       0 💤 ±0 
    1 files   ±0       0 ±0 

Results for commit 31887d0. ± Comparison against base commit 7d93728.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Dec 15, 2023

Visit the preview URL for this PR (updated for commit 31887d0):

(expires Mon, 25 Dec 2023 15:32:15 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: e508f9012944951194447cb8885950b451a24403

@pcheremu pcheremu self-assigned this Dec 15, 2023
@pcheremu pcheremu added the automation writing/fixing/maintaining auto-tests/e2e-tests label Dec 15, 2023
Copy link

github-actions bot commented Dec 15, 2023

Unit Test Results

       3 files  ±0     247 suites  ±0   8m 44s ⏱️ +3s
1 970 tests ±0  1 969 ✔️ ±0  1 💤 ±0  0 ±0 
1 996 runs  ±0  1 995 ✔️ ±0  1 💤 ±0  0 ±0 

Results for commit 31887d0. ± Comparison against base commit 7d93728.

♻️ This comment has been updated with latest results.

@@ -0,0 +1,60 @@
name: Test API
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It probably, makes sense to change the name to Integration test - API or something like that. Making that more specific is a better approach.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

@pcheremu pcheremu requested a review from abilevych December 19, 2023 08:51
@abilevych abilevych merged commit 533b4ed into integration-tests Dec 19, 2023
34 checks passed
@abilevych abilevych deleted the QA-414-ci-run-on-fixed-branch branch December 19, 2023 12:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automation writing/fixing/maintaining auto-tests/e2e-tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants