Skip to content
Snippets Groups Projects
Commit bce6afde authored by Adam G-H's avatar Adam G-H
Browse files

And TestSiteExcluder

parent f7ecd795
No related branches found
No related tags found
1 merge request!263Issue #3274323: Split ExcludedPathsSubscriber into multiple classes
This commit is part of merge request !263. Comments created here will be created in the context of that merge request.
...@@ -13,7 +13,6 @@ use Drupal\package_manager\EventSubscriber\SqliteDatabaseExcluder; ...@@ -13,7 +13,6 @@ use Drupal\package_manager\EventSubscriber\SqliteDatabaseExcluder;
* @covers \Drupal\package_manager\EventSubscriber\SiteFilesExcluder * @covers \Drupal\package_manager\EventSubscriber\SiteFilesExcluder
* @covers \Drupal\package_manager\EventSubscriber\SqliteDatabaseExcluder * @covers \Drupal\package_manager\EventSubscriber\SqliteDatabaseExcluder
* @covers \Drupal\package_manager\EventSubscriber\TestSiteExcluder * @covers \Drupal\package_manager\EventSubscriber\TestSiteExcluder
* @covers \Drupal\package_manager\EventSubscriber\VendorHardeningExcluder
* *
* @group package_manager * @group package_manager
*/ */
...@@ -84,7 +83,6 @@ class ExcludedPathsTest extends PackageManagerKernelTestBase { ...@@ -84,7 +83,6 @@ class ExcludedPathsTest extends PackageManagerKernelTestBase {
$stage_dir = $stage->getStageDirectory(); $stage_dir = $stage->getStageDirectory();
$ignore = [ $ignore = [
'sites/simpletest',
"$site_path/files/ignore.txt", "$site_path/files/ignore.txt",
'private/ignore.txt', 'private/ignore.txt',
"$site_path/settings.php", "$site_path/settings.php",
......
<?php
namespace Drupal\Tests\package_manager\Kernel;
/**
* @covers \Drupal\package_manager\EventSubscriber\TestSiteExcluder
*
* @group package_manager
*/
class TestSiteExcluderTest extends PackageManagerKernelTestBase {
/**
* {@inheritdoc}
*/
protected function setUp(): void {
// In this test, we want to disable the lock file validator because, even
// though both the active and stage directories will have a valid lock file,
// this validator will complain because they don't differ at all.
$this->disableValidators[] = 'package_manager.validator.lock_file';
parent::setUp();
}
/**
* Tests that Git directories are excluded from staging operations.
*/
public function testGitDirectoriesExcluded(): void {
// In this test, we want to perform the actual staging operations so that we
// can be sure that files are staged as expected.
$this->container->get('module_installer')->uninstall([
'package_manager_bypass',
]);
// Ensure we have an up-to-date container.
$this->container = $this->container->get('kernel')->getContainer();
$this->createTestProject();
$active_dir = $this->container->get('package_manager.path_locator')
->getProjectRoot();
$stage = $this->createStage();
$stage->create();
$stage_dir = $stage->getStageDirectory();
$ignored = [
'sites/simpletest',
];
foreach ($ignored as $path) {
$this->assertFileExists("$active_dir/$path");
$this->assertFileDoesNotExist("$stage_dir/$path");
}
$stage->apply();
// The ignored files should still be in the active directory.
foreach ($ignored as $path) {
$this->assertFileExists("$active_dir/$path");
}
}
}
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment