diff options
Diffstat (limited to 'crates/shirabe/tests')
| -rw-r--r-- | crates/shirabe/tests/command/diagnose_command_test.rs | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/crates/shirabe/tests/command/diagnose_command_test.rs b/crates/shirabe/tests/command/diagnose_command_test.rs index 95c713ff..521cea84 100644 --- a/crates/shirabe/tests/command/diagnose_command_test.rs +++ b/crates/shirabe/tests/command/diagnose_command_test.rs @@ -7,11 +7,11 @@ use shirabe_php_shim::PhpMixed; #[test] #[serial] -#[ignore = "DiagnoseCommand::check_platform captures phpinfo() via ob_start()/ob_get_clean(), \ - which are todo!() in shirabe-php-shim (PHP output buffering and runtime introspection \ - are unmodeled), so the command panics before producing output; beyond that, diagnose \ - checks live http/https connectivity to packagist and the github.com rate limit, so the \ - test also requires real network access (as the PHP original does)"] +#[ignore = "DiagnoseCommand::check_http passes &config.borrow() into a call chain that reaches \ + CurlDownloader::download, which then does self.config.borrow_mut() on the same \ + Config RefCell, panicking with 'RefCell already borrowed'; beyond that, diagnose checks \ + live http/https connectivity to packagist and the github.com rate limit, so the test \ + also requires real network access (as the PHP original does)"] fn test_cmd_fail() { let tear_down = init_temp_composer( Some(&serde_json::json!({ "name": "foo/bar", "description": "test pkg" })), @@ -51,11 +51,11 @@ Checking github.com rate limit: " #[test] #[serial] -#[ignore = "DiagnoseCommand::check_platform captures phpinfo() via ob_start()/ob_get_clean(), \ - which are todo!() in shirabe-php-shim (PHP output buffering and runtime introspection \ - are unmodeled), so the command panics before producing output; beyond that, diagnose \ - checks live http/https connectivity to packagist and the github.com rate limit, so the \ - test also requires real network access (as the PHP original does)"] +#[ignore = "DiagnoseCommand::check_http passes &config.borrow() into a call chain that reaches \ + CurlDownloader::download, which then does self.config.borrow_mut() on the same \ + Config RefCell, panicking with 'RefCell already borrowed'; beyond that, diagnose checks \ + live http/https connectivity to packagist and the github.com rate limit, so the test \ + also requires real network access (as the PHP original does)"] fn test_cmd_success() { let tear_down = init_temp_composer( Some(&serde_json::json!({ |