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.

Sign up for GitHub

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

Jump to bottom

Integration tests for prod config #2161

Merged
merged 17 commits into from
Nov 13, 2023
Merged

Integration tests for prod config #2161

merged 17 commits into from
Nov 13, 2023

Conversation

stepanLav
Copy link
Member

@stepanLav stepanLav commented Oct 25, 2023 •
edited
Loading

  • Made some refactoring in order to simplify test setups
  • Add workflow to run tests, if any chains.json were changed
  • Publish test results to GitHub action summary with comment about tests.

@novasamatech novasamatech deleted a comment from github-actions bot Nov 7, 2023
@novasamatech novasamatech deleted a comment from github-actions bot Nov 7, 2023
@novasamatech novasamatech deleted a comment from github-actions bot Nov 7, 2023
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Nov 7, 2023 •
edited
Loading

🧪 Test results
https://github.com/novasamatech/nova-utils/actions/runs/6784300797/attempts/1#summary-18440335730

@stepanLav stepanLav changed the title first implementation Integration tests for prod config Nov 8, 2023
leohar
leohar reviewed Nov 9, 2023
View reviewed changes
chains/v15/chains.json Outdated Show resolved Hide resolved
leohar
leohar requested changes Nov 9, 2023
View reviewed changes
chains/v15/chains_dev.json Outdated Show resolved Hide resolved
@stepanLav stepanLav requested a review from leohar November 10, 2023 09:25
@stepanLav stepanLav merged commit 882d645 into master Nov 13, 2023
1 check passed
@stepanLav stepanLav deleted the feat/run_tests_on_pr branch November 13, 2023 08:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@leohar leohar leohar approved these changes

@pgolovkin pgolovkin pgolovkin approved these changes

@ERussel ERussel Awaiting requested review from ERussel

@lynx56 lynx56 Awaiting requested review from lynx56

@valentunn valentunn Awaiting requested review from valentunn

@antonijzelinskij antonijzelinskij Awaiting requested review from antonijzelinskij

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@stepanLav @pgolovkin @leohar