Skip to content
This repository has been archived by the owner on Feb 26, 2024. It is now read-only.

test: ensure tests that are skipped in development are run in CI #4353

Merged
merged 4 commits into from
Apr 20, 2023

Conversation

jeffsmale90
Copy link
Contributor

@jeffsmale90 jeffsmale90 commented Apr 13, 2023

Previously, tests that depend on an INFURA_KEY would be skipped if it were not provided. After this change these tests will fail in CI if no INFURA_KEY is provided.

Additionally, a test which was previously skipped is now run on Windows and Linux agents in Github Actions only. ( > server > listens on given interface only)

…'listens on given interface only' test which was incorrectly skipped in CI.
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Apr 13, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: dcb8bde
Status: ✅  Deploy successful!
Preview URL: https://9eecb02b.ganache.pages.dev
Branch Preview URL: https://test-improve-infura-integrat.ganache.pages.dev

View logs

@jeffsmale90 jeffsmale90 marked this pull request as ready for review April 13, 2023 03:45
@jeffsmale90 jeffsmale90 changed the title test: improve resilience of tests that depend on INFURA_KEY being set test: ensure tests that are skipped in development are run in CI Apr 13, 2023
Copy link
Member

@davidmurdoch davidmurdoch left a comment

Choose a reason for hiding this comment

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

just one small nit

Co-authored-by: David Murdoch <187813+davidmurdoch@users.noreply.github.com>
@jeffsmale90 jeffsmale90 merged commit 1281876 into develop Apr 20, 2023
@jeffsmale90 jeffsmale90 deleted the test/improve_infura_integration_tests branch April 20, 2023 02:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants