#55 Throw Exception when cannot resolve URL attribute

dev/tests/unit/Magento/FunctionalTestFramework/Test/Objects/ActionObjectTest.php

@@ -24,6 +24,8 @@
  */
 class ActionObjectTest extends MagentoTestCase
 {
+    const STUB_PAGE_URL_WITH_NO_ATTRIBUTE = '{{PageObject}}/some/path';
+
     /**
      * Before test functionality
      * @return void
@@ -227,16 +229,17 @@ public function testResolveUrl()
     }
 
     /**
-     * {{PageObject}} should not be replaced and should elicit a warning in console
+     * {{PageObject}} should not be replaced and should throw an exception!
      *
      * @throws /Exception
      */
     public function testResolveUrlWithNoAttribute()
     {
-        // Set up mocks
+        // Given
         $actionObject = new ActionObject('merge123', 'amOnPage', [
-            'url' => '{{PageObject}}'
+            'url' => self::STUB_PAGE_URL_WITH_NO_ATTRIBUTE
         ]);
+
         $pageObject = new PageObject('PageObject', '/replacement/url.html', 'Test', [], false, "test");
         $pageObjectList = ["PageObject" => $pageObject];
         $instance = AspectMock::double(
@@ -245,20 +248,14 @@ public function testResolveUrlWithNoAttribute()
         )->make(); // bypass the private constructor
         AspectMock::double(PageObjectHandler::class, ['getInstance' => $instance]);
 
-        // Call the method under test
-        $actionObject->resolveReferences();
-
-        // Expect this warning to get generated
-        TestLoggingUtil::getInstance()->validateMockLogStatement(
-            "warning",
-            "page url attribute not found and is required",
-            ['action' => $actionObject->getType(), 'url' => '{{PageObject}}', 'stepKey' => $actionObject->getStepKey()]
-        );
-
-        // Verify
+        // Expect
+        $this->expectExceptionMessage('Can not resolve replacements: "{{PageObject}}"');
         $expected = [
-            'url' => '{{PageObject}}'
+            'url' => self::STUB_PAGE_URL_WITH_NO_ATTRIBUTE
         ];
+
+        // When
+        $actionObject->resolveReferences();
         $this->assertEquals($expected, $actionObject->getCustomActionAttributes());
     }
 

src/Magento/FunctionalTestingFramework/Test/Objects/ActionObject.php

@@ -404,6 +404,14 @@ private function resolveUrlReference()
         $url = $this->actionAttributes[ActionObject::ACTION_ATTRIBUTE_URL];
 
         $replacement = $this->findAndReplaceReferences(PageObjectHandler::getInstance(), $url);
+
+        $missingReferences = $this->getMissingReferences($replacement);
+        if (!empty($missingReferences)) {
+            throw new TestReferenceException(
+                sprintf('Can not resolve replacements: "%s"', implode('", "', $missingReferences))
+            );
+        }
+
         if ($replacement) {
             $this->resolvedCustomAttributes[ActionObject::ACTION_ATTRIBUTE_URL] = $replacement;
             $allPages = PageObjectHandler::getInstance()->getAllObjects();
@@ -417,6 +425,22 @@ private function resolveUrlReference()
         }
     }
 
+    /**
+     * Returns array of missing references
+     *
+     * @param string $replacement
+     * @return array
+     */
+    private function getMissingReferences($replacement): array
+    {
+        $mustachePattern = '/({{[\w]+}})|({{[\w]+\.[\w\[\]]+}})|({{[\w]+\.[\w]+\((?(?!}}).)+\)}})/';
+        preg_match_all($mustachePattern, $replacement, $matches);
+
+        return array_filter($matches[1], function ($match) {
+            return !empty($match) && false === strpos($match, '_ENV.');
+        });
+    }
+
     /**
      * Look up the value for EntityDataObjectName.Key and set it as the corresponding attribute in the resolved custom
      * attributes.