Add tests

bmitch · Mar 3, 2022 · a31bad5 · a31bad5
1 parent 59d55f3
commit a31bad5
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 2 deletions.
diff --git a/src/Command/AssessComplexityCommand.php b/src/Command/AssessComplexityCommand.php
@@ -34,7 +34,9 @@ protected function configure(): void
     protected function execute(InputInterface $input, OutputInterface $output): int
     {
         $file = (string) $input->getArgument('file');
-        $contents = \file_get_contents($file);
+        $contents = \is_file($file)
+            ? \file_get_contents($file)
+            : false;
 
         if (false === $contents) {
             $output->writeln('0');

diff --git a/tests/Integration/Command/AssessComplexityCommandTest.php b/tests/Integration/Command/AssessComplexityCommandTest.php
@@ -28,7 +28,7 @@ protected function tearDown()
     }
 
     /** @test */
-    public function it_returns_the_cyclomatic_complexity()
+    public function it_returns_the_cyclomatic_complexity_greater_than_zero()
     {
         $exitCode = $this->commandTester->execute(['file' => __FILE__]);
         $result = \rtrim($this->commandTester->getDisplay());
@@ -37,4 +37,15 @@ public function it_returns_the_cyclomatic_complexity()
         $this->assertTrue(\ctype_digit($result), 'The result of the command must be an integer');
         $this->assertGreaterThan(0, (int) $result);
     }
+
+    /** @test */
+    public function it_returns_zero_for_non_existing_file()
+    {
+        $exitCode = $this->commandTester->execute(['file' => 'nonexisting-file.php']);
+        $result = \rtrim($this->commandTester->getDisplay());
+
+        $this->assertEquals(0, $exitCode);
+        $this->assertTrue(\ctype_digit($result), 'The result of the command must be an integer');
+        $this->assertEquals(0, (int) $result);
+    }
 }