Merge pull request #725 from crynobone/caller-2

[5.x] Add ProvidesBrowser::getCallerName();
laravel · Jan 27, 2020 · 6f59695 · 6f59695
2 parents effe73d + 2dca2ab
commit 6f59695
Showing 1 changed file with 12 additions and 2 deletions.
diff --git a/src/Concerns/ProvidesBrowser.php b/src/Concerns/ProvidesBrowser.php
@@ -141,7 +141,7 @@ protected function captureFailuresFor($browsers)
                 $browser->fitContent();
             }
 
-            $name = str_replace('\\', '_', get_class($this)).'_'.$this->getName(false);
+            $name = $this->getCallerName();
 
             $browser->screenshot('failure-'.$name.'-'.$key);
         });
@@ -156,7 +156,7 @@ protected function captureFailuresFor($browsers)
     protected function storeConsoleLogsFor($browsers)
     {
         $browsers->each(function ($browser, $key) {
-            $name = str_replace('\\', '_', get_class($this)).'_'.$this->getName(false);
+            $name = $this->getCallerName();
 
             $browser->storeConsoleLog($name.'-'.$key);
         });
@@ -201,6 +201,16 @@ protected function createWebDriver()
         }, 50);
     }
 
+    /**
+     * Get the browser caller name.
+     *
+     * @return string
+     */
+    protected function getCallerName()
+    {
+        return str_replace('\\', '_', get_class($this)).'_'.$this->getName(false);
+    }
+
     /**
      * Create the RemoteWebDriver instance.
      *