Skip to content

fix(tests): params cleanup in pow_tests/get_next_work#5067

Merged
PastaPastaPasta merged 1 commit into
dashpay:developfrom
UdjinM6:fix_test_params
Nov 2, 2022
Merged

fix(tests): params cleanup in pow_tests/get_next_work#5067
PastaPastaPasta merged 1 commit into
dashpay:developfrom
UdjinM6:fix_test_params

Conversation

@UdjinM6
Copy link
Copy Markdown

@UdjinM6 UdjinM6 commented Oct 30, 2022

Issue being fixed or feature implemented

Avoid leaking devnet param into other tests.

https://gitlab.com/UdjinM6/dash/-/jobs/3221459032#L3035: note lots of Error: -port must be specified when -devnet and -listen are specified

What was done?

How Has This Been Tested?

0aa5dc0251 https://gitlab.com/UdjinM6/dash/-/jobs/3247713299

Breaking Changes

none

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

For repository code-owners and collaborators only

  • I have assigned this pull request to a milestone

@UdjinM6 UdjinM6 added this to the 18.2 milestone Oct 30, 2022
Copy link
Copy Markdown
Member

@PastaPastaPasta PastaPastaPasta left a comment

Choose a reason for hiding this comment

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

utACK for squash merge

@PastaPastaPasta PastaPastaPasta changed the title fix: params cleanup in pow_tests/get_next_work fix(tests): params cleanup in pow_tests/get_next_work Nov 2, 2022
@PastaPastaPasta PastaPastaPasta merged commit 083c860 into dashpay:develop Nov 2, 2022
@UdjinM6 UdjinM6 deleted the fix_test_params branch November 2, 2022 16:53
gades pushed a commit to cosanta/cosanta-core that referenced this pull request Jul 13, 2024
Sign up for free to 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.

2 participants