• 
      

    Fix `rbt status` returning blank summary for draft review requests

    Review Request #11174 — Created Sept. 18, 2020 and submitted — Latest diff uploaded

    Information

    RBTools
    release-1.0.x
    8f635ea...

    Reviewers

    When displaying the summary for the review request, the code attempts to get the
    summary from request.summary, which is null for draft reviews. In this
    code change, we added a condition on where to get the summary - if the
    draft exists, we get the draft summary; otherwise we get it from the
    published review request.

    The issue was fixed by the code change.

    1. Reproduced issue in 4855, confirmed the summary for a draft review shows as expected after the code change
    +--------+-----------------------------------------------------+---------------+
    | Status |                   Review Request                    |    Branch     |
    +========+=====================================================+===============+
    | Draft  | r/11174 - fix 4855:rbtools 1.0.2 returns blank      | release-1.0.x |
    |        | summary for draft review requests                   |               |
    +--------+-----------------------------------------------------+---------------+
    
    1. Ran the unit tests using ./tests/runtests.py, all passed.