Skip to content

Test address generator#1895

Closed
g1itch wants to merge 1 commit intoBitmessage:v0.6from
g1itch:addressgenerator
Closed

Test address generator#1895
g1itch wants to merge 1 commit intoBitmessage:v0.6from
g1itch:addressgenerator

Conversation

@g1itch
Copy link
Copy Markdown
Collaborator

@g1itch g1itch commented Dec 6, 2021

Hello!

I failed to start a test case for singleWorker and decided to begin with something easier. This test succeeds on py27, but for python3 I should probably mix this with an excerpt of my network branch.

@g1itch g1itch added the test label Dec 6, 2021
@g1itch g1itch force-pushed the addressgenerator branch 2 times, most recently from f184db9 to 80d3c3f Compare December 9, 2021 15:46
Copy link
Copy Markdown
Member

@PeterSurda PeterSurda left a comment

Choose a reason for hiding this comment

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

I like it, an interesting workaround for the problems with the current design.

@g1itch
Copy link
Copy Markdown
Collaborator Author

g1itch commented Dec 15, 2021

I like it, an interesting workaround for the problems with the current design.

It is mostly the network-unbundling. Only the last commit is from this PR.

@g1itch
Copy link
Copy Markdown
Collaborator Author

g1itch commented Dec 18, 2021

BTW, I cannot reproduce the buildbot/travis_bionic_python_2.7 failure locally. I have no clue why it happens.

@PeterSurda
Copy link
Copy Markdown
Member

BTW, I cannot reproduce the buildbot/travis_bionic_python_2.7 failure locally. I have no clue why it happens.

Yea that looks weird. I think it happens because you're importing pathmagic and that screws up from pybitmessage.randomtrackingdict import RandomTrackingDict.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants