ISSUE-28656 - Add basic unit test compatibility for new Inline app state dependency

Author: Zach Nanninga

lib/internal/Magento/Framework/Translate/Inline.php

@@ -131,9 +131,9 @@ public function isAllowed()
             if (!$this->scope instanceof \Magento\Framework\App\ScopeInterface) {
                 $scope = $this->scopeResolver->getScope($this->scope);
             }
-            $this->isAllowed = $this->isAreaAllowed()
-                && $this->config->isActive($scope)
-                && $this->config->isDevAllowed($scope);
+            $this->isAllowed = $this->config->isActive($scope)
+                && $this->config->isDevAllowed($scope)
+                && $this->isAreaAllowed();
         }
         return $this->state->isEnabled() && $this->isAllowed;
     }

lib/internal/Magento/Framework/Translate/Test/Unit/InlineTest.php

@@ -7,8 +7,10 @@
 
 namespace Magento\Framework\Translate\Test\Unit;
 
+use Magento\Framework\App\Area;
 use Magento\Framework\App\Config\ScopeConfigInterface;
 use Magento\Framework\App\ScopeResolverInterface;
+use Magento\Framework\App\State as AppState;
 use Magento\Framework\Translate\Inline;
 use Magento\Framework\Translate\Inline\ConfigInterface;
 use Magento\Framework\Translate\Inline\ParserFactory;
@@ -51,6 +53,11 @@ class InlineTest extends TestCase
      */
     protected $stateMock;
 
+    /**
+     * @var AppState|MockObject
+     */
+    protected $appStateMock;
+
     protected function setUp(): void
     {
         $this->scopeResolverMock =
@@ -60,6 +67,7 @@ protected function setUp(): void
         $this->configMock = $this->getMockForAbstractClass(ConfigInterface::class);
         $this->parserMock = $this->getMockForAbstractClass(ParserInterface::class);
         $this->stateMock = $this->getMockForAbstractClass(StateInterface::class);
+        $this->appStateMock = $this->createMock(AppState::class);
     }
 
     /**
@@ -79,7 +87,8 @@ public function testIsAllowed($isEnabled, $isActive, $isDevAllowed, $result)
             $this->layoutMock,
             $this->configMock,
             $this->parserMock,
-            $this->stateMock
+            $this->stateMock,
+            $this->appStateMock
         );
 
         $this->assertEquals($result, $model->isAllowed());
@@ -111,7 +120,8 @@ public function testGetParser()
             $this->layoutMock,
             $this->configMock,
             $this->parserMock,
-            $this->stateMock
+            $this->stateMock,
+            $this->appStateMock
         );
         $this->assertEquals($this->parserMock, $model->getParser());
     }
@@ -133,6 +143,7 @@ public function testProcessResponseBodyStripInline($body, $expected)
             $this->configMock,
             $this->parserMock,
             $this->stateMock,
+            $this->appStateMock,
             '',
             '',
             $scope
@@ -201,6 +212,7 @@ public function testProcessResponseBody($scope, $body, $expected)
             $this->configMock,
             $this->parserMock,
             $this->stateMock,
+            $this->appStateMock,
             '',
             '',
             $scope
@@ -266,6 +278,7 @@ public function testProcessResponseBodyGetInlineScript($scope, $body, $expected)
             $this->configMock,
             $this->parserMock,
             $this->stateMock,
+            $this->appStateMock,
             '',
             '',
             $scope
@@ -292,8 +305,13 @@ public function processResponseBodyGetInlineScriptDataProvider()
      * @param bool $isDevAllowed
      * @param null|string $scope
      */
-    protected function prepareIsAllowed($isEnabled, $isActive, $isDevAllowed, $scope = null)
-    {
+    protected function prepareIsAllowed(
+        $isEnabled,
+        $isActive,
+        $isDevAllowed,
+        $scope = null,
+        $area = Area::AREA_FRONTEND
+    ) {
         $scopeMock = $this->getMockForAbstractClass(ScopeConfigInterface::class);
         $this->stateMock->expects($this->any())->method('isEnabled')->willReturn($isEnabled);
         $this->scopeResolverMock->expects(
@@ -323,5 +341,13 @@ protected function prepareIsAllowed($isEnabled, $isActive, $isDevAllowed, $scope
         )->willReturn(
             $isDevAllowed
         );
+
+        $this->appStateMock->expects(
+            ($isActive && $isDevAllowed) ? $this->once() : $this->never()
+        )->method(
+            'getAreaCode'
+        )->willReturn(
+            $area
+        );
     }
 }