Merge pull request #23 from tattersoftware/navbar

Release

Author: MGatner

src/Views/Templates/form.php

@@ -1,5 +1,10 @@
-<?= $this->extend(config('Layouts')->outbox) ?>
-<?= $this->section('main') ?>
+<?php $this->extend(config('Layouts')->outbox) ?>
+<?php $this->section('navbar') ?>
+
+	<?= view('Tatter\Outbox\Views\navbar') ?>
+
+<?php $this->endSection() ?>
+<?php $this->section('main') ?>
 
 	<div class="row">
 		<div class="col">
@@ -39,4 +44,4 @@
 		</div>
 	</div>
 
-<?= $this->endSection() ?>
+<?php $this->endSection() ?>

src/Views/Templates/index.php

@@ -1,5 +1,10 @@
-<?= $this->extend(config('Layouts')->outbox) ?>
-<?= $this->section('main') ?>
+<?php $this->extend(config('Layouts')->outbox) ?>
+<?php $this->section('navbar') ?>
+
+	<?= view('Tatter\Outbox\Views\navbar') ?>
+
+<?php $this->endSection() ?>
+<?php $this->section('main') ?>
 
 	<div class="row">
 		<div class="col">
@@ -47,4 +52,4 @@
 		</div>
 	</div>
 
-<?= $this->endSection() ?>
+<?php $this->endSection() ?>

src/Views/Templates/send.php

@@ -1,5 +1,10 @@
-<?= $this->extend(config('Layouts')->outbox) ?>
-<?= $this->section('main') ?>
+<?php $this->extend(config('Layouts')->outbox) ?>
+<?php $this->section('navbar') ?>
+
+	<?= view('Tatter\Outbox\Views\navbar') ?>
+
+<?php $this->endSection() ?>
+<?php $this->section('main') ?>
 
 	<div class="row">
 		<div class="col">
@@ -71,4 +76,4 @@ class="form-control"
 		</div>
 	</div>
 
-<?= $this->endSection() ?>
+<?php $this->endSection() ?>

tests/unit/ControllerTest.php

@@ -0,0 +1,68 @@
+<?php
+
+use CodeIgniter\Exceptions\PageNotFoundException;
+use CodeIgniter\Test\ControllerTestTrait;
+use CodeIgniter\Test\DatabaseTestTrait;
+use Tatter\Outbox\Controllers\Templates;
+use Tatter\Outbox\Entities\Template;
+use Tatter\Outbox\Models\TemplateModel;
+use Tests\Support\OutboxTestCase;
+
+/**
+ * @internal
+ */
+final class ControllerTest extends OutboxTestCase
+{
+    use ControllerTestTrait;
+    use DatabaseTestTrait;
+
+    /**
+     * Our Controller set by the trait
+     *
+     * @var Templates|null
+     */
+    protected $controller;
+
+    /**
+     * A test Template
+     *
+     * @var Template
+     */
+    private $template;
+
+    protected function setUp(): void
+    {
+        parent::setUp();
+
+        $this->controller(Templates::class);
+
+        $this->template = new Template([
+            'name'    => 'Test Template',
+            'subject' => 'Some {subject}',
+            'body'    => '<p>{number}</p>',
+        ]);
+        $this->template->id = model(TemplateModel::class)->insert($this->template);
+    }
+
+    public function testGetTemplate()
+    {
+        $result = $this->controller->getTemplate($this->template->id);
+
+        $this->assertSame($this->template->name, $result->name);
+    }
+
+    public function testGetTemplateThrows()
+    {
+        $this->expectException(PageNotFoundException::class);
+
+        $this->controller->getTemplate(42);
+    }
+
+    public function testIndexListsTemplates()
+    {
+        $result = $this->execute('index');
+
+        $result->assertStatus(200);
+        $result->assertSee($this->template->name);
+    }
+}