Remove the TypeCollectionHelper module

This module provided only a single method, known_resource_types which
just delegated to environment. This served almost no purpose except to
confuse anyone searching for where known_resource_types came from (and
they would then have to figure out where environment came from). Between
the require and include, it's not actually reducing code duplication,
and it's misleading about where that data is coming from and who it
belongs to.