From 7e071e8abc2e62ca3c803b7f91de66f190a198e1 Mon Sep 17 00:00:00 2001
From: oleibman <eclipsechasers2@yahoo.com>
Date: Sun, 14 Mar 2021 12:04:07 -0700
Subject: [PATCH] Coverage for Helper/Samples (#1920)

* Coverage for Helper/Samples

I was perplexed by the fact that Helper/Samples seemed to be entirely uncovered when running the test suite, since I know all the samples are run as part of the test. I think that what must be happening is that the Helper code is invoked mostly as part of a Data Provider (and therefore not counted), not as part of the test proper (which would count). So, this change adds a small number of tests which result in Samples being 100% covered.

Covering one statement was tricky - simulating the inability to create a test directory. Mocking, a technique I have not used before, solves this problem admirably.

* Suggestions From Mark Baker

Tests changed from assertEquals to assertSame.

Added @covers annotation to test class.

Validate parameter for method being mocked.
---
 src/PhpSpreadsheet/Helper/Sample.php          | 13 ++++---
 .../Helper/SampleCoverageTest.php             | 39 +++++++++++++++++++
 2 files changed, 47 insertions(+), 5 deletions(-)
 create mode 100644 tests/PhpSpreadsheetTests/Helper/SampleCoverageTest.php

diff --git a/src/PhpSpreadsheet/Helper/Sample.php b/src/PhpSpreadsheet/Helper/Sample.php
index a91b195e..c84c3930 100644
--- a/src/PhpSpreadsheet/Helper/Sample.php
+++ b/src/PhpSpreadsheet/Helper/Sample.php
@@ -71,7 +71,7 @@ class Sample
     /**
      * Returns an array of all known samples.
      *
-     * @return string[] [$name => $path]
+     * @return string[][] [$name => $path]
      */
     public function getSamples()
     {
@@ -132,6 +132,11 @@ class Sample
         $this->logEndingNotes();
     }
 
+    protected function isDirOrMkdir(string $folder): bool
+    {
+        return \is_dir($folder) || \mkdir($folder);
+    }
+
     /**
      * Returns the temporary directory and make sure it exists.
      *
@@ -140,10 +145,8 @@ class Sample
     private function getTemporaryFolder()
     {
         $tempFolder = sys_get_temp_dir() . '/phpspreadsheet';
-        if (!is_dir($tempFolder)) {
-            if (!mkdir($tempFolder) && !is_dir($tempFolder)) {
-                throw new RuntimeException(sprintf('Directory "%s" was not created', $tempFolder));
-            }
+        if (!$this->isDirOrMkdir($tempFolder)) {
+            throw new RuntimeException(sprintf('Directory "%s" was not created', $tempFolder));
         }
 
         return $tempFolder;
diff --git a/tests/PhpSpreadsheetTests/Helper/SampleCoverageTest.php b/tests/PhpSpreadsheetTests/Helper/SampleCoverageTest.php
new file mode 100644
index 00000000..f3f616fe
--- /dev/null
+++ b/tests/PhpSpreadsheetTests/Helper/SampleCoverageTest.php
@@ -0,0 +1,39 @@
+<?php
+
+namespace PhpOffice\PhpSpreadsheetTests\Helper;
+
+use PhpOffice\PhpSpreadsheet\Helper\Sample;
+use PHPUnit\Framework\TestCase;
+use RuntimeException;
+
+/**
+ * @covers \PhpOffice\PhpSpreadsheet\Helper\Sample
+ */
+class SampleCoverageTest extends TestCase
+{
+    public function testSample(): void
+    {
+        $helper = new Sample();
+        $samples = $helper->getSamples();
+        self::assertArrayHasKey('Basic', $samples);
+        $basic = $samples['Basic'];
+        self::assertArrayHasKey('02 Types', $basic);
+        self::assertSame('Basic/02_Types.php', $basic['02 Types']);
+        self::assertSame('phpunit', $helper->getPageTitle());
+        self::assertSame('<h1>phpunit</h1>', $helper->getPageHeading());
+    }
+
+    public function testDirectoryFail(): void
+    {
+        $this->expectException(RuntimeException::class);
+
+        $helper = $this->getMockBuilder(Sample::class)
+            ->onlyMethods(['isDirOrMkdir'])
+            ->getMock();
+        $helper->expects(self::once())
+            ->method('isDirOrMkdir')
+            ->with(self::isType('string'))
+            ->willReturn(false);
+        self::assertSame('', $helper->getFilename('a.xlsx'));
+    }
+}