Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test(requirement-checker): Fix the requirement checker e2e tests #1178

Merged
merged 1 commit into from
Nov 19, 2023
Merged

test(requirement-checker): Fix the requirement checker e2e tests #1178

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 4 additions & 7 deletions requirement-checker/fixtures/fail-complete/expected-boxmin-stderr.dist
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,11 +6,11 @@ Box Requirements Checker
> PHP is not using any php.ini file.

> Checking Box requirements:
✘ The application requires a version matching "^10".
✔ The application requires the extension "phar".
✘ This application requires a PHP version matching "^10".
✔ This application requires the extension "phar".
✔ The package "package-with-extensions" requires the extension "json".
✘ The package "package-with-extensions" requires the extension "ldap".
✘ The package "package-with-extensions" requires the extension "random".
✔ The package "package-with-extensions" requires the extension "random".


[ERROR] Your system is not ready to run the application.
Expand All @@ -19,11 +19,8 @@ Box Requirements Checker
Fix the following mandatory requirements:
=========================================

* The application requires a version matching "^10".
* This application requires a PHP version matching "^10".
* The package "package-with-extensions" requires the extension "ldap". You
either need to enable it or request the application to be shipped with a
polyfill for this extension.
* The package "package-with-extensions" requires the extension "random". You
either need to enable it or request the application to be shipped with a
polyfill for this extension.

6 changes: 3 additions & 3 deletions requirement-checker/fixtures/fail-complete/expected-composermin-stderr.dist
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,8 @@ Box Requirements Checker
> PHP is not using any php.ini file.

> Checking Box requirements:
✘ The application requires a version matching "^10".
✔ The application requires the extension "phar".
✘ This application requires a PHP version matching "^10".
✔ This application requires the extension "phar".
✔ The package "package-with-extensions" requires the extension "json".
✘ The package "package-with-extensions" requires the extension "ldap".
✘ The package "package-with-extensions" requires the extension "random".
Expand All @@ -19,7 +19,7 @@ Box Requirements Checker
Fix the following mandatory requirements:
=========================================

* The application requires a version matching "^10".
* This application requires a PHP version matching "^10".
* The package "package-with-extensions" requires the extension "ldap". You
either need to enable it or request the application to be shipped with a
polyfill for this extension.
Expand Down
8 changes: 4 additions & 4 deletions requirement-checker/fixtures/fail-conflict/expected-boxmin-stderr.dist
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,8 @@ Box Requirements Checker
> PHP is not using any php.ini file.

> Checking Box requirements:
✔ The application requires the extension "phar".
✘ The application conflicts with the extension "pdo".
✔ This application requires the extension "phar".
✘ This application conflicts with the extension "pdo".


[ERROR] Your system is not ready to run the application.
Expand All @@ -16,6 +16,6 @@ Box Requirements Checker
Fix the following mandatory requirements:
=========================================

* The application conflicts with the extension "pdo". You need to disable it in
order to run this application.
* This application conflicts with the extension "pdo". You need to disable it
in order to run this application.

4 changes: 2 additions & 2 deletions requirement-checker/fixtures/pass-complete/expected-boxmin-stderr.dist
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,8 @@ Box Requirements Checker
> PHP is not using any php.ini file.

> Checking Box requirements:
✔ The application requires a version matching ">=5.3".
✔ The application requires the extension "phar".
✔ This application requires a PHP version matching ">=5.3".
✔ This application requires the extension "phar".


[OK] Your system is ready to run the application.
Expand Down
4 changes: 2 additions & 2 deletions requirement-checker/fixtures/pass-complete/expected-composermin-stderr.dist
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,8 @@ Box Requirements Checker
> PHP is not using any php.ini file.

> Checking Box requirements:
✔ The application requires a version matching ">=5.3".
✔ The application requires the extension "phar".
✔ This application requires a PHP version matching ">=5.3".
✔ This application requires the extension "phar".


[OK] Your system is ready to run the application.
Expand Down