Fix `rbt status` returning blank summary for draft review requests
Review Request #11174 — Created Sept. 18, 2020 and submitted — Latest diff uploaded
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.
- 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 | | +--------+-----------------------------------------------------+---------------+
- Ran the unit tests using
./tests/runtests.py
, all passed.