Improve guess support in rbt post.

Review Request #5614 — Created March 12, 2014 and submitted — Latest diff uploaded

Information

RBTools
master
ee401fd...

Reviewers

This adds new abilities to rbt post's guess fields support. The old
guess support was a boolean state, which only supported opt-in and not
opt-out. That is, if GUESS_FIELDS was set to True in .reviewboardrc, it
couldn't be turned off when posting.

It's now a tri-state of "yes", "no", and "auto", which can be set on the
command line or .reviewboardrc.

If "yes", it behaves as before, with the fields being updated from the
commit message.

If "no", guessing will be turned off (which allows the command line to
override the config file).

If "auto", guessing will be turned on only for new review requests, but
turned off when updating.

If no --guess-* options or configuration variables are set, the
default will be "auto", making for a nicer experience when posting brand
new review requests.

If an old-style --guess-* option is passed without a value (in the form
of --guess-*=value or -gvalue), then the prior logic of forcing guessing
will be used. That is, "yes" is implied.

While "yes", "no", and "auto" are all valid values, we still support
True and False in .reviewboardrc, for backwards-compatibility.

Posted this change by doing rbt post, without any parameters.

Temporarily turned off actual publishing of the change and just outputted
the results of the guess flags. Then I did tests with all three values
in .reviewboardrc and with/without command line arguments and their possible
values. Saw the resulting values that I expected.

Turned posting back on, removed the settings from .reviewboardrc, and posted
a change without explicitly passing any guess arguments. The review request
had the fields filled in from my commit message.

Made changes to the description and summary on my review request, and updated
using -r. The changes weren't overwritten. Tested also with -u.

Then I tried with --guess-fields=no, and it didn't overwrite either.

Tried with yes and it did overwrite.

Tried again without a value and it overwrote again.

Posted a new change with --guess-fields=no and it didn't provide any defaults.

Unit tests pass.

Built the docs, read through, and verified all links.

    Loading...