Add DiffHelper::getStyleSheet()

jfcherng · jfcherng · commit 5e0f0dae7c2c · 2019-02-26T05:48:43.000+08:00
Signed-off-by: Jack Cherng &lt;jfcherng@gmail.com&gt;
diff --git a/example/demo.php b/example/demo.php
@@ -1,17 +1,20 @@
+<?php
+
+include __DIR__ . '/../vendor/autoload.php';
+
+use Jfcherng\Diff\DiffHelper;
+
+?>
 <!DOCTYPE html>
 <html>
     <head>
         <meta charset="UTF-8" />
         <title>jfcherng/php-diff - Examples</title>
-        <link href="./diff-table.css" rel="stylesheet" type="text/css" />
+        <style><?= DiffHelper::getStyleSheet() ?></style>
     </head>
     <body>
         <?php
 
-            include __DIR__ . '/../vendor/autoload.php';
-
-            use Jfcherng\Diff\DiffHelper;
-
             // include two sample files for comparison
             $oldFilePath = __DIR__ . '/old_file.txt';
             $newFilePath = __DIR__ . '/new_file.txt';
diff --git a/src/DiffHelper.php b/src/DiffHelper.php
@@ -73,6 +73,23 @@ public static function getAvailableTemplates(): array
         return \array_keys(self::getTemplatesInfo());
     }
 
+    /**
+     * Get the content of the CSS style sheet for HTML templates.
+     *
+     * @throws \LogicException   path is a directory
+     * @throws \RuntimeException path cannot be opened
+     *
+     * @return string
+     */
+    public static function getStyleSheet(): string
+    {
+        static $filePath = __DIR__ . '/../example/diff-table.css';
+
+        $cssFile = new \SplFileObject($filePath, 'r');
+
+        return $cssFile->fread($cssFile->getSize());
+    }
+
     /**
      * All-in-one static method to calculate the diff between two strings (or arrays of strings).
      *
diff --git a/tests/DiffHelperTest.php b/tests/DiffHelperTest.php
@@ -85,7 +85,7 @@ public function testCalculateUnified(string $old, string $new, array $expecteds)
     }
 
     /**
-     * Test the DiffHelper::calculate with the'Context' template.
+     * Test the DiffHelper::calculate with the 'Context' template.
      *
      * @covers       \Jfcherng\Diff\DiffHelper::calculate
      * @dataProvider calculateDataProvider
@@ -101,4 +101,14 @@ public function testCalculateContext(string $old, string $new, array $expecteds)
             DiffHelper::calculate($old, $new, 'Context')
         );
     }
+
+    /**
+     * Test the DiffHelper::getStyleSheet.
+     *
+     * @covers \Jfcherng\Diff\DiffHelper::getStyleSheet
+     */
+    public function testGetStyleSheet(): void
+    {
+        $this->assertIsString(DiffHelper::getStyleSheet());
+    }
 }

Original file line number	Diff line number	Diff line change
`@@ -85,7 +85,7 @@ public function testCalculateUnified(string $old, string $new, array $expecteds)`
`85`	`85`	`}`
`86`	`86`
`87`	`87`	`/**`
`88`		`- * Test the DiffHelper::calculate with the'Context' template.`
	`88`	`+ * Test the DiffHelper::calculate with the 'Context' template.`
`89`	`89`	`*`
`90`	`90`	`* @covers \Jfcherng\Diff\DiffHelper::calculate`
`91`	`91`	`* @dataProvider calculateDataProvider`
`@@ -101,4 +101,14 @@ public function testCalculateContext(string $old, string $new, array $expecteds)`
`101`	`101`	`DiffHelper::calculate($old, $new, 'Context')`
`102`	`102`	`);`
`103`	`103`	`}`
	`104`	`+`
	`105`	`+ /**`
	`106`	`+ * Test the DiffHelper::getStyleSheet.`
	`107`	`+ *`
	`108`	`+ * @covers \Jfcherng\Diff\DiffHelper::getStyleSheet`
	`109`	`+ */`
	`110`	`+ public function testGetStyleSheet(): void`
	`111`	`+ {`
	`112`	`+ $this->assertIsString(DiffHelper::getStyleSheet());`
	`113`	`+ }`
`104`	`114`	`}`