Add tests

system/Publisher/Publisher.php

@@ -108,8 +108,8 @@ public static function discover($directory = 'Publishers'): array
 			return [];
 		}
 
-		// Loop over each file checking to see if it is a Primer
-		foreach ($files as $file)
+		// Loop over each file checking to see if it is a Publisher
+		foreach (array_unique($files) as $file)
 		{
 			$className = $locator->findQualifiedNameFromPath($file);
 
@@ -228,8 +228,10 @@ private static function wipeDirectory(string $directory)
 			$attempts = 10;
 			while ((bool) $attempts && ! delete_files($directory, true, false, true))
 			{
+				// @codeCoverageIgnoreStart
 				$attempts--;
 				usleep(100000); // .1s
+				// @codeCoverageIgnoreEnd
 			}
 
 			@rmdir($directory);
@@ -270,6 +272,13 @@ private static function safeCopyFile(string $from, string $to, bool $replace): v
 			unlink($to);
 		}
 
+		// Make sure the directory exists
+		$directory = pathinfo($to, PATHINFO_DIRNAME);
+		if (! is_dir($directory))
+		{
+			mkdir($directory, 0775, true);
+		}
+
 		// Allow copy() to throw errors
 		copy($from, $to);
 	}
@@ -311,10 +320,11 @@ public function __destruct()
 	 * This method should be reimplemented by
 	 * child classes intended for discovery.
 	 *
-	 * @return void
+	 * @return bool
 	 */
 	public function publish()
 	{
+		return $this->addPath('/')->merge(true);
 	}
 
 	//--------------------------------------------------------------------
@@ -686,7 +696,7 @@ public function merge(bool $replace = true): bool
 		$this->errors = [];
 
 		// Get the file from source for special handling
-		$sourced = self::matchFiles($this->getFiles(), $this->source);
+		$sourced = self::filterFiles($this->getFiles(), $this->source);
 
 		// Handle everything else with a flat copy
 		$this->files = array_diff($this->files, $sourced);