From c57dfc631caf98df46ac014812f962863f0bfa61 Mon Sep 17 00:00:00 2001
From: Igor Scheller <igor.scheller@igorshp.de>
Date: Tue, 19 Sep 2017 19:50:59 +0200
Subject: engelsystem_provider should not be loaded in phpunit, changed dir
 structure

---
 phpunit.xml                                    |   9 +-
 test/Logger/EngelsystemLoggerTest.php          | 131 ------------------------
 test/model/LogEntriesModelTest.php             |  33 ------
 test/model/RoomModelTest.php                   |  35 -------
 tests/Feature/Logger/EngelsystemLoggerTest.php | 136 +++++++++++++++++++++++++
 tests/Feature/model/LogEntriesModelTest.php    |  38 +++++++
 tests/Feature/model/RoomModelTest.php          |  40 ++++++++
 7 files changed, 217 insertions(+), 205 deletions(-)
 delete mode 100644 test/Logger/EngelsystemLoggerTest.php
 delete mode 100644 test/model/LogEntriesModelTest.php
 delete mode 100644 test/model/RoomModelTest.php
 create mode 100644 tests/Feature/Logger/EngelsystemLoggerTest.php
 create mode 100644 tests/Feature/model/LogEntriesModelTest.php
 create mode 100644 tests/Feature/model/RoomModelTest.php

diff --git a/phpunit.xml b/phpunit.xml
index ee5ae3e8..e78a54f3 100644
--- a/phpunit.xml
+++ b/phpunit.xml
@@ -1,14 +1,11 @@
 <phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
-         bootstrap="./includes/engelsystem_provider.php"
+         bootstrap="./includes/autoload.php"
          xsi:noNamespaceSchemaLocation="http://schema.phpunit.de/6.3/phpunit.xsd"
          colors="true"
 >
     <testsuites>
-        <testsuite name="Models">
-            <directory>./test/model/</directory>
-        </testsuite>
-        <testsuite name="Logger">
-            <directory>./test/Logger/</directory>
+        <testsuite name="Feature">
+            <directory>./tests/Feature</directory>
         </testsuite>
     </testsuites>
     <filter>
diff --git a/test/Logger/EngelsystemLoggerTest.php b/test/Logger/EngelsystemLoggerTest.php
deleted file mode 100644
index 2219cdb2..00000000
--- a/test/Logger/EngelsystemLoggerTest.php
+++ /dev/null
@@ -1,131 +0,0 @@
-<?php
-
-namespace Engelsystem\Test\Logger;
-
-use Engelsystem\Logger\EngelsystemLogger;
-use PHPUnit\Framework\TestCase;
-use Psr\Log\InvalidArgumentException;
-use Psr\Log\LoggerInterface;
-use Psr\Log\LogLevel;
-
-class EngelsystemLoggerTest extends TestCase
-{
-    /**
-     * @return LoggerInterface
-     */
-    public function getLogger()
-    {
-        return new EngelsystemLogger();
-    }
-
-    public function testImplements()
-    {
-        $this->assertInstanceOf('Psr\Log\LoggerInterface', $this->getLogger());
-    }
-
-    /**
-     * @dataProvider provideLogLevels
-     * @param string $level
-     */
-    public function testAllLevels($level)
-    {
-        $logger = $this->getLogger();
-
-        LogEntries_clear_all();
-
-        $logger->log($level, 'First log message');
-        $logger->{$level}('Second log message');
-
-        $entries = LogEntries();
-        $this->assertCount(2, $entries);
-    }
-
-    /**
-     * @return string[]
-     */
-    public function provideLogLevels()
-    {
-        return [
-            [LogLevel::ALERT],
-            [LogLevel::CRITICAL],
-            [LogLevel::DEBUG],
-            [LogLevel::EMERGENCY],
-            [LogLevel::ERROR],
-            [LogLevel::INFO],
-            [LogLevel::NOTICE],
-            [LogLevel::WARNING],
-        ];
-    }
-
-    public function testContextReplacement()
-    {
-        $logger = $this->getLogger();
-        LogEntries_clear_all();
-
-        $logger->log(LogLevel::INFO, 'My username is {username}', ['username' => 'Foo']);
-
-        $entry = $this->getLastEntry();
-        $this->assertEquals('My username is Foo', $entry['message']);
-        $this->assertEquals(LogLevel::INFO, $entry['level']);
-
-        foreach (
-            [
-                ['Data and {context}', []],
-                ['Data and ', ['context' => null]],
-                ['Data and {context}', ['context' => new \stdClass()]],
-            ] as $data
-        ) {
-            list($result, $context) = $data;
-
-            $logger->log(LogLevel::INFO, 'Data and {context}', $context);
-
-            $entry = $this->getLastEntry();
-            $this->assertEquals($result, $entry['message']);
-        }
-    }
-
-    public function testContextToString()
-    {
-        $logger = $this->getLogger();
-        LogEntries_clear_all();
-
-        $mock = $this->getMockBuilder('someDataProvider')
-            ->setMethods(['__toString'])
-            ->getMock();
-
-        $mock->expects($this->atLeastOnce())
-            ->method('__toString')
-            ->will($this->returnValue('FooBar'));
-
-        $logger->log(LogLevel::INFO, 'Some data and {context}', ['context' => $mock]);
-
-        $entry = $this->getLastEntry();
-        $this->assertEquals('Some data and FooBar', $entry['message']);
-    }
-
-    /**
-     * @expectedException InvalidArgumentException
-     */
-    public function testThrowExceptionOnInvalidLevel()
-    {
-        $logger = $this->getLogger();
-
-        $logger->log('This log level should never be defined', 'Some message');
-    }
-
-    /**
-     * @return array
-     */
-    public function getLastEntry()
-    {
-        $entries = LogEntries();
-        $entry = array_pop($entries);
-
-        return $entry;
-    }
-
-    public function tearDown()
-    {
-        LogEntries_clear_all();
-    }
-}
diff --git a/test/model/LogEntriesModelTest.php b/test/model/LogEntriesModelTest.php
deleted file mode 100644
index 761725c4..00000000
--- a/test/model/LogEntriesModelTest.php
+++ /dev/null
@@ -1,33 +0,0 @@
-<?php
-
-namespace Engelsystem\Test;
-
-use PHPUnit\Framework\TestCase;
-use Psr\Log\LogLevel;
-
-class LogEntriesModelTest extends TestCase
-{
-    public function testCreateLogEntry()
-    {
-        LogEntries_clear_all();
-        $count = count(LogEntries());
-        $this->assertNotFalse(LogEntry_create(LogLevel::WARNING, 'test_LogEntry_create'));
-
-        // There should be one more log entry now
-        $this->assertEquals(count(LogEntries()), $count + 1);
-    }
-
-    public function testClearAllLogEntries()
-    {
-        LogEntry_create(LogLevel::WARNING, 'test');
-        $this->assertTrue(count(LogEntries()) > 0);
-
-        $this->assertNotFalse(LogEntries_clear_all());
-        $this->assertCount(0, LogEntries());
-    }
-
-    public function tearDown()
-    {
-        LogEntries_clear_all();
-    }
-}
diff --git a/test/model/RoomModelTest.php b/test/model/RoomModelTest.php
deleted file mode 100644
index 4205845b..00000000
--- a/test/model/RoomModelTest.php
+++ /dev/null
@@ -1,35 +0,0 @@
-<?php
-
-namespace Engelsystem\Test;
-
-use PHPUnit\Framework\TestCase;
-
-class RoomModelTest extends TestCase
-{
-    private $room_id = null;
-
-    public function create_Room()
-    {
-        $this->room_id = Room_create('test', false, true, '');
-    }
-
-    public function test_Room()
-    {
-        $this->create_Room();
-
-        $room = Room($this->room_id);
-
-        $this->assertNotFalse($room);
-        $this->assertNotNull($room);
-        $this->assertEquals($room['Name'], 'test');
-
-        $this->assertNull(Room(-1));
-    }
-
-    public function tearDown()
-    {
-        if ($this->room_id != null) {
-            Room_delete($this->room_id);
-        }
-    }
-}
diff --git a/tests/Feature/Logger/EngelsystemLoggerTest.php b/tests/Feature/Logger/EngelsystemLoggerTest.php
new file mode 100644
index 00000000..3b6572f5
--- /dev/null
+++ b/tests/Feature/Logger/EngelsystemLoggerTest.php
@@ -0,0 +1,136 @@
+<?php
+
+namespace Engelsystem\Test\Logger;
+
+use Engelsystem\Logger\EngelsystemLogger;
+use PHPUnit\Framework\TestCase;
+use Psr\Log\InvalidArgumentException;
+use Psr\Log\LoggerInterface;
+use Psr\Log\LogLevel;
+
+class EngelsystemLoggerTest extends TestCase
+{
+    public static function setUpBeforeClass()
+    {
+        require_once __DIR__ . '/../../../includes/engelsystem_provider.php';
+    }
+
+    /**
+     * @return LoggerInterface
+     */
+    public function getLogger()
+    {
+        return new EngelsystemLogger();
+    }
+
+    public function testImplements()
+    {
+        $this->assertInstanceOf('Psr\Log\LoggerInterface', $this->getLogger());
+    }
+
+    /**
+     * @dataProvider provideLogLevels
+     * @param string $level
+     */
+    public function testAllLevels($level)
+    {
+        $logger = $this->getLogger();
+
+        LogEntries_clear_all();
+
+        $logger->log($level, 'First log message');
+        $logger->{$level}('Second log message');
+
+        $entries = LogEntries();
+        $this->assertCount(2, $entries);
+    }
+
+    /**
+     * @return string[]
+     */
+    public function provideLogLevels()
+    {
+        return [
+            [LogLevel::ALERT],
+            [LogLevel::CRITICAL],
+            [LogLevel::DEBUG],
+            [LogLevel::EMERGENCY],
+            [LogLevel::ERROR],
+            [LogLevel::INFO],
+            [LogLevel::NOTICE],
+            [LogLevel::WARNING],
+        ];
+    }
+
+    public function testContextReplacement()
+    {
+        $logger = $this->getLogger();
+        LogEntries_clear_all();
+
+        $logger->log(LogLevel::INFO, 'My username is {username}', ['username' => 'Foo']);
+
+        $entry = $this->getLastEntry();
+        $this->assertEquals('My username is Foo', $entry['message']);
+        $this->assertEquals(LogLevel::INFO, $entry['level']);
+
+        foreach (
+            [
+                ['Data and {context}', []],
+                ['Data and ', ['context' => null]],
+                ['Data and {context}', ['context' => new \stdClass()]],
+            ] as $data
+        ) {
+            list($result, $context) = $data;
+
+            $logger->log(LogLevel::INFO, 'Data and {context}', $context);
+
+            $entry = $this->getLastEntry();
+            $this->assertEquals($result, $entry['message']);
+        }
+    }
+
+    public function testContextToString()
+    {
+        $logger = $this->getLogger();
+        LogEntries_clear_all();
+
+        $mock = $this->getMockBuilder('someDataProvider')
+            ->setMethods(['__toString'])
+            ->getMock();
+
+        $mock->expects($this->atLeastOnce())
+            ->method('__toString')
+            ->will($this->returnValue('FooBar'));
+
+        $logger->log(LogLevel::INFO, 'Some data and {context}', ['context' => $mock]);
+
+        $entry = $this->getLastEntry();
+        $this->assertEquals('Some data and FooBar', $entry['message']);
+    }
+
+    /**
+     * @expectedException InvalidArgumentException
+     */
+    public function testThrowExceptionOnInvalidLevel()
+    {
+        $logger = $this->getLogger();
+
+        $logger->log('This log level should never be defined', 'Some message');
+    }
+
+    /**
+     * @return array
+     */
+    public function getLastEntry()
+    {
+        $entries = LogEntries();
+        $entry = array_pop($entries);
+
+        return $entry;
+    }
+
+    public function tearDown()
+    {
+        LogEntries_clear_all();
+    }
+}
diff --git a/tests/Feature/model/LogEntriesModelTest.php b/tests/Feature/model/LogEntriesModelTest.php
new file mode 100644
index 00000000..6d7b0ebc
--- /dev/null
+++ b/tests/Feature/model/LogEntriesModelTest.php
@@ -0,0 +1,38 @@
+<?php
+
+namespace Engelsystem\Test;
+
+use PHPUnit\Framework\TestCase;
+use Psr\Log\LogLevel;
+
+class LogEntriesModelTest extends TestCase
+{
+    public static function setUpBeforeClass()
+    {
+        require_once __DIR__ . '/../../../includes/engelsystem_provider.php';
+    }
+
+    public function testCreateLogEntry()
+    {
+        LogEntries_clear_all();
+        $count = count(LogEntries());
+        $this->assertNotFalse(LogEntry_create(LogLevel::WARNING, 'test_LogEntry_create'));
+
+        // There should be one more log entry now
+        $this->assertEquals(count(LogEntries()), $count + 1);
+    }
+
+    public function testClearAllLogEntries()
+    {
+        LogEntry_create(LogLevel::WARNING, 'test');
+        $this->assertTrue(count(LogEntries()) > 0);
+
+        $this->assertNotFalse(LogEntries_clear_all());
+        $this->assertCount(0, LogEntries());
+    }
+
+    public function tearDown()
+    {
+        LogEntries_clear_all();
+    }
+}
diff --git a/tests/Feature/model/RoomModelTest.php b/tests/Feature/model/RoomModelTest.php
new file mode 100644
index 00000000..96be84a2
--- /dev/null
+++ b/tests/Feature/model/RoomModelTest.php
@@ -0,0 +1,40 @@
+<?php
+
+namespace Engelsystem\Test;
+
+use PHPUnit\Framework\TestCase;
+
+class RoomModelTest extends TestCase
+{
+    private $room_id = null;
+
+    public static function setUpBeforeClass()
+    {
+        require_once __DIR__ . '/../../../includes/engelsystem_provider.php';
+    }
+
+    public function create_Room()
+    {
+        $this->room_id = Room_create('test', false, true, '');
+    }
+
+    public function test_Room()
+    {
+        $this->create_Room();
+
+        $room = Room($this->room_id);
+
+        $this->assertNotFalse($room);
+        $this->assertNotNull($room);
+        $this->assertEquals($room['Name'], 'test');
+
+        $this->assertNull(Room(-1));
+    }
+
+    public function tearDown()
+    {
+        if ($this->room_id != null) {
+            Room_delete($this->room_id);
+        }
+    }
+}
-- 
cgit v1.2.3-54-g00ecf