New Tests

[jsas4coding]

No description provided.

[Copilot]

Pull Request Overview

This PR bolsters test coverage by introducing comprehensive failure scenarios alongside refactoring existing tests across multiple internal packages.

• Add failure-case subtests for version fetching
• Split and enrich TestRunUpdate with success and diverse failure scenarios
• Extend file-system, extractor, and downloader tests to cover error paths and edge cases

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
internal/version/version_test.go	Added failure-scenario tests for FetchLatestVersion
internal/updater/updater_test.go	Split TestRunUpdate into success and failure subtests; added new cases
internal/fs/fs_test.go	Refactored copy/move tests; added failure scenarios and EXDEV fallback
internal/extractor/extractor_test.go	Introduced failure-case tests for ExtractZip
internal/downloader/downloader_test.go	Added table-driven failure scenarios for DownloadPhpMyAdmin

Comments suppressed due to low confidence (2)

internal/updater/updater_test.go:255

• The comment at line 255 is in Portuguese while the rest of the tests use English. Consider translating it to English for consistency (e.g., "// Intentionally omit original config.inc.php to simulate restore failure").

// Intencionalmente não criaremos o config.inc.php original → vai falhar ao restaurar

internal/downloader/downloader_test.go:106

• [nitpick] This local variable version shadows the imported version package. Consider renaming it to versionStr or downloadVersion to avoid confusion.

version := "5.2.2"