Skip to content

refactor: run tests with own udp tracker#98

Merged
josecelano merged 2 commits intoe2e-tests-for-udp-trackerfrom
mick/e2e-tests-for-udp-tracker
Oct 7, 2022
Merged

refactor: run tests with own udp tracker#98
josecelano merged 2 commits intoe2e-tests-for-udp-trackerfrom
mick/e2e-tests-for-udp-tracker

Conversation

@mickvandijke
Copy link
Copy Markdown
Member

All tests will start their own udp tracker on an ephemeral port instead of using a singleton tracker.

@josecelano josecelano merged commit 91250f6 into e2e-tests-for-udp-tracker Oct 7, 2022
@mickvandijke mickvandijke deleted the mick/e2e-tests-for-udp-tracker branch October 7, 2022 17:53
josecelano pushed a commit that referenced this pull request Oct 19, 2022
* refactor: run tests with own udp tracker

* fixup! refactor: run tests with own udp tracker
josecelano pushed a commit that referenced this pull request Oct 19, 2022
* refactor: run tests with own udp tracker

* fixup! refactor: run tests with own udp tracker
josecelano pushed a commit that referenced this pull request Oct 21, 2022
* refactor: run tests with own udp tracker

* fixup! refactor: run tests with own udp tracker
josecelano added a commit that referenced this pull request Oct 24, 2022
5dcea43 fix: initialize loggin once (Jose Celano)
d2c69fa fix: test tear up (Jose Celano)
508803a refactor: extract function (Jose Celano)
1f258c1 test: integration test for udp scrape request (Jose Celano)
f5aee03 fix: re-format with rust nightly (Jose Celano)
946e80a refactor: run tests with own udp tracker (#98) (Mick van Dijke)
a27adf2 test: integration tests for udp tracker (Jose Celano)

Pull request description:

  I'm trying to reproduce the error in issue #42. I've only created the scaffolding for integration tests so far.

  If there is no bug I want to use this scaffolding for other integration tests. If we consider them helpful.

ACKs for top commit:
  da2ce7:
    ACK 5dcea43

Tree-SHA512: 1a939e0ac18d8c8261747a21c934dd55911a4b6bab0620d370337f599ea4e978783a4b9bef285970396f0ed06c101aa9b1d07cf39f92b3a3417329158c3ee5c1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants