aboutsummaryrefslogtreecommitdiffhomepage
path: root/crates/shirabe/tests/installer/library_installer_test.rs
diff options
context:
space:
mode:
authornsfisis <nsfisis@gmail.com>2026-08-04 02:25:28 +0900
committernsfisis <nsfisis@gmail.com>2026-08-04 05:43:18 +0900
commita02fc7d728a9973a3275a0f47604081c4439b424 (patch)
treeb1a73d2eb6b17bb6c204318b58d75ae6cf46cc7a /crates/shirabe/tests/installer/library_installer_test.rs
parent20f7a7826ae048d249e0d837ca6393a5f09c9ba6 (diff)
downloadphp-shirabe-a02fc7d728a9973a3275a0f47604081c4439b424.tar.gz
php-shirabe-a02fc7d728a9973a3275a0f47604081c4439b424.tar.zst
php-shirabe-a02fc7d728a9973a3275a0f47604081c4439b424.zip
feat(plugin): activate plugins through the PHP RPC worker
Implement the remainder of PluginManager::registerPackage: the plugin autoload map is built by the ported createLoader/parseAutoloads and served to the worker over the existing reverse-RPC autoloader, files entries go through a composerRequire-equivalent glue call, and already-defined classes take the upstream _composer_tmp rename/eval path. Instantiation uses the new NewObject/CallPhpMethod lanes backed by a P table in the worker; PhpPluginProxy adapts the resulting handle to PluginInterface, with $composer/$io exposed to plugin callbacks via an R table (unsupported methods stay explicit errors). Hand-written proxy stubs cover Composer, PartialComposer and the IO hierarchy, and the stub autoloader is re-prepended after loading the Composer PHP runtime so its vendor autoloader cannot shadow proxied FQCNs. FilesystemRepository::write now mirrors InstalledVersions::reload into a running worker (class_exists-guarded, so an unloaded class keeps its upstream lazy-load behavior), removing the previously undefined observation window. The installer pipeline passes the installed repository as a shared handle instead of a long-lived `&mut dyn`: plugin registration runs inside InstallationManager::execute and re-enters the same local repository through the RepositoryManager, which would panic on the RefCell re-borrow under the old shape. PluginInterface lifecycle methods now take an owned ComposerHandle (plugins retain $composer past the call) and return anyhow::Result (PHP plugin code may throw); the plugin list uses shared ownership so the identity comparison of removePlugin survives the dual storage in registeredPlugins, matching PHP reference semantics. Ports the activate/upgrade/uninstall tests of PluginInstallerTest, serialized across the shared worker process whose persistent class table is exactly what exercises the rename path. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Diffstat (limited to 'crates/shirabe/tests/installer/library_installer_test.rs')
-rw-r--r--crates/shirabe/tests/installer/library_installer_test.rs120
1 files changed, 48 insertions, 72 deletions
diff --git a/crates/shirabe/tests/installer/library_installer_test.rs b/crates/shirabe/tests/installer/library_installer_test.rs
index baa4f13a..6402d903 100644
--- a/crates/shirabe/tests/installer/library_installer_test.rs
+++ b/crates/shirabe/tests/installer/library_installer_test.rs
@@ -12,9 +12,7 @@ use shirabe::installer::{BinaryInstallerInterface, InstallerInterface, LibraryIn
use shirabe::io::IOInterface;
use shirabe::io::null_io::NullIO;
use shirabe::package::PackageInterfaceHandle;
-use shirabe::repository::InstalledArrayRepository;
-use shirabe::repository::RepositoryInterface;
-use shirabe::repository::WritableRepositoryInterface;
+use shirabe::repository::{InstalledArrayRepository, InstalledRepositoryInterfaceHandle};
use shirabe::util::filesystem::Filesystem;
use shirabe_php_shim::PhpMixed;
use std::fs;
@@ -199,32 +197,27 @@ fn test_is_installed() {
let library = LibraryInstaller::new(setup.io.clone(), setup.composer.clone(), None, None, None);
let package = get_package("test/pkg", "1.0.0");
- let mut repository = InstalledArrayRepository::new().unwrap();
- assert!(
- !library
- .is_installed(&mut repository, package.clone())
- .unwrap()
- );
+ let repository =
+ InstalledRepositoryInterfaceHandle::new(InstalledArrayRepository::new().unwrap());
+ assert!(!library.is_installed(&repository, package.clone()).unwrap());
// package being in repo is not enough to be installed
- repository.add_package(package.clone()).unwrap();
- assert!(
- !library
- .is_installed(&mut repository, package.clone())
- .unwrap()
- );
+ repository
+ .borrow_mut()
+ .add_package(package.clone())
+ .unwrap();
+ assert!(!library.is_installed(&repository, package.clone()).unwrap());
// package being in repo and vendor/pkg/foo dir present means it is seen as installed
let pkg_dir = format!("{}/{}", setup.vendor_dir, package.get_pretty_name());
fs::create_dir_all(&pkg_dir).unwrap();
- assert!(
- library
- .is_installed(&mut repository, package.clone())
- .unwrap()
- );
+ assert!(library.is_installed(&repository, package.clone()).unwrap());
- repository.remove_package(package.clone()).unwrap();
- assert!(!library.is_installed(&mut repository, package).unwrap());
+ repository
+ .borrow_mut()
+ .remove_package(package.clone())
+ .unwrap();
+ assert!(!library.is_installed(&repository, package).unwrap());
tear_down(&mut setup);
}
@@ -250,18 +243,13 @@ fn test_install() {
let library = LibraryInstaller::new(setup.io.clone(), setup.composer.clone(), None, None, None);
- let mut repository = InstalledArrayRepository::new().unwrap();
+ let repository =
+ InstalledRepositoryInterfaceHandle::new(InstalledArrayRepository::new().unwrap());
- run(library.install(
- &std::cell::RefCell::new(
- &mut repository as &mut dyn shirabe::repository::InstalledRepositoryInterface,
- ),
- package.clone(),
- ))
- .unwrap();
+ run(library.install(&repository, package.clone())).unwrap();
// PHP asserts repository->addPackage was called once with $package.
- assert!(repository.has_package(package).unwrap());
+ assert!(repository.borrow_mut().has_package(package).unwrap());
assert!(
std::path::Path::new(&setup.vendor_dir).exists(),
@@ -308,20 +296,17 @@ fn test_update() {
.returning(|_, _, _| Ok(None));
set_download_manager(&setup, dm);
- let mut repository = InstalledArrayRepository::new().unwrap();
- repository.add_package(initial.clone()).unwrap();
+ let repository =
+ InstalledRepositoryInterfaceHandle::new(InstalledArrayRepository::new().unwrap());
+ repository
+ .borrow_mut()
+ .add_package(initial.clone())
+ .unwrap();
// The default Filesystem is fine; the LibraryInstaller's own filesystem performs the rename.
let library = LibraryInstaller::new(setup.io.clone(), setup.composer.clone(), None, None, None);
- run(library.update(
- &std::cell::RefCell::new(
- &mut repository as &mut dyn shirabe::repository::InstalledRepositoryInterface,
- ),
- initial.clone(),
- target.clone(),
- ))
- .unwrap();
+ run(library.update(&repository, initial.clone(), target.clone())).unwrap();
assert!(
std::path::Path::new(&new_target_dir).exists(),
@@ -329,8 +314,13 @@ fn test_update() {
);
assert!(!std::path::Path::new(&old_target_dir).exists());
- assert!(!repository.has_package(initial.clone()).unwrap());
- assert!(repository.has_package(target.clone()).unwrap());
+ assert!(
+ !repository
+ .borrow_mut()
+ .has_package(initial.clone())
+ .unwrap()
+ );
+ assert!(repository.borrow_mut().has_package(target.clone()).unwrap());
assert!(
std::path::Path::new(&setup.vendor_dir).exists(),
@@ -342,16 +332,7 @@ fn test_update() {
);
// Updating again, with the initial package no longer installed, fails.
- assert!(
- run(library.update(
- &std::cell::RefCell::new(
- &mut repository as &mut dyn shirabe::repository::InstalledRepositoryInterface
- ),
- initial,
- target
- ))
- .is_err()
- );
+ assert!(run(library.update(&repository, initial, target)).is_err());
tear_down(&mut setup);
}
@@ -380,30 +361,25 @@ fn test_uninstall() {
// PHP mocks hasPackage to return (true, false) over two calls; a real repository
// seeded with the package reproduces this naturally: present, then absent after
// the first uninstall removes it.
- let mut repository = InstalledArrayRepository::new().unwrap();
- repository.add_package(package.clone()).unwrap();
-
- run(library.uninstall(
- &std::cell::RefCell::new(
- &mut repository as &mut dyn shirabe::repository::InstalledRepositoryInterface,
- ),
- package.clone(),
- ))
- .unwrap();
+ let repository =
+ InstalledRepositoryInterfaceHandle::new(InstalledArrayRepository::new().unwrap());
+ repository
+ .borrow_mut()
+ .add_package(package.clone())
+ .unwrap();
- assert!(!repository.has_package(package.clone()).unwrap());
+ run(library.uninstall(&repository, package.clone())).unwrap();
- // Uninstalling again, with the package no longer installed, fails.
assert!(
- run(library.uninstall(
- &std::cell::RefCell::new(
- &mut repository as &mut dyn shirabe::repository::InstalledRepositoryInterface
- ),
- package
- ))
- .is_err()
+ !repository
+ .borrow_mut()
+ .has_package(package.clone())
+ .unwrap()
);
+ // Uninstalling again, with the package no longer installed, fails.
+ assert!(run(library.uninstall(&repository, package)).is_err());
+
tear_down(&mut setup);
}