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.

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

Ticket31304 #1195

Merged
merged 6 commits into from Aug 6, 2019
Merged

Ticket31304 #1195

merged 6 commits into from Aug 6, 2019

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@nmathewson @coveralls @torproject-pusher
@nmathewson
Copy link
Contributor

@nmathewson nmathewson commented Aug 1, 2019

No description provided.

nmathewson added 5 commits Aug 1, 2019
@nmathewson
make dist: only include files from practracker dir intentionally.
3945282 
Previously, we included temporary files and whatnot, which is not
good.

Fixes bug 31311; bugfix on 0.4.1.1-alpha.
@nmathewson
Merge branch 'ticket31311_041' into ticket31304
49c696f
@nmathewson
Distribute practracker unit and integration tests.
30da1b6
@nmathewson
Port practracker unit tests to python 3
5d98b54
@nmathewson
practracker: Add unit tests to test script, and test script to makefile
fa60fee 
This makes all of the practracker tests get run by make check, and
hence by our CI.

Closes ticket 31304.
@coveralls
Copy link

@coveralls coveralls commented Aug 1, 2019 •
edited

Pull Request Test Coverage Report for Build 5673

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 62.814%
Totals Coverage Status
Change from base Build 5667: 0.0%
Covered Lines: 47183
Relevant Lines: 75116
💛 - Coveralls

@nmathewson
Adjust test_practracker.sh to work on windows
1440c2c 
The required change is to ignore trailing CRs when diffing files.
@torproject-pusher torproject-pusher merged commit 1440c2c into torproject:master Aug 6, 2019
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment