2020752 - [PHABBUGZ] Improve testing of phabricator and bmo integration

Status: RESOLVED FIXED

(bugzilla.mozilla.org :: Continous Integration, enhancement, P2)

Description

[David Lawrence [:dkl]]

With bug 2015155, a new flag was added that phabbugz uses to let Bugzilla know it is OK to use text/x-phabricator-request as the content type for the new attachment. We should add a test for this new behavior and also look for other ways to expand integration testing so that manual testing will not be as needed.

Comment 1

[Hackbot]

Automated triage: no action taken. This bug requires changes to the BMO codebase (PhabBugz extension + CI integration tests), but the source tree available to this bot is mozilla-central, so there is nothing here to patch or verify. Needs manual attention from a BMO developer.

If you'd like to provide feedback on this comment, please use the 👍 or 👎 reaction.

Comment 2

[David Lawrence [:dkl]]

Come up with a robust test suite to test the integration of Bugzilla and Phabricator

• All testing scripts should live under extensions/PhabBugz
• All API calls to Phabricator should be mocked and not require an actual Phabricator server to operate. Refer to the Conduit documentation in Phabricator for exact data structures.
• Each of the modules Revision, Project, Policy, User, etc. should have test cases to exercise the different operations of each.
• Test cases need to also test the periodic checking for new stories, users, and group syncing and the operations performed when new items are present.
• Different failure conditions of Phabricator such as network issues, next ID or cursor does not exist, etc.

Comment 3

[BMO Github Automation]

Attachment: [mozilla/bmo] Bug 2020752 - [PHABBUGZ] Improve testing of phabricator and bmo integration (#2790)

(via GitHub webhook, authenticated as shared-secret)

Comment 4

[BMO Github Automation]

Authored by https://github.com/dklawren
https://github.com/mozilla/bmo/commit/a45200d8ca48c07da34642ae60d3442e6192cab1
[master] Bug 2020752 - [PHABBUGZ] Improve testing of phabricator and bmo integration

(via GitHub webhook, authenticated as shared-secret)