Montior no longer called for LC methods

Paul Hammant added a comment - 13/Oct/05 8:54 PM Fixed as of revision 2650. New test method DefaultPicoContainerTestCase.testStartCapturedByMonitor()

Jörg Schaible added a comment - 14/Oct/05 3:12 PM Close, but no cigar: public void testCanReallyChangeMonitor() throws SecurityException, NoSuchMethodException { Method start = Startable.class.getMethod("start", null); Method stop = Startable.class.getMethod("stop", null); Mock mockMonitor1 = mock(ComponentMonitor.class, "Monitor1"); Mock mockMonitor2 = mock(ComponentMonitor.class, "Monitor2"); DefaultPicoContainer pico = new DefaultPicoContainer((ComponentMonitor)mockMonitor1.proxy()); pico.registerComponentImplementation(MyStartable.class); mockMonitor1.expects(once()).method("instantiating"); mockMonitor1.expects(once()).method("instantiated"); mockMonitor1.expects(once()).method("invoking").with(eq(start), ANYTHING); mockMonitor1.expects(once()).method("invoked").with(eq(start), ANYTHING, ANYTHING); mockMonitor1.expects(once()).method("invoking").with(eq(stop), ANYTHING); mockMonitor1.expects(once()).method("invoked").with(eq(stop), ANYTHING, ANYTHING); pico.start(); pico.stop(); Startable startable = (Startable)pico.getComponentInstance(MyStartable.class); assertNotNull(startable); pico.changeMonitor((ComponentMonitor)mockMonitor2.proxy()); mockMonitor2.expects(once()).method("invoking").with(eq(start), ANYTHING); mockMonitor2.expects(once()).method("invoked").with(eq(start), ANYTHING, ANYTHING); mockMonitor2.expects(once()).method("invoking").with(eq(stop), ANYTHING); mockMonitor2.expects(once()).method("invoked").with(eq(stop), ANYTHING, ANYTHING); pico.start(); pico.stop(); }

Paul Hammant added a comment - 16/Oct/05 8:25 AM Fixed again. There were a number of things that were done twice (under debugging) re the changeMonitor and instantiate of default DefaultLifecycleStrategy that have been eliminated with the latest commit (2654). Things are more logical now; more CDI.