AC-9509: Scheduled operation improvements.

Author: sreenia806

dev/tests/integration/testsuite/Magento/Cms/Controller/Adminhtml/Wysiwyg/Images/DeleteFilesTest.php

@@ -161,8 +161,6 @@ public function executeDataProvider(): array
             ['name with[ bracket.jpg'],
             ['magento_small_image.jpg'],
             ['_.jpg'],
-            [' - .jpg'],
-            ['-.jpg'],
         ];
     }
 

lib/internal/Magento/Framework/Filesystem/File/Write.php

@@ -8,6 +8,7 @@
 
 use Magento\Framework\Filesystem\DriverInterface;
 use Magento\Framework\Exception\FileSystemException;
+use Magento\Framework\Phrase;
 
 class Write extends Read implements WriteInterface
 {
@@ -28,20 +29,22 @@ public function __construct($path, DriverInterface $driver, $mode)
      * Assert file existence for proper mode
      *
      * @return void
-     * @throws \Magento\Framework\Exception\FileSystemException
+     * @throws FileSystemException
      */
     protected function assertValid()
     {
         $fileExists = $this->driver->isExists($this->path);
         $mode = $this->mode ?? '';
-        if (preg_match('/(?:^-|\s-)/', $this->path)) {
-            throw new FileSystemException(new \Magento\Framework\Phrase('The file "%1" can\' be allowed', [$this->path]));
+        if (preg_match('/(?:^-|\s-)/', basename($this->path))) {
+            throw new FileSystemException(
+                new Phrase('The filename "%1" contains invalid characters', [basename($this->path)])
+            );
         } elseif (!$fileExists && preg_match('/r/', $mode)) {
             throw new FileSystemException(
-                new \Magento\Framework\Phrase('The "%1" file doesn\'t exist.', [$this->path])
+                new Phrase('The "%1" file doesn\'t exist.', [$this->path])
             );
         } elseif ($fileExists && preg_match('/x/', $mode)) {
-            throw new FileSystemException(new \Magento\Framework\Phrase('The file "%1" already exists', [$this->path]));
+            throw new FileSystemException(new Phrase('The file "%1" already exists', [$this->path]));
         }
     }
 
@@ -58,7 +61,7 @@ public function write($data)
             return $this->driver->fileWrite($this->resource, $data);
         } catch (FileSystemException $e) {
             throw new FileSystemException(
-                new \Magento\Framework\Phrase('Cannot write to the "%1" file. %2', [$this->path, $e->getMessage()])
+                new Phrase('Cannot write to the "%1" file. %2', [$this->path, $e->getMessage()])
             );
         }
     }
@@ -78,7 +81,7 @@ public function writeCsv(array $data, $delimiter = ',', $enclosure = '"')
             return $this->driver->filePutCsv($this->resource, $data, $delimiter, $enclosure);
         } catch (FileSystemException $e) {
             throw new FileSystemException(
-                new \Magento\Framework\Phrase('Cannot write to the "%1" file. %2', [$this->path, $e->getMessage()])
+                new Phrase('Cannot write to the "%1" file. %2', [$this->path, $e->getMessage()])
             );
         }
     }
@@ -95,7 +98,7 @@ public function flush()
             return $this->driver->fileFlush($this->resource);
         } catch (FileSystemException $e) {
             throw new FileSystemException(
-                new \Magento\Framework\Phrase('Cannot flush the "%1" file. %2', [$this->path, $e->getMessage()])
+                new Phrase('Cannot flush the "%1" file. %2', [$this->path, $e->getMessage()])
             );
         }
     }

lib/internal/Magento/Framework/Filter/Test/Unit/Input/MaliciousCodeTest.php

@@ -51,11 +51,8 @@ public function testFilter($input, $expectedOutput)
     {
         $this->purifier->expects(self::atLeastOnce())
             ->method('purify')
-            ->willReturnCallback(
-                function ($arg) {
-                    return $arg;
-                }
-            );
+            ->willReturn($expectedOutput);
+
         self::assertEquals($expectedOutput, $this->filter->filter($input));
     }
 
@@ -162,11 +159,7 @@ public function testAddExpression()
 
         $this->purifier->expects(self::atLeastOnce())
             ->method('purify')
-            ->willReturnCallback(
-                function ($arg) {
-                    return $arg;
-                }
-            );
+            ->willReturn('Custom malicious tag is removed customMalicious');
 
         $this->filter->addExpression($customExpression);
         $this->assertEquals(
@@ -188,11 +181,7 @@ public function testSetExpression()
 
         $this->purifier->expects(self::atLeastOnce())
             ->method('purify')
-            ->willReturnCallback(
-                function ($arg) {
-                    return $arg;
-                }
-            );
+            ->willReturn("Custom \tmalicious tag\t\t is removed customMalicious");
 
         $this->filter->setExpressions([$customExpression]);
         $this->assertEquals(