improved --cider mode (thx Claude)

Author: dg

src/Framework/Console.php

@@ -9,7 +9,7 @@
 
 namespace Tester;
 
-use const PHP_SAPI, STDOUT;
+use const PHP_SAPI, STDOUT, STR_PAD_BOTH, STR_PAD_LEFT, STR_PAD_RIGHT;
 
 
 /**
@@ -96,12 +96,70 @@ public static function showCursor(bool $visible): string
 	}
 
 
+	/**
+	 * Returns ANSI sequence to move cursor up.
+	 */
+	public static function cursorUp(int $lines): string
+	{
+		return "\e[{$lines}A";
+	}
+
+
 	/**
 	 * Returns display width of string (number of terminal columns).
 	 */
 	public static function textWidth(string $text): int
 	{
+		$text = self::stripAnsi($text);
 		return preg_match_all('/./su', $text)
 			+ preg_match_all('/[\x{1F300}-\x{1F9FF}]/u', $text); // emoji are 2-wide
 	}
+
+
+	/**
+	 * Pads text to specified display width.
+	 */
+	public static function pad(
+		string $text,
+		int $width,
+		string $char = ' ',
+		int $type = STR_PAD_RIGHT,
+	): string
+	{
+		$padding = $width - self::textWidth($text);
+		if ($padding <= 0) {
+			return $text;
+		}
+
+		return match ($type) {
+			STR_PAD_LEFT => str_repeat($char, $padding) . $text,
+			STR_PAD_RIGHT => $text . str_repeat($char, $padding),
+			STR_PAD_BOTH => str_repeat($char, intdiv($padding, 2)) . $text . str_repeat($char, $padding - intdiv($padding, 2)),
+		};
+	}
+
+
+	/**
+	 * Truncates text to max display width, adding ellipsis if needed.
+	 */
+	public static function truncate(string $text, int $maxWidth, string $ellipsis = '…'): string
+	{
+		if (self::textWidth($text) <= $maxWidth) {
+			return $text;
+		}
+
+		$ellipsisWidth = self::textWidth($ellipsis);
+		$result = '';
+		$width = 0;
+		foreach (preg_split('//u', $text, -1, PREG_SPLIT_NO_EMPTY) as $char) {
+			$charWidth = preg_match('/[\x{1F300}-\x{1F9FF}]/u', $char) ? 2 : 1;
+			if ($width + $charWidth + $ellipsisWidth > $maxWidth) {
+				break;
+			}
+			$result .= $char;
+			$width += $charWidth;
+		}
+
+		return $result . $ellipsis;
+	}
 }

src/Runner/Output/ConsolePrinter.php

@@ -11,10 +11,12 @@
 
 use Tester;
 use Tester\Console;
+use Tester\Environment;
+use Tester\Runner\Job;
 use Tester\Runner\Runner;
 use Tester\Runner\Test;
-use function sprintf, strlen;
-use const DIRECTORY_SEPARATOR;
+use function count, fwrite, sprintf, str_repeat, strlen;
+use const DIRECTORY_SEPARATOR, STR_PAD_BOTH;
 
 
 /**
@@ -26,13 +28,18 @@ class ConsolePrinter implements Tester\Runner\OutputHandler
 	public const ModeCider = 2;
 	public const ModeLine = 3;
 
+	private const MaxDisplayedThreads = 20;
+
 	/** @var resource */
 	private $file;
 	private string $buffer;
 	private float $time;
 	private int $count;
 	private array $results;
 	private ?string $baseDir;
+	private int $panelWidth = 60;
+	private int $panelHeight = 0;
+	private \WeakMap $startTimes;
 
 
 	public function __construct(
@@ -42,6 +49,7 @@ public function __construct(
 		private int $mode = self::ModeDots,
 	) {
 		$this->file = fopen($file ?? 'php://output', 'w');
+		$this->startTimes = new \WeakMap;
 	}
 
 
@@ -52,6 +60,9 @@ public function begin(): void
 		$this->baseDir = null;
 		$this->results = [Test::Passed => 0, Test::Skipped => 0, Test::Failed => 0];
 		$this->time = -microtime(as_float: true);
+		if ($this->mode === self::ModeCider && $this->runner->threadCount < 2) {
+			$this->mode = self::ModeLine;
+		}
 		fwrite($this->file, Console::showCursor(false)
 			. $this->runner->getInterpreter()->getShortInfo()
 			. ' | ' . $this->runner->getInterpreter()->getCommandLine()
@@ -89,7 +100,7 @@ public function finish(Test $test): void
 		$this->results[$result]++;
 		fwrite($this->file, match ($this->mode) {
 			self::ModeDots => [Test::Passed => '.', Test::Skipped => 's', Test::Failed => Console::colorize('F', 'white/red')][$result],
-			self::ModeCider => [Test::Passed => '🍏', Test::Skipped => 's', Test::Failed => '🍎'][$result],
+			self::ModeCider => '',
 			self::ModeLine => $this->generateFinishLine($test),
 		});
 
@@ -106,6 +117,12 @@ public function finish(Test $test): void
 
 	public function end(): void
 	{
+		if ($this->panelHeight) {
+			fwrite($this->file, Console::cursorUp($this->panelHeight)
+				. str_repeat(Console::ClearLine . "\n", $this->panelHeight)
+				. Console::cursorUp($this->panelHeight));
+		}
+
 		$run = array_sum($this->results);
 		fwrite($this->file, Console::showCursor(true));
 		fwrite($this->file, !$this->count ? "No tests found\n" :
@@ -159,4 +176,76 @@ private function generateFinishLine(Test $test): string
 			$message,
 		);
 	}
+
+
+	public function jobStarted(Job $job): void
+	{
+		$this->startTimes[$job] = microtime(true);
+	}
+
+
+	/**
+	 * @param Job[] $running
+	 */
+	public function tick(array $running): void
+	{
+		if ($this->mode !== self::ModeCider) {
+			return;
+		}
+
+		// Move cursor up to overwrite previous output
+		if ($this->panelHeight) {
+			fwrite($this->file, Console::cursorUp($this->panelHeight));
+		}
+
+		$lines = [];
+
+		// Header with progress bar
+		$barWidth = $this->panelWidth - 12;
+		$filled = (int) round($barWidth * ($this->runner->getFinishedCount() / $this->runner->getJobCount()));
+		$lines[] = '╭' . Console::pad(' ' . str_repeat('█', $filled) . str_repeat('░', $barWidth - $filled) . ' ', $this->panelWidth - 2, '─', STR_PAD_BOTH) . '╮';
+
+		$threadJobs = [];
+		foreach ($running as $job) {
+			$threadJobs[(int) $job->getEnvironmentVariable(Environment::VariableThread)] = $job;
+		}
+
+		// Thread lines
+		$numWidth = strlen((string) $this->runner->threadCount);
+		$displayCount = min($this->runner->threadCount, self::MaxDisplayedThreads);
+
+		for ($t = 1; $t <= $displayCount; $t++) {
+			if (isset($threadJobs[$t])) {
+				$job = $threadJobs[$t];
+				$name = basename($job->getTest()->getFile());
+				$time = sprintf('%0.1fs', microtime(true) - ($this->startTimes[$job] ?? microtime(true)));
+				$nameWidth = $this->panelWidth - $numWidth - strlen($time) - 7;
+				$name = Console::pad(Console::truncate($name, $nameWidth), $nameWidth);
+				$line = Console::colorize(sprintf("%{$numWidth}d:", $t), 'lime') . " $name " . Console::colorize($time, 'yellow');
+			} else {
+				$line = Console::pad(Console::colorize(sprintf("%{$numWidth}d: -", $t), 'gray'), $this->panelWidth - 4);
+			}
+			$lines[] = '│ ' . $line . ' │';
+		}
+
+		if ($this->runner->threadCount > self::MaxDisplayedThreads) {
+			$more = $this->runner->threadCount - self::MaxDisplayedThreads;
+			$ellipsis = Console::colorize("… and $more more", 'gray');
+			$lines[] = '│' . Console::pad($ellipsis, $this->panelWidth - 2) . '│';
+		}
+
+		// Footer: (85 tests, 🍏×74 🍎×2, 9.0s)
+		$summary = "($this->count tests, "
+			. ($this->results[Test::Passed] ? "🍏×{$this->results[Test::Passed]}" : '')
+			. ($this->results[Test::Failed] ? " 🍎×{$this->results[Test::Failed]}" : '')
+			. ', ' . sprintf('%0.1fs', $this->time + microtime(true)) . ')';
+		$lines[] = '╰' . Console::pad($summary, $this->panelWidth - 2, '─', STR_PAD_BOTH) . '╯';
+
+		foreach ($lines as $line) {
+			fwrite($this->file, "\r" . $line . Console::ClearLine . "\n");
+		}
+		fflush($this->file);
+
+		$this->panelHeight = count($lines);
+	}
 }

src/Runner/OutputHandler.php

@@ -12,6 +12,8 @@
 
 /**
  * Runner output.
+ * @method void jobStarted(Job $job) called when a job starts running
+ * @method void tick(Job[] $running) called periodically during test execution
  */
 interface OutputHandler
 {

src/Runner/Runner.php

@@ -39,6 +39,8 @@ class Runner
 	private ?string $tempDir = null;
 	private bool $result;
 	private array $lastResults = [];
+	private int $jobCount = 0;
+	private int $finishedCount = 0;
 
 
 	public function __construct(PhpInterpreter $interpreter)
@@ -89,6 +91,8 @@ public function run(): bool
 		foreach ($this->paths as $path) {
 			$this->findTests($path);
 		}
+		$this->finishedCount = 0;
+		$this->jobCount = count($this->jobs);
 
 		if ($this->tempDir) {
 			usort(
@@ -98,18 +102,27 @@ public function run(): bool
 		}
 
 		$threads = range(1, $this->threadCount);
-
 		$async = $this->threadCount > 1 && count($this->jobs) > 1;
 
 		try {
 			while (($this->jobs || $running) && !$this->interrupted) {
 				while ($threads && $this->jobs) {
 					$running[] = $job = array_shift($this->jobs);
 					$job->setEnvironmentVariable(Environment::VariableThread, (string) array_shift($threads));
+					foreach ($this->outputHandlers as $handler) {
+						if (method_exists($handler, 'jobStarted')) {
+							$handler->jobStarted($job);
+						}
+					}
 					$job->run(async: $async);
 				}
 
 				if ($async) {
+					foreach ($this->outputHandlers as $handler) {
+						if (method_exists($handler, 'tick')) {
+							$handler->tick($running);
+						}
+					}
 					usleep(Job::RunSleep); // stream_select() doesn't work with proc_open()
 				}
 
@@ -120,6 +133,7 @@ public function run(): bool
 
 					if (!$job->isRunning()) {
 						$threads[] = $job->getEnvironmentVariable(Environment::VariableThread);
+						$this->finishedCount++;
 						$this->testHandler->assess($job);
 						unset($running[$key]);
 					}
@@ -210,6 +224,18 @@ public function getInterpreter(): PhpInterpreter
 	}
 
 
+	public function getJobCount(): int
+	{
+		return $this->jobCount;
+	}
+
+
+	public function getFinishedCount(): int
+	{
+		return $this->finishedCount;
+	}
+
+
 	private function getLastResult(Test $test): int
 	{
 		$signature = $test->getSignature();

tests/Framework/Console.pad.phpt

@@ -0,0 +1,37 @@
+<?php
+
+declare(strict_types=1);
+
+use Tester\Assert;
+use Tester\Console;
+
+require __DIR__ . '/../bootstrap.php';
+
+
+// pad() - right (default)
+Assert::same('hello     ', Console::pad('hello', 10));
+Assert::same('hello     ', Console::pad('hello', 10, ' ', STR_PAD_RIGHT));
+
+// pad() - left
+Assert::same('     hello', Console::pad('hello', 10, ' ', STR_PAD_LEFT));
+
+// pad() - center (both)
+Assert::same('  hello   ', Console::pad('hello', 10, ' ', STR_PAD_BOTH));
+Assert::same('   hi   ', Console::pad('hi', 8, ' ', STR_PAD_BOTH)); // odd padding: 3 left, 3 right
+Assert::same('  hi   ', Console::pad('hi', 7, ' ', STR_PAD_BOTH)); // odd padding: 2 left, 3 right
+
+// pad() - text already at width
+Assert::same('hello', Console::pad('hello', 5));
+Assert::same('hello', Console::pad('hello', 3)); // over width, no change
+
+// pad() - unicode padding character
+Assert::same('hello─────', Console::pad('hello', 10, '─'));
+Assert::same('─────hello', Console::pad('hello', 10, '─', STR_PAD_LEFT));
+Assert::same('──hello───', Console::pad('hello', 10, '─', STR_PAD_BOTH));
+
+// pad() - text with emoji (emoji is 2-wide)
+Assert::same('🍏×5    ', Console::pad('🍏×5', 8)); // 🍏(2) + ×(1) + 5(1) = 4 width, need 4 spaces
+Assert::same('    🍏×5', Console::pad('🍏×5', 8, ' ', STR_PAD_LEFT));
+
+// pad() - empty string
+Assert::same('     ', Console::pad('', 5));

tests/Framework/Console.truncate.phpt

@@ -0,0 +1,37 @@
+<?php
+
+declare(strict_types=1);
+
+use Tester\Assert;
+use Tester\Console;
+
+require __DIR__ . '/../bootstrap.php';
+
+
+// truncate() - no truncation needed
+Assert::same('hello', Console::truncate('hello', 10));
+Assert::same('hello', Console::truncate('hello', 5)); // exactly at width
+
+// truncate() - basic truncation
+Assert::same('hel…', Console::truncate('hello', 4));
+Assert::same('h…', Console::truncate('hello', 2));
+Assert::same('hello wor…', Console::truncate('hello world', 10));
+
+// truncate() - custom ellipsis
+Assert::same('hello...', Console::truncate('hello world', 8, '...'));
+Assert::same('hell...', Console::truncate('hello world', 7, '...'));
+
+// truncate() - unicode text
+Assert::same('příl…', Console::truncate('příliš', 5));
+Assert::same('žlu…', Console::truncate('žluťoučký', 4));
+
+// truncate() - text with emoji (emoji is 2-wide)
+Assert::same('🍏…', Console::truncate('🍏🍎🍏', 4)); // 🍏(2) + …(1) = 3, fits in 4
+Assert::same('🍏🍎…', Console::truncate('🍏🍎🍏', 5)); // 🍏(2) + 🍎(2) + …(1) = 5, fits in 5
+Assert::same('a🍏…', Console::truncate('a🍏🍎', 4)); // a(1) + 🍏(2) + …(1) = 4, fits in 4
+
+// truncate() - edge case: maxWidth smaller than ellipsis
+Assert::same('…', Console::truncate('hello', 1));
+
+// truncate() - empty string
+Assert::same('', Console::truncate('', 5));