tree 6960a1bda3226bd29a9fbfc046613c252d06a2cc
parent 83da3b1e25af5c85e0a1a75a9a5f91a3e1755849
author Gal Paikin <paiking@google.com> 1599554315 +0300
committer Gal Paikin <paiking@google.com> 1600186712 +0300

Do not allow creations of comments with invalid inReplyTo

We used to throw IllegalStateException until I06db2d8f removed
them.

Instead of returning the IllegalStateException, this change added
BadRequestException in all places that can add inReplyTo to comments
or drafts.

Also fixed a test that allowed invalid inReplyTo when updating drafts.

Also added a way to retrieve tags and unresolved from comments to the
test API, since they are used here.

As a follow-up change, the logic in PostReview, CreateDraftComment
and PutDraftComment should actually be the same logic.

Change-Id: I343425abd918a260f3fc71d5c26d18c42297465a