diff --git a/docs/CHANGELOG.md b/docs/CHANGELOG.md index 68faf7f..8dd990c 100644 --- a/docs/CHANGELOG.md +++ b/docs/CHANGELOG.md @@ -11,6 +11,7 @@ The format of this change log follows the advice given at [Keep a CHANGELOG](htt ## [Unreleased] ### Fixed - `moodle-plugin-ci validate` now only regards required language strings as present if they are assigned to the `$string` array. Before, other array variables were accepted although Moodle would not recognise them. +- `moodle-plugin-ci install` now provides an option `--skip-db-install` to skip DB installation if execution of unit tests and behat test will not be required. ## [2.1.1] ### Fixed diff --git a/src/Command/InstallCommand.php b/src/Command/InstallCommand.php index fc155f6..bb24bc2 100644 --- a/src/Command/InstallCommand.php +++ b/src/Command/InstallCommand.php @@ -87,7 +87,8 @@ protected function configure() ->addOption('db-host', null, InputOption::VALUE_REQUIRED, 'Database host', 'localhost') ->addOption('not-paths', null, InputOption::VALUE_REQUIRED, 'CSV of file paths to exclude', $paths) ->addOption('not-names', null, InputOption::VALUE_REQUIRED, 'CSV of file names to exclude', $names) - ->addOption('extra-plugins', null, InputOption::VALUE_REQUIRED, 'Directory of extra plugins to install', $extra); + ->addOption('extra-plugins', null, InputOption::VALUE_REQUIRED, 'Directory of extra plugins to install', $extra) + ->addOption('skip-db-install', null, InputOption::VALUE_REQUIRED, 'Skip DB installation, e.g. if only static analyses are required', false); } protected function initialize(InputInterface $input, OutputInterface $output) @@ -163,6 +164,7 @@ public function initializeInstallerFactory(InputInterface $input) $input->getOption('db-pass'), $input->getOption('db-host') ); + $factory->skipDbInstall = $input->getOption('skip-db-install'); return $factory; } diff --git a/src/Installer/InstallerFactory.php b/src/Installer/InstallerFactory.php index a959ee1..d12c9c1 100644 --- a/src/Installer/InstallerFactory.php +++ b/src/Installer/InstallerFactory.php @@ -68,6 +68,11 @@ class InstallerFactory */ public $pluginsDir; + /** + * @var bool + */ + public $skipDbInstall; + /** * Given a big bag of install options, add installers to the collection. * @@ -79,7 +84,9 @@ public function addInstallers(InstallerCollection $installers) $installers->add(new PluginInstaller($this->moodle, $this->plugin, $this->pluginsDir, $this->dumper)); $installers->add(new VendorInstaller($this->moodle, $this->plugin, $this->execute)); - if ($this->plugin->hasBehatFeatures() || $this->plugin->hasUnitTests()) { + if ($this->skipDbInstall) { + return; + } elseif ($this->plugin->hasBehatFeatures() || $this->plugin->hasUnitTests()) { $installers->add(new TestSuiteInstaller($this->moodle, $this->plugin, $this->execute)); } }