[00:13:00] <cscott>	 made a suggestions
[00:13:30] <cscott>	 it's a false positive -- or rather, the information that $pageBundle is non-null is outside the scope of phan's knowledge.
[00:13:44] <cscott>	 But we can assert non-null rather than suppress the warning, which should keep it from being flakey at least.
[00:25:01] <Krinkle>	 cscott: afaik tests proper (as in, not base classes and utils in tests dir, listed in the tests-specific autoloader) don't require either a namespace or auto loader. PHPUnit traverses the directory and runs them.
[00:26:13] <Krinkle>	 What's the value on coming up with a namespace for all of them?
[00:28:16] <Krinkle>	 I've been cleaning up a number of boundaries and directories as part of T364652 as well
[00:28:16] <stashbot>	 T364652: Make component boundaries in MediaWiki core easier to discover - https://phabricator.wikimedia.org/T364652