better ex

add test
atk4 · Jan 12, 2023 · 2b6a0f7 · 2b6a0f7
1 parent 9361910
commit 2b6a0f7
Show file tree

Hide file tree

Showing 4 changed files with 45 additions and 13 deletions.
diff --git a/src/AppScopeTrait.php b/src/AppScopeTrait.php
@@ -4,6 +4,8 @@
 
 namespace Atk4\Core;
 
+use Atk4\Ui\App;
+
 /**
  * Typical software design will create the application scope. Most frameworks
  * relies on "static" properties, methods and classes. This does puts some
@@ -14,7 +16,7 @@
  */
 trait AppScopeTrait
 {
-    /** @var \Atk4\Ui\App Always points to current application. */
+    /** @var App */
     private $_app;
 
     /**
@@ -47,8 +49,8 @@ trait AppScopeTrait
 
     protected function assertInstanceOfApp(object $app): void
     {
-        if (!$app instanceof \Atk4\Ui\App) {
-            // called from phpunit, allow to use/test this trait without \Atk4\Ui\App class
+        if (!$app instanceof App) {
+            // called from phpunit, allow to test this trait without Atk4\Ui\App class
             if (class_exists(\PHPUnit\Framework\TestCase::class, false)) {
                 foreach (debug_backtrace(\DEBUG_BACKTRACE_IGNORE_ARGS) as $frame) {
                     if (str_starts_with($frame['class'] ?? '', 'Atk4\Core\Tests\\')) {
@@ -57,7 +59,7 @@ protected function assertInstanceOfApp(object $app): void
                 }
             }
 
-            throw new Exception('App must be instance of \Atk4\Ui\App');
+            throw new Exception('App must be instance of Atk4\Ui\App');
         }
     }
 
@@ -67,27 +69,33 @@ public function issetApp(): bool
     }
 
     /**
-     * @return \Atk4\Ui\App
+     * @return App
      */
     public function getApp()
     {
-        $this->assertInstanceOfApp($this->_app);
+        $app = $this->_app;
+        if ($app === null) {
+            throw new Exception('App is not set');
+        }
 
-        return $this->_app;
+        return $app;
     }
 
     /**
-     * @param \Atk4\Ui\App $app
+     * @param App $app
      *
      * @return static
      */
     public function setApp(object $app)
     {
         $this->assertInstanceOfApp($app);
-        if ($this->issetApp() && $this->getApp() !== $app) {
-            if ($this->getApp()->catchExceptions || $this->getApp()->alwaysRun) { // allow to replace App created by AbstractView::initDefaultApp() - TODO fix
-                throw new Exception('App cannot be replaced');
+
+        if ($this->issetApp()) {
+            if ($this->getApp() === $app) {
+                return $this;
             }
+
+            throw new Exception('App cannot be replaced');
         }
 
         $this->_app = $app;

diff --git a/src/Factory.php b/src/Factory.php
@@ -93,7 +93,7 @@ protected function _mergeSeeds(...$seeds)
 
         if (count($injection) > 0) {
             if (!TraitUtil::hasDiContainerTrait($obj)) {
-                throw (new Exception('Property injection is possible only to objects that use \Atk4\Core\DiContainerTrait trait'))
+                throw (new Exception('Property injection is possible only to objects that use Atk4\Core\DiContainerTrait trait'))
                     ->addMoreInfo('object', $obj)
                     ->addMoreInfo('injection', $injection);
             }

diff --git a/src/TraitUtil.php b/src/TraitUtil.php
@@ -28,7 +28,7 @@ public static function hasTrait($class, string $traitName): bool
         // prevent mass use for other than internal use then we can decide
         // if we want to keep support this or replace with pure interfaces
         if (!str_starts_with($traitName, 'Atk4\Core\\')) {
-            throw new Exception('Core::hasTrait is not indended for use with other than \Atk4\Core\* traits');
+            throw new Exception('TraitUtil::hasTrait is not indended for use with other than Atk4\Core\* traits');
         }
 
         if (!isset(self::$_hasTraitCache[$class][$traitName])) {

diff --git a/tests/AppScopeTraitTest.php b/tests/AppScopeTraitTest.php
@@ -6,6 +6,7 @@
 
 use Atk4\Core\AppScopeTrait;
 use Atk4\Core\ContainerTrait;
+use Atk4\Core\Exception;
 use Atk4\Core\NameTrait;
 use Atk4\Core\Phpunit\TestCase;
 use Atk4\Core\TrackableTrait;
@@ -39,6 +40,29 @@ public function testConstruct(): void
         static::assertNull($this->getProtected($child, '_app'));
         static::assertFalse($child->issetOwner());
     }
+
+    public function testAppNotSetException(): void
+    {
+        $m = new AppScopeMock();
+
+        $this->expectException(Exception::class);
+        $this->expectErrorMessage('App is not set');
+        $m->getApp();
+    }
+
+    public function testAppReplaceException(): void
+    {
+        $m = new AppScopeMock();
+        $fakeApp = new \stdClass();
+        $fakeApp2 = new \stdClass();
+        $m->setApp($fakeApp);
+
+        $m->setApp($fakeApp);
+
+        $this->expectException(Exception::class);
+        $this->expectErrorMessage('App cannot be replaced');
+        $m->setApp($fakeApp2);
+    }
 }
 
 class AppScopeMock