Merge pull request #17 from takaoyuri/patch1

SamMousa · web-flow · commit 237c6cb42d3e · 2020-08-06T09:39:31.000+02:00
Add support for saving multi byte filename
diff --git a/src/Codeception/Module/WebDriver.php b/src/Codeception/Module/WebDriver.php
@@ -509,7 +509,7 @@ public function _after(TestInterface $test)
     public function _failed(TestInterface $test, $fail)
     {
         $this->debugWebDriverLogs($test);
-        $filename = preg_replace('~\W~', '.', Descriptor::getTestSignatureUnique($test));
+        $filename = preg_replace('~[^a-zA-Z0-9\x80-\xff]~', '.', Descriptor::getTestSignatureUnique($test));
         $outputDir = codecept_output_dir();
         $this->_saveScreenshot($report = $outputDir . mb_strcut($filename, 0, 245, 'utf-8') . '.fail.png');
         $test->getMetadata()->addReport('png', $report);

Original file line number	Diff line number	Diff line change
`@@ -509,7 +509,7 @@ public function _after(TestInterface $test)`
`509`	`509`	`public function _failed(TestInterface $test, $fail)`
`510`	`510`	`{`
`511`	`511`	`$this->debugWebDriverLogs($test);`
`512`		`- $filename = preg_replace('~\W~', '.', Descriptor::getTestSignatureUnique($test));`
	`512`	`+ $filename = preg_replace('~[^a-zA-Z0-9\x80-\xff]~', '.', Descriptor::getTestSignatureUnique($test));`
`513`	`513`	`$outputDir = codecept_output_dir();`
`514`	`514`	`$this->_saveScreenshot($report = $outputDir . mb_strcut($filename, 0, 245, 'utf-8') . '.fail.png');`
`515`	`515`	`$test->getMetadata()->addReport('png', $report);`