Skip to content

test: Fix dynamically_smth_masternode helpers, extend feature_dip3_v19.py - #5402

Merged
UdjinM6 merged 2 commits into
dashpay:developfrom
UdjinM6:fix_test_v19
May 31, 2023
Merged

test: Fix dynamically_smth_masternode helpers, extend feature_dip3_v19.py#5402
UdjinM6 merged 2 commits into
dashpay:developfrom
UdjinM6:fix_test_v19

Conversation

@UdjinM6

Copy link
Copy Markdown

Issue being fixed or feature implemented

fix a couple of issues in helpers, extend feature_dip3_v19.py to check more after v19 fork

What was done?

pls see individual PRs

How Has This Been Tested?

run tests

Breaking Changes

n/a

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have added or updated relevant unit/integration/functional/e2e tests
  • I have made corresponding changes to the documentation
  • I have assigned this pull request to a milestone (for repository code-owners and collaborators only)

@UdjinM6UdjinM6 added this to the 20 milestone May 30, 2023
@UdjinM6UdjinM6 mentioned this pull request May 30, 2023
5 tasks
@UdjinM6UdjinM6 modified the milestones: 20, 19.2May 30, 2023

@ogabrielidesogabrielides left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

utACK

@PastaPastaPastaPastaPastaPasta left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

utACK for squash merge.

@UdjinM6
UdjinM6 merged commit b741f3a into dashpay:developMay 31, 2023
@UdjinM6
UdjinM6 deleted the fix_test_v19 branch May 31, 2023 20:34
UdjinM6 added a commit that referenced this pull request Jun 4, 2023
## Issue being fixed or feature implemented
same as #5392, alternative solution
~based on #5402 atm, will rebase later~
## What was done?
pls see individual commits
## How Has This Been Tested?
reorg mainnet around forkpoint with a patched client (to allow low
difficulty), run tests
## Breaking Changes
Another evodb migration is required. Going back to an older version or
migrating after the fork requires reindexing.
## Checklist:
- [x] I have performed a self-review of my own code
- [x] I have commented my code, particularly in hard-to-understand areas
- [ ] I have added or updated relevant unit/integration/functional/e2e
tests
- [ ] I have made corresponding changes to the documentation
- [x] I have assigned this pull request to a milestone _(for repository
code-owners and collaborators only)_
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Jun 11, 2023
…9.py (dashpay#5402)
## Issue being fixed or feature implemented
fix a couple of issues in helpers, extend feature_dip3_v19.py to check
more after v19 fork
## What was done?
pls see individual PRs
## How Has This Been Tested?
run tests
## Breaking Changes
n/a
## Checklist:
- [x] I have performed a self-review of my own code
- [x] I have commented my code, particularly in hard-to-understand areas
- [ ] I have added or updated relevant unit/integration/functional/e2e
tests
- [ ] I have made corresponding changes to the documentation
- [x] I have assigned this pull request to a milestone _(for repository
code-owners and collaborators only)_
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Jun 11, 2023
same as dashpay#5392, alternative solution
~based on dashpay#5402 atm, will rebase later~
pls see individual commits
reorg mainnet around forkpoint with a patched client (to allow low
difficulty), run tests
Another evodb migration is required. Going back to an older version or
migrating after the fork requires reindexing.
- [x] I have performed a self-review of my own code
- [x] I have commented my code, particularly in hard-to-understand areas
- [ ] I have added or updated relevant unit/integration/functional/e2e
tests
- [ ] I have made corresponding changes to the documentation
- [x] I have assigned this pull request to a milestone _(for repository
code-owners and collaborators only)_
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@UdjinM6@ogabrielides@PastaPastaPasta