#81 Testing TorManager + LoadProfile bug fix

Merged
sarah merged 1 commits from tor into master 2 years ago
sarah commented 2 years ago
There is no content yet.
dan commented 2 years ago
Owner

how come you moved the torrc logic to app? seems like it would be likewise useful to server?

how come you moved the torrc logic to app? seems like it would be likewise useful to server?
sarah commented 2 years ago
Owner

Because I didn’t want the unit test to cobble the user version of the cwtch profile. Servers should have their own config settings separate from users, so it made sense to keep them apart.

Because I didn't want the unit test to cobble the user version of the cwtch profile. Servers should have their own config settings separate from users, so it made sense to keep them apart.
dan commented 2 years ago
Owner

Add to tests.sh and then merge!

  • adding to my TODO to make tests.sh more automatic
Add to tests.sh and then merge! - adding to my TODO to make tests.sh more automatic
sarah closed this pull request 2 years ago
The pull request has been merged as f435de4b71.
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.