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

feat(BUX-572): regression tests #227

Merged
merged 28 commits into from
Apr 3, 2024
Merged

feat(BUX-572): regression tests #227

merged 28 commits into from
Apr 3, 2024

Conversation

wregulski
Copy link
Contributor

@wregulski wregulski commented Mar 20, 2024 •
edited
Loading

Pull Request Checklist

  • 📖 I created my PR using provided : CODE_STANDARDS
  • 📖 I have read the short Code of Conduct: CODE_OF_CONDUCT
  • 🏠 I tested my changes locally.
  • ✅ I have provided tests for my changes.
  • 📝 I have used conventional commits.
  • 📗 I have updated any related documentation.
  • 💾 PR was issued based on the Github or Jira issue.

@wregulski wregulski self-assigned this Mar 20, 2024
@wregulski wregulski requested a review from a team as a code owner March 20, 2024 19:30
@github-actions GitHub Actions
Copy link
Contributor

Manual Tests

ℹ️ Remember to ask team members to perform manual tests and to assign tested label after testing.

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Mar 20, 2024 •
edited
Loading

Manual Tests

💚 Manual testing by @chris-4chain resulted in success.

chris-4chain
chris-4chain reviewed Mar 21, 2024
View reviewed changes
config/defaults.go Show resolved Hide resolved
regressiontests/regression_test.go Outdated Show resolved Hide resolved
@wregulski wregulski force-pushed the feat/572-regression-tests branch 4 times, most recently from 92acea6 to afa311c Compare March 25, 2024 09:29
chris-4chain
chris-4chain reviewed Mar 27, 2024
View reviewed changes
regressiontests/fixtures.go Outdated Show resolved Hide resolved
regressiontests/regression_test.go Outdated Show resolved Hide resolved
regressiontests/regression_test.go Outdated Show resolved Hide resolved
regressiontests/regression_test.go Outdated Show resolved Hide resolved
regressiontests/sqlite.config.yaml Outdated Show resolved Hide resolved
@wregulski wregulski changed the title !DRAFT! feat(BUX-572): regression tests feat(BUX-572): regression tests Mar 29, 2024
@wregulski wregulski requested a review from a team March 29, 2024 11:03
chris-4chain
chris-4chain reviewed Mar 29, 2024
View reviewed changes
README.md Show resolved Hide resolved
regressiontests/regression_test.go Show resolved Hide resolved
wregulski and others added 14 commits April 2, 2024 13:08
@wregulski
feat(BUX-572): working attach and sync process with postgres database
e9157f1
@wregulski
feat: compare woc tip of chain with local tip of chain
3e7b870
@wregulski
feat(BUX-572): no-sec for default token variable
b0b26ed
@wregulski
feat(BUX-572): add all required assertions
99a76db
@wregulski
feat(BUX-562): add sqlite support on a flag
030294b
@wregulski
feat(BUX-562): replace process kill with signal sigterm
40f710f
@wregulski
fix(BUX-562): adjusts http port to 8080
f4c7e2f
@github-actions @wregulski
chore: update generated files
64cbfd2
@dorzepowski @wregulski
fix(BUX-667): display application version in swagger.
7d50526
@wregulski
feat(BUX-572): working solution with killable process
de70669
@wregulski
feat(BUX-572): add more active listening for sync process
1d407a4
@wregulski
fix(BUX-572): remove unnecessary sleep while starting testcontainer
fbf8051
@wregulski
fix(BUX-572): remove underscore from merkleroot fixture name
a5905a0
@wregulski
chore(BUX-572): move reusable part of http server to const
c43a0b6
dorzepowski
dorzepowski requested changes Apr 2, 2024
View reviewed changes
Copy link
Contributor

@dorzepowski dorzepowski left a comment

Choose a reason for hiding this comment

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

Very good work ❤️

regressiontests/regression_test.go Outdated Show resolved Hide resolved
regressiontests/regression_test.go Show resolved Hide resolved
.github/workflows/regression-tests.yml Outdated Show resolved Hide resolved
.github/workflows/regression-tests.yml Outdated Show resolved Hide resolved
regressiontests/regression_test.go Show resolved Hide resolved
@chris-4chain
Copy link
Contributor

testing 🧪

@chris-4chain chris-4chain added the tested PR was tested by a team member label Apr 3, 2024
@wregulski wregulski merged commit e3e4a35 into master Apr 3, 2024
9 checks passed
@wregulski wregulski deleted the feat/572-regression-tests branch April 3, 2024 11:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chris-4chain chris-4chain chris-4chain approved these changes

@dorzepowski dorzepowski dorzepowski approved these changes

Assignees

@wregulski wregulski

Labels
tested PR was tested by a team member
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@wregulski @chris-4chain @dorzepowski