(REF) Move testGitLabIssue1108() to its own class
BEFORE: `MailingSystemTest` (which extends `BaseMailingSystemTest`)
includes `testGitLabIssue1108($isMultilingual)`. This is
a transactional test (per `BaseMailingSystemTest::setUp()`).
AFTER: `MultingualSystemTest` includes `testGitLabIssue1108()`.
This is not a transactional test.
COMMENTS: This resolves some circumstantial flakiness in the tests.
* The problem appeared when adding an unrelated test `BaseMailingSystemTest` -- the
new test failed because of a conflict with `testGitLabIssue1108()`.
* You could also produce the problem in other cases by switching around the order of
`testGitLabIssue1108()` (e.g. hack data-provider `multiLingual()`)
* I believe the root problem is that `BaseMailingSystemTest` is written as
a transactional test -- but `testGitLabIssue1108()` does large-scale
schema changes (whenever it toggles multilingual), which makes it
non-transactional.
* The patch here prevents these kind of conflicts by putting
`testGitLabIssue1108()` in a separate (non-transactional) context.