Fix TestCase.create_webhook to set events as a list.

Review Request #8712 — Created Feb. 3, 2017 and submitted — Latest diff uploaded

Information

Review Board
release-3.0.x
41caa5a...

Reviewers

django-multiselectfield will do some comma-splitting when it loads a value from
the db, but if you set the value in the field to a string and immediately read
it back without committing to the db, it will return a string rather than a
list. In this case, webhooks that were created by a testcase running under
Django 1.8 (which dumps each test case into an atomic block) were failing
because the value read back was '*' instead of ['*'].

Ran unit tests on Django 1.6 and 1.8.

    Loading...