Merge pull request #40 from xefi/issue-39

Closes #39 - Bad namespace in the generated mixin
xefi · Nov 28, 2024 · 9f97fcc · 9f97fcc
2 parents 82ce3c0 + 2468f93
commit 9f97fcc
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 1 deletion.
diff --git a/src/Manifests/ContainerMixinManifest.php b/src/Manifests/ContainerMixinManifest.php
@@ -161,7 +161,7 @@ protected function write(string $docComment): void
 
         file_put_contents(
             $this->containerMixinPath,
-            "<?php namespace Xefi\\Faker\\Container{\n{$docComment}\n\tclass ContainerMixin{}"
+            "<?php namespace Xefi\\Faker\\Container;\n{$docComment}\n\tclass ContainerMixin{}"
         );
     }
 }
diff --git a/tests/Unit/ContainerMixinManifestTest.php b/tests/Unit/ContainerMixinManifestTest.php
@@ -61,6 +61,25 @@ public function testContainerMixinBuild()
         unlink('/tmp/ContainerMixin.php');
     }
 
+    public function testContainerMixinNamespace()
+    {
+        @unlink('/tmp/ContainerMixin.php');
+        $container = new \Xefi\Faker\Container\Container(shouldBuildContainerMixin: false);
+        $manifest = new \Xefi\Faker\Manifests\ContainerMixinManifest(__DIR__.'/../Support', '/tmp/ContainerMixin.php');
+        $manifest->build($container->getExtensionMethods(), $container->getExtensions());
+
+        $this->assertFileExists('/tmp/ContainerMixin.php');
+
+        $containerMixinContent = file_get_contents('/tmp/ContainerMixin.php');
+
+        $this->assertStringContainsString(
+            'namespace Xefi\\Faker\\Container;',
+            $containerMixinContent
+        );
+
+        unlink('/tmp/ContainerMixin.php');
+    }
+
     public function testShouldRecompile()
     {
         @unlink('/tmp/ContainerMixin.php');