• 
      

    Fix trivial publishing and add a unit test for the backend.

    Review Request #7336 — Created May 26, 2015 and submitted

    Information

    Review Board
    release-2.5.x
    a59c6ee...

    Reviewers

    This change fixes up the plumbing of the "trivial" option, and adds a new unit
    test to verify that publishing a draft with ?trivial=1 does not send an e-mail.

    • Configured e-mail with the console backend and checked that publishing with
      the box both checked and un-checked did the correct thing.
    • Turned off e-mail and saw that the check-box no longer appeared, and that
      publishing still worked.
    • Ran unit tests.
    Description From Last Updated

    This seems better as an assertion, since it shouldn't happen.

    chipx86chipx86
    reviewbot
    1. Tool: Pyflakes
      Processed Files:
          reviewboard/webapi/tests/test_review_request_draft.py
          reviewboard/reviews/models/review_request_draft.py
          reviewboard/reviews/models/review_request.py
      
      Ignored Files:
          reviewboard/static/rb/js/models/reviewRequestEditorModel.js
          reviewboard/static/rb/js/views/reviewRequestEditorView.js
      
      
      
      Tool: PEP8 Style Checker
      Processed Files:
          reviewboard/webapi/tests/test_review_request_draft.py
          reviewboard/reviews/models/review_request_draft.py
          reviewboard/reviews/models/review_request.py
      
      Ignored Files:
          reviewboard/static/rb/js/models/reviewRequestEditorModel.js
          reviewboard/static/rb/js/views/reviewRequestEditorView.js
      
      
    2. 
        
    chipx86
    1. 
        
    2. Show all issues

      This seems better as an assertion, since it shouldn't happen.

      1. Actually not true, since the check-box doesn't exist when e-mails are turned off on the server.

      2. Ohh, I didn't realize that. I thought it was always present but maybe hidden.

    3. Is there an issue with this being anything but true or false?

      1. No. I can change it to falsy.

    4. 
        
    david
    reviewbot
    1. Tool: Pyflakes
      Processed Files:
          reviewboard/webapi/tests/test_review_request_draft.py
          reviewboard/reviews/models/review_request_draft.py
          reviewboard/reviews/models/review_request.py
      
      Ignored Files:
          reviewboard/static/rb/js/models/reviewRequestEditorModel.js
          reviewboard/static/rb/js/views/reviewRequestEditorView.js
      
      
      
      Tool: PEP8 Style Checker
      Processed Files:
          reviewboard/webapi/tests/test_review_request_draft.py
          reviewboard/reviews/models/review_request_draft.py
          reviewboard/reviews/models/review_request.py
      
      Ignored Files:
          reviewboard/static/rb/js/models/reviewRequestEditorModel.js
          reviewboard/static/rb/js/views/reviewRequestEditorView.js
      
      
    2. 
        
    chipx86
    1. Ship It!
    2. 
        
    david
    Review request changed
    Status:
    Completed
    Change Summary:
    Pushed to release-2.5.x (4be7c7a)