CloudNebulaProject/ips
1
0
Fork 0
mirror of https://codeberg.org/Toasterson/ips.git synced 2026-04-11 05:40:41 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Remove redundant file checks and update test assertions for publisher-specific file paths

Browse source 
- Eliminate obsolete assertions for the root `file` directory as it is no longer created.
- Update tests to validate file paths in `publisher`-specific directories, aligning with the updated repository structure.
- Simplify `create_directories` by removing unnecessary `file` directory creation.
This commit is contained in:
Till Wegmueller 2025-07-31 00:29:53 +02:00
parent 99dd0fe87c
commit fc00304038
No known key found for this signature in database
4 changed files with 2 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
libips/src/repository/file_backend.rs
View file

@ -1737,7 +1737,6 @@ impl FileBackend {
fn create_directories(&self) -> Result<()> { fn create_directories(&self) -> Result<()> {
// Create the main repository directories // Create the main repository directories
fs::create_dir_all(self.path.join("publisher"))?; fs::create_dir_all(self.path.join("publisher"))?;
fs::create_dir_all(self.path.join("file"))?;
fs::create_dir_all(self.path.join("index"))?; fs::create_dir_all(self.path.join("index"))?;
fs::create_dir_all(self.path.join("trans"))?; fs::create_dir_all(self.path.join("trans"))?;
fs::create_dir_all(self.path.join("obsoleted"))?; fs::create_dir_all(self.path.join("obsoleted"))?;

5
libips/src/repository/tests.rs
View file

@ -182,7 +182,6 @@ mod tests {
// Check that the repository was created // Check that the repository was created
assert!(repo_path.exists()); assert!(repo_path.exists());
assert!(repo_path.join("publisher").exists()); assert!(repo_path.join("publisher").exists());
assert!(repo_path.join("file").exists());
assert!(repo_path.join("index").exists()); assert!(repo_path.join("index").exists());
assert!(repo_path.join("trans").exists()); assert!(repo_path.join("trans").exists());
assert!(repo_path.join(REPOSITORY_CONFIG_FILENAME).exists()); assert!(repo_path.join(REPOSITORY_CONFIG_FILENAME).exists());
@ -270,8 +269,8 @@ mod tests {
let manifest_path = manifest_dir.join("example.p5m"); let manifest_path = manifest_dir.join("example.p5m");
publish_package(&mut repo, &manifest_path, &prototype_dir, "test").unwrap(); publish_package(&mut repo, &manifest_path, &prototype_dir, "test").unwrap();
// Check that the files were published // Check that the files were published in the publisher-specific directory
assert!(repo_path.join("file").exists()); assert!(repo_path.join("publisher").join("test").join("file").exists());
// Get repository information // Get repository information
let repo_info = repo.get_info().unwrap(); let repo_info = repo.get_info().unwrap();

2
pkg6repo/src/e2e_tests.rs
View file

@ -136,9 +136,7 @@ mod e2e_tests {
// Check that the repository was created // Check that the repository was created
assert!(repo_path.exists()); assert!(repo_path.exists());
assert!(repo_path.join("publisher").exists()); assert!(repo_path.join("publisher").exists());
assert!(repo_path.join("file").exists());
assert!(repo_path.join("index").exists()); assert!(repo_path.join("index").exists());
assert!(repo_path.join("pkg").exists());
assert!(repo_path.join("trans").exists()); assert!(repo_path.join("trans").exists());
assert!(repo_path.join("pkg6.repository").exists()); assert!(repo_path.join("pkg6.repository").exists());

1
pkg6repo/src/tests.rs
View file

@ -48,7 +48,6 @@ mod tests {
// Check that the repository was created // Check that the repository was created
assert!(repo_path.exists()); assert!(repo_path.exists());
assert!(repo_path.join("publisher").exists()); assert!(repo_path.join("publisher").exists());
assert!(repo_path.join("file").exists());
assert!(repo_path.join("index").exists()); assert!(repo_path.join("index").exists());
assert!(repo_path.join("trans").exists()); assert!(repo_path.join("trans").exists());
assert!(repo_path.join(REPOSITORY_CONFIG_FILENAME).exists()); assert!(repo_path.join(REPOSITORY_CONFIG_FILENAME).exists());