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

[Testing] Supplier staking E2E test #498

Merged
merged 6 commits into from
Apr 28, 2024
Merged

[Testing] Supplier staking E2E test #498

merged 6 commits into from
Apr 28, 2024

Conversation

Olshansk
Copy link
Member

Summary

E2E Tests for staking the supplier: make test_e2e_supplier

Issue

Type of change

Select one or more:

  • New feature, functionality or library
  • Bug fix
  • Code health or cleanup
  • Documentation
  • Other (specify)

Testing

Local Testing (only if making code changes)

  • Unit Tests: make go_develop_and_test
  • LocalNet E2E Tests: make test_e2e
  • See quickstart guide for instructions

PR Testing (only if making code changes)

  • DevNet E2E Tests: Add the devnet-test-e2e label to the PR.
    • THIS IS VERY EXPENSIVE, so only do it after all the reviews are complete.
    • Optionally run make trigger_ci if you want to re-trigger tests without any code changes
    • If tests fail, try re-running failed tests only using the GitHub UI as shown here

@Olshansk Olshansk added this to the Shannon MainNet milestone Apr 25, 2024
@Olshansk Olshansk requested a review from red-0ne April 25, 2024 22:00
@Olshansk Olshansk self-assigned this Apr 25, 2024
@Olshansk Olshansk added supplier Changes related to the Supplier actor testing Test (or test utils) additions, fixes, improvements or other devnet-test-e2e labels Apr 25, 2024
Copy link

The CI will now also run the e2e tests on devnet, which increases the time it takes to complete all CI checks. If you just created a pull request, you might need to push another commit to produce a container image DevNet can utilize to spin up infrastructure. You can use make trigger_ci to push an empty commit.

@github-actions github-actions bot added devnet push-image CI related - pushes images to ghcr.io labels Apr 25, 2024
@Olshansk Olshansk requested a review from okdas April 25, 2024 23:22
Copy link
Contributor

@red-0ne red-0ne left a comment

Choose a reason for hiding this comment

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

Left a couple of comments but nothing blocking 👍

e2e/tests/init_test.go Outdated Show resolved Hide resolved
e2e/tests/init_test.go Outdated Show resolved Hide resolved
Copy link
Contributor

@red-0ne red-0ne left a comment

Choose a reason for hiding this comment

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

Love it!

I want this to add gateways to it.

@red-0ne red-0ne merged commit a752fca into main Apr 28, 2024
9 checks passed
bryanchriswhite added a commit that referenced this pull request Apr 29, 2024
…cept

* pokt/main:
  [Dashboard] Supplier logs dashboard (#502)
  [Utility] Update SessionSettlement when app goes into a negative amount (#509)
  [Testing] Supplier staking E2E test (#498)
  [RelayMiner] Support https backend urls (#507)
  [Bug] Fix pprof endpoint (#505)
  [LocalNet] Remove tilt-restart-wrapper (#504)
  [LocalNet][Hotfix] Turn off pproff by default (#500)
  [Tooling] Add pprof endpoints and documentation (#484)
  [Documentation] Basic blockchain operations (#454)
  [BlockClient] Refactor BlockClient to fetch latest block at init (#444)
@bryanchriswhite bryanchriswhite removed push-image CI related - pushes images to ghcr.io devnet-test-e2e labels May 16, 2024
@github-actions github-actions bot removed the devnet label May 16, 2024
@Olshansk Olshansk deleted the supplier_stake_e2e branch May 29, 2024 16:43
okdas pushed a commit that referenced this pull request Nov 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
supplier Changes related to the Supplier actor testing Test (or test utils) additions, fixes, improvements or other utility
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

3 participants