Fixed a bug regarding the replace_mixins option (See #99)

Compiler.php

@@ -1257,6 +1257,8 @@ protected function handleMixins(Node $node)
                     "Duplicate mixin name $mixinNode->name",
                     $mixinNode
                 );
+            else if (isset($this->mixins[$mixinNode->name]))
+                $this->mixins[$mixinNode->name]->parent->remove($this->mixins[$mixinNode->name]);
 
             $this->mixins[$mixinNode->name] = $mixinNode;
         }

Test/MixinTest.php

@@ -2,6 +2,7 @@
 
 namespace Tale\Test\Jade;
 
+use Tale\Jade\Compiler\Exception;
 use Tale\Jade\Renderer;
 
 class MixinTest extends \PHPUnit_Framework_TestCase
@@ -82,4 +83,25 @@ public function testScoping()
 
         $this->assertEquals('<h1>Parent Mixin</h1><div class="parent-block"><h2>Child Mixin</h2><div class="child-block"><span>This is test content</span><span>This content is fed from outside</span></div></div>', $this->renderer->render('scoping', ['fedFromOutside' => 'This content is fed from outside']));
     }
+
+    public function testDuplicateWithoutOverwrite()
+    {
+
+        $this->setExpectedException(Exception::class);
+        $this->renderer->render('duplicate');
+    }
+
+    public function testDuplicateWithOverwrite()
+    {
+
+        $this->renderer->getCompiler()->setOption('replace_mixins', true);
+        $this->assertEquals('<p>Testing</p>', $this->renderer->render('duplicate'));
+    }
+
+    public function testDuplicateInInclude()
+    {
+
+        $this->renderer->getCompiler()->setOption('replace_mixins', true);
+        $this->assertEquals('<p>Testing</p>', $this->renderer->render('duplicate-in-include'));
+    }
 }

Test/views/mixins/duplicate-in-include.jade

@@ -0,0 +1,4 @@
+
+mixin test
+
+include duplicate

Test/views/mixins/duplicate.jade

@@ -0,0 +1,9 @@
+
+mixin test
+    block
+
+mixin test
+    block
+
++test
+    p Testing