Merge pull request #222 from civicrm/master-sql

cv sql - Fix recent regression. Add sniff test.
civicrm · Sep 28, 2024 · 7f295ea · 7f295ea
2 parents 7e7b22b + c8d80a5
commit 7f295ea
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 0 deletions.
diff --git a/src/Command/SqlCliCommand.php b/src/Command/SqlCliCommand.php
@@ -41,6 +41,7 @@ protected function initialize(InputInterface $input, OutputInterface $output) {
     if ($input->getOption('dry-run') && $output->getVerbosity() <= OutputInterface::VERBOSITY_NORMAL) {
       $output->setVerbosity(OutputInterface::VERBOSITY_VERBOSE);
     }
+    parent::initialize($input, $output);
   }
 
   protected function execute(InputInterface $input, OutputInterface $output): int {

diff --git a/tests/Command/SqlCliCommandTest.php b/tests/Command/SqlCliCommandTest.php
@@ -0,0 +1,21 @@
+<?php
+namespace Civi\Cv\Command;
+
+use Civi\Cv\Util\Process;
+
+/**
+ * @group std
+ */
+class SqlCliCommandTest extends \Civi\Cv\CivilTestCase {
+
+  public function setUp(): void {
+    parent::setUp();
+  }
+
+  public function testSqlPipe() {
+    $query = 'select id, display_name from civicrm_contact limit 1;';
+    $p = Process::runOk($this->cv("sql")->setInput($query));
+    $this->assertMatchesRegularExpression('/id\s+display_name/', $p->getOutput());
+  }
+
+}