Issue #3243422: Add a kernel test of WritableFileSystemValidator

src/Validator/WritableFileSystemValidator.php

@@ -48,40 +48,27 @@ class WritableFileSystemValidator implements EventSubscriberInterface {
   }
 
   /**
-   * Wrapper around is_writable().
-   *
-   * @param string $path
-   *   The path to check.
+   * Checks that the file system is writable.
    *
-   * @return bool
-   *   TRUE if the given path is writable, otherwise FALSE.
+   * @param \Drupal\automatic_updates\Event\UpdateEvent $event
+   *   The event object.
    *
    * @todo It might make sense to use a more sophisticated method of testing
    *   writability than is_writable(), since it's not clear if that can return
    *   false negatives/positives due to things like SELinux, exotic file
    *   systems, and so forth.
    */
-  protected function isWritable(string $path): bool {
-    return is_writable($path);
-  }
-
-  /**
-   * Checks that the file system is writable.
-   *
-   * @param \Drupal\automatic_updates\Event\UpdateEvent $event
-   *   The event object.
-   */
   public function checkPermissions(UpdateEvent $event): void {
     $messages = [];
 
-    if (!$this->isWritable($this->appRoot)) {
+    if (!is_writable($this->appRoot)) {
       $messages[] = $this->t('The Drupal directory "@dir" is not writable.', [
         '@dir' => $this->appRoot,
       ]);
     }
 
     $dir = $this->pathLocator->getVendorDirectory();
-    if (!$this->isWritable($dir)) {
+    if (!is_writable($dir)) {
       $messages[] = $this->t('The vendor directory "@dir" is not writable.', ['@dir' => $dir]);
     }