updates

Author: ljonesfl

src/Cms/Cli/Commands/User/ResetPasswordCommand.php

@@ -163,7 +163,13 @@ public function execute( array $parameters = [] ): int
 			$user->setFailedLoginAttempts( 0 );
 			$user->setLockedUntil( null );
 
-			$repository->update( $user );
+			$success = $repository->update( $user );
+
+			if( !$success )
+			{
+				$this->output->error( "Failed to update password in database" );
+				return 1;
+			}
 
 			$this->output->success( "Password reset successfully for user: " . $user->getUsername() );
 			$this->output->writeln( "" );

tests/Unit/Cms/Cli/Commands/User/ResetPasswordCommandTest.php

@@ -458,4 +458,49 @@ public function testExecuteWithEmailOption(): void
 		$prompts = $this->inputReader->getPromptHistory();
 		$this->assertCount(3, $prompts);
 	}
+
+	public function testExecuteFailsWhenDatabaseUpdateFails(): void
+	{
+		$this->inputReader->addResponses([
+			'testuser',
+			'yes',
+			'NewSecurePass123!',
+			'NewSecurePass123!'
+		]);
+
+		$user = new User();
+		$user->setId(1);
+		$user->setUsername('testuser');
+		$user->setEmail('test@example.com');
+		$user->setRole('admin');
+
+		$repository = $this->createMock(DatabaseUserRepository::class);
+		$repository->expects($this->once())
+			->method('findByUsername')
+			->willReturn($user);
+
+		// update() returns false, simulating database failure
+		$repository->expects($this->once())
+			->method('update')
+			->willReturn(false);
+
+		$settings = $this->createMock(SettingManager::class);
+		Registry::getInstance()->set('Settings', $settings);
+
+		$command = $this->getMockBuilder(ResetPasswordCommand::class)
+			->onlyMethods(['getUserRepository'])
+			->getMock();
+		$command->expects($this->once())
+			->method('getUserRepository')
+			->willReturn($repository);
+
+		$command->setInput($this->input);
+		$command->setOutput($this->output);
+		$command->setInputReader($this->inputReader);
+
+		$exitCode = $command->execute();
+
+		// Should fail with exit code 1
+		$this->assertEquals(1, $exitCode);
+	}
 }

versionlog.md

@@ -1,5 +1,5 @@
 ## 0.8.42
-* Adds cms:user:password-reset command to reset a user's password from the CLI.
+* Adds cms:user:reset-passwordLe command to reset a user's password from the CLI.
 
 ## 0.8.41 2026-01-14
 * Fix for site name in error pages.