Add direction tests

The-3Labs-Team · Mar 2, 2024 · bc17155 · bc17155
1 parent 1124688
commit bc17155
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 4 deletions.
diff --git a/src/Readability.php b/src/Readability.php
@@ -23,8 +23,8 @@ public function parse(string $content): self
             $this->content->parse($content);
         } catch (ParseException $e) {
             $this->content = null;
-            error_log('Cannot parse: '.$e->getMessage());
-            throw new Exception('Cannot parse: '.$e->getMessage());
+            error_log('Cannot parse: ' . $e->getMessage());
+            throw new Exception('Cannot parse: ' . $e->getMessage());
         }
 
         return $this;
@@ -110,7 +110,6 @@ public function getContent(): string
     public function getDirection(): string
     {
         $this->checkContent();
-
         return $this->content->getDirection();
     }
 

diff --git a/tests/ReadabilityTest.php b/tests/ReadabilityTest.php
@@ -48,6 +48,12 @@
         ->and($images)->toContain('/static/img/bitcoin/privacy.svg');
 });
 
+it('can parse and get the direction', function () {
+    $html = file_get_contents(__DIR__.'/fixtures/demo.html');
+    $direction = Readability::parse($html)->getDirection();
+    expect($direction)->toBe('ltr');
+});
+
 it('can parse and get the content', function () {
     $html = file_get_contents(__DIR__.'/fixtures/demo.html');
     $readability = Readability::parse($html);

diff --git a/tests/fixtures/demo.html b/tests/fixtures/demo.html
@@ -119,7 +119,7 @@ <h4>October 31, 2008</h4>
 
 
         </div>
-        <div id="CONTENT" class="doc-content"><h2 id="abstract"><a href="#abstract">Abstract</a></h2>
+        <div id="CONTENT" class="doc-content" dir="ltr"><h2 id="abstract"><a href="#abstract">Abstract</a></h2>
 
             <p>A purely peer-to-peer version of electronic cash would allow online payments to be sent directly from one party to another without going through a financial institution.