Tests

1 files changed, 18 insertions, 3 deletions

diff --git a/tests/app/ModuleTest.php b/tests/app/ModuleTest.php
index 73ef4f1a2f..649be3c092 100644
--- a/tests/app/ModuleTest.php
+++ b/tests/app/ModuleTest.php
@@ -17,13 +17,16 @@ declare(strict_types=1);
 
 namespace Fisharebest\Webtrees;
 
+use Fisharebest\Webtrees\Module\ModuleAnalyticsInterface;
 use Fisharebest\Webtrees\Module\ModuleBlockInterface;
 use Fisharebest\Webtrees\Module\ModuleChartInterface;
 use Fisharebest\Webtrees\Module\ModuleConfigInterface;
 use Fisharebest\Webtrees\Module\ModuleHistoricEventsInterface;
+use Fisharebest\Webtrees\Module\ModuleInterface;
 use Fisharebest\Webtrees\Module\ModuleMenuInterface;
 use Fisharebest\Webtrees\Module\ModuleReportInterface;
 use Fisharebest\Webtrees\Module\ModuleSidebarInterface;
+use Fisharebest\Webtrees\Module\ModuleTabInterface;
 use Fisharebest\Webtrees\Module\ModuleThemeInterface;
 use Fisharebest\Webtrees\Module\TreesMenuModule;
 
@@ -40,7 +43,7 @@ class ModuleTest extends \Fisharebest\Webtrees\TestCase
      * @covers \Fisharebest\Webtrees\Module::all
      * @covers \Fisharebest\Webtrees\Module::coreModules
      * @covers \Fisharebest\Webtrees\Module::customModules
-     * @covers \Fisharebest\Webtrees\Module::menuSorter
+     * @covers \Fisharebest\Webtrees\Module::moduleSorter
      * @return void
      */
     public function testAll(): void
@@ -74,13 +77,15 @@ class ModuleTest extends \Fisharebest\Webtrees\TestCase
      */
     public function testFindByInterface(): void
     {
+        $this->assertNotEmpty(Module::findByInterface(ModuleAnalyticsInterface::class)->all());
         $this->assertNotEmpty(Module::findByInterface(ModuleBlockInterface::class)->all());
         $this->assertNotEmpty(Module::findByInterface(ModuleChartInterface::class)->all());
         $this->assertNotEmpty(Module::findByInterface(ModuleConfigInterface::class)->all());
         $this->assertNotEmpty(Module::findByInterface(ModuleMenuInterface::class)->all());
-        $this->assertNotEmpty(Module::findByInterface(ModuleReportInterface::class)->all());
+        $this->assertNotEmpty(Module::findByInterface(ModuleInterface::class)->all());
         $this->assertNotEmpty(Module::findByInterface(ModuleReportInterface::class)->all());
         $this->assertNotEmpty(Module::findByInterface(ModuleSidebarInterface::class)->all());
+        $this->assertNotEmpty(Module::findByInterface(ModuleTabInterface::class)->all());
 
         // THe core modules do not contain any of these.
         $this->assertEmpty(Module::findByInterface(ModuleHistoricEventsInterface::class)->all());
@@ -88,12 +93,22 @@ class ModuleTest extends \Fisharebest\Webtrees\TestCase
     }
 
     /**
+     * @covers \Fisharebest\Webtrees\Module::findByClass
+     * @return void
+     */
+    public function testFindByClass(): void
+    {
+        $this->assertNull(Module::findByClass('not-a-valid-class-name'));
+        $this->assertInstanceOf(TreesMenuModule::class, Module::findByClass(TreesMenuModule::class));
+    }
+
+    /**
      * @covers \Fisharebest\Webtrees\Module::findByName
      * @return void
      */
     public function testFindByName(): void
     {
-        $this->assertNull(Module::findByName(Module::class));
+        $this->assertNull(Module::findByName('not-a-valid-module-name'));
         $this->assertInstanceOf(TreesMenuModule::class, Module::findByName('trees-menu'));
     }
 }