Code review comment for lp://qastaging/~jamesh/unity-scope-scopes/fake-result-categories

Looks good to me. In general, I'm not sure that categories should be part of the metadata for a scope though. For some scopes, I imagine that the categories will actually determined by whatever data source the scope uses, so fixing the categories at scope installation time seems wrong?