On Thu, Oct 19, 2017, at 03:19 PM, Mark Banner wrote: > The only thing that might help (that has been discussed) is something > along the lines of flow - an analyser that could work out that 'spice()' > didn't exist, but Dave Townsend mentioned these don't seem to be viable > for us. > > Therefore I think the migration code should be having automated tests - > it is the only way we can reliably detect these sort of failures at the > moment.
That's fair. Note that we do have active code coverage results now, which shows me that the line containing the bug that started this whole thread still isn't covered by tests: https://codecov.io/gh/marco-c/gecko-dev/src/master/browser/components/customizableui/CustomizableUI.jsm#L461 -Ted _______________________________________________ dev-platform mailing list dev-platform@lists.mozilla.org https://lists.mozilla.org/listinfo/dev-platform