updated tests.

Author: ljonesfl

.gitignore

@@ -28,4 +28,4 @@ composer.lock
 .phpunit.cache
 examples/test.log
 
-coverage.xml
+coverage*

composer.json

@@ -25,7 +25,8 @@
     },
     "autoload-dev": {
         "psr-4": {
-            "Tests\\": "tests/"
+            "Tests\\": "tests/",
+            "Neuron\\Core\\": "../core/src/Core/"
         }
     },
     "bin": [

src/Cli/Commands/Core/VersionCommand.php

@@ -3,12 +3,24 @@
 namespace Neuron\Cli\Commands\Core;
 
 use Neuron\Cli\Commands\Command;
+use Neuron\Core\System\IFileSystem;
+use Neuron\Core\System\RealFileSystem;
 
 /**
  * Version command - displays version information for Neuron components
  */
 class VersionCommand extends Command
 {
+	private IFileSystem $fs;
+
+	/**
+	 * @param IFileSystem|null $fs File system implementation (null = use real file system)
+	 */
+	public function __construct( ?IFileSystem $fs = null )
+	{
+		$this->fs = $fs ?? new RealFileSystem();
+	}
+
 	/**
 	 * @inheritDoc
 	 */
@@ -159,57 +171,68 @@ private function showComponentVersion( string $componentName ): void
 
 	/**
 	 * Get the CLI component version
-	 * 
+	 *
 	 * @return string
 	 */
 	private function getCliVersion(): string
 	{
 		$composerJson = dirname( __DIR__, 4 ) . '/composer.json';
-		
-		if( file_exists( $composerJson ) )
+
+		if( $this->fs->fileExists( $composerJson ) )
 		{
-			$composer = json_decode( file_get_contents( $composerJson ), true );
-			return $composer['version'] ?? '0.1.0';
+			$content = $this->fs->readFile( $composerJson );
+			if( $content !== false )
+			{
+				$composer = json_decode( $content, true );
+				return $composer['version'] ?? '0.1.0';
+			}
 		}
-		
+
 		return '0.1.0';
 	}
 
 	/**
 	 * Get information about installed Neuron components
-	 * 
+	 *
 	 * @return array
 	 */
 	private function getInstalledComponents(): array
 	{
 		$components = [];
-		
+
 		// Find vendor directory
 		$vendorDir = $this->findVendorDirectory();
-		
+
 		if( !$vendorDir )
 		{
 			return $components;
 		}
-		
+
 		// Load from composer's installed.json
 		$installedJson = $vendorDir . '/composer/installed.json';
-		
-		if( !file_exists( $installedJson ) )
+
+		if( !$this->fs->fileExists( $installedJson ) )
 		{
 			return $components;
 		}
-		
-		$installed = json_decode( file_get_contents( $installedJson ), true );
-		
+
+		$content = $this->fs->readFile( $installedJson );
+
+		if( $content === false )
+		{
+			return $components;
+		}
+
+		$installed = json_decode( $content, true );
+
 		if( !$installed )
 		{
 			return $components;
 		}
-		
+
 		// Handle both composer 1.x and 2.x formats
 		$packages = isset( $installed['packages'] ) ? $installed['packages'] : $installed;
-		
+
 		foreach( $packages as $package )
 		{
 			// Only include neuron-php packages
@@ -218,35 +241,37 @@ private function getInstalledComponents(): array
 				$components[$package['name']] = $package;
 			}
 		}
-		
+
 		// Sort by name
 		ksort( $components );
-		
+
 		return $components;
 	}
 
 	/**
 	 * Find the vendor directory
-	 * 
+	 *
 	 * @return string|null
 	 */
 	private function findVendorDirectory(): ?string
 	{
+		$cwd = $this->fs->getcwd();
+
 		$locations = [
 			dirname( __DIR__, 4 ) . '/vendor',
 			dirname( __DIR__, 5 ) . '/vendor',
-			getcwd() . '/vendor',
+			$cwd . '/vendor',
 		];
-		
+
 		foreach( $locations as $location )
 		{
-			$path = realpath( $location );
-			if( $path && is_dir( $path ) )
+			$path = $this->fs->realpath( $location );
+			if( $path && $this->fs->isDir( $path ) )
 			{
 				return $path;
 			}
 		}
-		
+
 		return null;
 	}
 }

src/Cli/Console/Input.php

@@ -14,13 +14,16 @@ class Input
 	private array $arguments = [];
 	private array $options = [];
 	private array $rawArguments = [];
-	
+	private StreamInterface $stream;
+
 	/**
 	 * @param array $argv Command-line arguments (without script name)
+	 * @param StreamInterface|null $stream Stream for interactive I/O (null = use standard streams)
 	 */
-	public function __construct( array $argv = [] )
+	public function __construct( array $argv = [], ?StreamInterface $stream = null )
 	{
 		$this->argv = $argv;
+		$this->stream = $stream ?? new StandardStream();
 		$this->parseRaw();
 	}
 
@@ -239,28 +242,28 @@ public function getArgv(): array
 
 	/**
 	 * Check if input is interactive (connected to a TTY)
-	 * 
+	 *
 	 * @return bool
 	 */
 	public function isInteractive(): bool
 	{
-		return posix_isatty( STDIN );
+		return $this->stream->isInteractive();
 	}
 
 	/**
 	 * Read a line from standard input
-	 * 
+	 *
 	 * @param string $prompt Optional prompt to display
 	 * @return string|false
 	 */
 	public function readLine( string $prompt = '' ): string|false
 	{
 		if( $prompt )
 		{
-			echo $prompt;
+			$this->stream->write( $prompt );
 		}
-		
-		return fgets( STDIN );
+
+		return $this->stream->read();
 	}
 
 	/**
@@ -386,8 +389,8 @@ public function choice( string $question, array $choices, mixed $default = null,
 		}
 
 		// Display the question
-		echo $question . PHP_EOL;
-		
+		$this->stream->write( $question . PHP_EOL );
+
 		// Display choices
 		$index = 1;
 		$indexMap = [];
@@ -396,7 +399,7 @@ public function choice( string $question, array $choices, mixed $default = null,
 			$indexMap[$index] = $key;
 			$isDefault = ( $default !== null && ( $key === $default || $value === $default ) );
 			$marker = $isDefault ? ' (default)' : '';
-			echo "  [{$index}] {$value}{$marker}" . PHP_EOL;
+			$this->stream->write( "  [{$index}] {$value}{$marker}" . PHP_EOL );
 			$index++;
 		}
 
@@ -448,7 +451,7 @@ public function choice( string $question, array $choices, mixed $default = null,
 			if( $choice === null && $default === null )
 			{
 				// Invalid choice and no default, ask again
-				echo "Invalid choice. Please try again." . PHP_EOL;
+				$this->stream->write( "Invalid choice. Please try again." . PHP_EOL );
 				return $this->choice( $question, $choices, $default, $allowMultiple );
 			}
 

src/Cli/Console/StandardStream.php

@@ -0,0 +1,46 @@
+<?php
+
+namespace Neuron\Cli\Console;
+
+/**
+ * Standard stream implementation using real STDIN/STDOUT.
+ *
+ * This is the production implementation that wraps PHP's standard input/output streams.
+ * Uses STDIN for reading and STDOUT for writing, providing real terminal interaction.
+ */
+class StandardStream implements StreamInterface
+{
+	/**
+	 * @param resource $inputStream Input stream resource (default: STDIN)
+	 * @param resource $outputStream Output stream resource (default: STDOUT)
+	 */
+	public function __construct(
+		private mixed $inputStream = STDIN,
+		private mixed $outputStream = STDOUT
+	) {
+	}
+
+	/**
+	 * @inheritDoc
+	 */
+	public function read(): string|false
+	{
+		return fgets( $this->inputStream );
+	}
+
+	/**
+	 * @inheritDoc
+	 */
+	public function write( string $data ): void
+	{
+		fwrite( $this->outputStream, $data );
+	}
+
+	/**
+	 * @inheritDoc
+	 */
+	public function isInteractive(): bool
+	{
+		return posix_isatty( $this->inputStream );
+	}
+}

src/Cli/Console/StreamInterface.php

@@ -0,0 +1,34 @@
+<?php
+
+namespace Neuron\Cli\Console;
+
+/**
+ * Interface for input/output stream abstraction.
+ *
+ * Provides a testable abstraction over STDIN/STDOUT for interactive CLI operations.
+ * Implementations can be real streams for production or mock streams for testing.
+ */
+interface StreamInterface
+{
+	/**
+	 * Read a line from the input stream
+	 *
+	 * @return string|false Returns the read line or false on EOF
+	 */
+	public function read(): string|false;
+
+	/**
+	 * Write data to the output stream
+	 *
+	 * @param string $data Data to write
+	 * @return void
+	 */
+	public function write( string $data ): void;
+
+	/**
+	 * Check if the stream is connected to an interactive terminal
+	 *
+	 * @return bool True if interactive (TTY), false otherwise
+	 */
+	public function isInteractive(): bool;
+}