Merge branch 'weierophinney-hotfix/2821-loadmodule' into develop

src/ModuleManager.php

@@ -148,8 +148,8 @@ public function loadModule($moduleName)
         }
         $event->setModule($module);
 
-        $this->getEventManager()->trigger(ModuleEvent::EVENT_LOAD_MODULE, $this, $event);
         $this->loadedModules[$moduleName] = $module;
+        $this->getEventManager()->trigger(ModuleEvent::EVENT_LOAD_MODULE, $this, $event);
 
         $this->loadFinished = true;
 

test/ModuleManagerTest.php

@@ -11,10 +11,12 @@
 namespace ZendTest\ModuleManager;
 
 use PHPUnit_Framework_TestCase as TestCase;
+use stdClass;
 use Zend\EventManager\EventManager;
 use Zend\Loader\AutoloaderFactory;
 use Zend\ModuleManager\Listener\ListenerOptions;
 use Zend\ModuleManager\Listener\DefaultListenerAggregate;
+use Zend\ModuleManager\ModuleEvent;
 use Zend\ModuleManager\ModuleManager;
 use InvalidArgumentException;
 
@@ -142,4 +144,21 @@ public function testCanLoadModuleDuringTheLoadModuleEvent()
         $this->assertTrue(isset($config['loaded']));
         $this->assertSame('oh, yeah baby!', $config['loaded']);
     }
+
+    public function testModuleIsMarkedAsLoadedWhenLoadModuleEventIsTriggered()
+    {
+        $test          = new stdClass;
+        $moduleManager = new ModuleManager(array('BarModule'));
+        $events        = $moduleManager->getEventManager();
+        $events->attachAggregate($this->defaultListeners);
+        $events->attach(ModuleEvent::EVENT_LOAD_MODULE, function ($e) use ($test) {
+            $test->modules = $e->getTarget()->getLoadedModules(false);
+        });
+
+        $moduleManager->loadModules();
+
+        $this->assertTrue(isset($test->modules));
+        $this->assertArrayHasKey('BarModule', $test->modules);
+        $this->assertInstanceOf('BarModule\Module', $test->modules['BarModule']);
+    }
 }