removed solved fixme's

diff --git a/t/lib/TestAppCustomContainer/NoSugarContainer.pm b/t/lib/TestAppCustomContainer/NoSugarContainer.pm
index 13721b8..b8dd626 100644 (file)
--- a/t/lib/TestAppCustomContainer/NoSugarContainer.pm
+++ b/t/lib/TestAppCustomContainer/NoSugarContainer.pm
@@ -39,10 +39,6 @@ sub BUILD {
         Catalyst::IOC::ConstructorInjection->new(
             name         => 'model_Baz',
             class        => 'TestAppCustomContainer::Model::Baz',
-# t0m - I'm fine with this - sugar can just s/Request/+Catalyst::IOC::LifeCycle::Request/
-#       Also, 'Request' is fine as a name for a lifecycle IMO.
-# FIXME - it should simply be Request (or InstancePerRequest, etc)
-# see Bread/Board/Service.pm line 47
             lifecycle    => '+Catalyst::IOC::LifeCycle::Request',
             dependencies => [
                 depends_on( '/application_name' ),
@@ -70,10 +66,6 @@ sub BUILD {
     $self->get_sub_container('model')->add_service(
         Catalyst::IOC::BlockInjection->new(
             name         => 'Quux',
-
-# FIXME - it should probably be our
-# Catalyst::IOC::LifeCycle::Singleton
-# t0m - I think normal Singleton is fine here, it's per app lifetime.
             lifecycle    => 'Singleton',
             dependencies => [
                 depends_on( '/component/model_Quux' ),