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

Ticket31637: integration tests for configuration parsing #1324

Merged
merged 9 commits into from Sep 16, 2019
Merged

Ticket31637: integration tests for configuration parsing #1324

merged 9 commits into from Sep 16, 2019

Conversation

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

@nmathewson nmathewson commented Sep 14, 2019

No description provided.

nmathewson added 9 commits Sep 14, 2019
@nmathewson
Add a test script to try parsing and encoding Tor configurations
7c7e840 
This script takes a set of example torrcs and command-lines from
src/test/conf_examples.  If a success is expected, it runs "tor
--dump-config" and compares the result with the one we expect.  If a
failure is expected, it runs "tor --verify-config" and greps for the
error we expect.
@nmathewson
Move our first 3 example conf_examples to named directories
2f7be96 
I'm not planning to use "example" as the name for all of them, but
these first three _are_ simple examples.
@nmathewson
Add test for +Option, /Option, and Option
850a00d 
Here we use these directives to replace, extend, or clear values in
torrc.defaults and in torrc.
@nmathewson
Add a changes file for ticket 31637.
98ab381
@nmathewson
test_parseconf: run each test from inside its directory.
2802569 
We need this to test includes and relative paths.
@nmathewson
Add an integration test for %include.
80e858e
@nmathewson
Add a few error test cases
617679d
@nmathewson
Add a test with many non-default options.
7e9ee3a
@nmathewson
test_parseconf.sh: On --dump-config failure, --verify-config
11cf4d9
@nmathewson nmathewson mentioned this pull request Sep 14, 2019
Closed
@torproject-pusher torproject-pusher merged commit 11cf4d9 into torproject:master Sep 16, 2019
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment