Strip trailing slash from path

Fixes #79
blade-ui-kit · Sep 4, 2020 · 91dc18f · 91dc18f
1 parent b47ad40
commit 91dc18f
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 0 deletions.
diff --git a/src/Factory.php b/src/Factory.php
@@ -54,6 +54,8 @@ public function add(string $set, array $options): self
             throw CannotRegisterIconSet::prefixNotUnique($set, $collidingSet);
         }
 
+        $options['path'] = rtrim($options['path'], '/');
+
         if ($this->filesystem->missing($options['path'])) {
             throw CannotRegisterIconSet::nonExistingPath($set, $options['path']);
         }

diff --git a/tests/FactoryTest.php b/tests/FactoryTest.php
@@ -249,6 +249,19 @@ public function it_throws_an_exception_when_no_icon_is_found()
         $factory->svg('money');
     }
 
+    /** @test */
+    public function it_trims_the_trailing_slash_from_the_path()
+    {
+        $factory = $this->prepareSets();
+
+        $factory->add('default', [
+            'path' => __DIR__ . '/resources/svg/',
+            'prefix' => '',
+        ]);
+
+        $this->assertSame(__DIR__ . '/resources/svg', $factory->all()['default']['path']);
+    }
+
     protected function getPackageProviders($app): array
     {
         return [BladeIconsServiceProvider::class];