Commit ae4d8c9c authored by Mike Perry's avatar Mike Perry Committed by Nick Mathewson
Browse files

Fix a breakage in test_options.c. 

IMO, these tests should be calling options_init() to properly set everything
to default values, but when that is done, about a dozen tests fail. Setting
the one default value that broke the tests for my branch. Sorry for being
lame.
parent 20a3d4ef

src/test/test_options.c

+6 −0
Original line number Diff line number Diff line
@@ -354,6 +354,12 @@ get_options_test_data(const char *conf) 
  result->opt = options_new();

  result->old_opt = options_new();

  result->def_opt = options_new();



  // XXX: Really, all of these options should be set to defaults

  // with options_init(), but about a dozen tests break when I do that.

  // Being kinda lame and just fixing the immedate breakage for now..

  result->opt->ConnectionPadding = -1; // default must be "auto"



  rv = config_get_lines(conf, &cl, 1);

  tt_assert(rv == 0);

  rv = config_assign(&options_format, result->opt, cl, 0, &msg);