updates tests

src/writer/AmazonSesWriter.php

@@ -13,19 +13,18 @@
  */
 class AmazonSesWriter extends AbstractEmailWriter
 {
+   /**
+    * {@inheritdoc}
+    */
    public function write()
-   {
-      $client = $this->getSesClient();
-      $options = $this->getEmailOptions();
-      $result = $client->sendEmail($options);
-   }
-
-   private function getSesClient()
    {
       return SesClient::factory([
          "profile" => Config::getRequired("amazon.aws.ses.profile"),
          "region"  => Config::getRequired("amazon.aws.ses.region")
       ]);
+
+      $options = $this->getEmailOptions();
+      $this->sendEmail($client, $options);
    }
 
    private function getEmailOptions()
@@ -87,5 +86,10 @@ private function getEmailOptions()
          "ReturnPath" => $this->getReturnPath()
       ];
    }
+
+   private function sendEmail($client, $options)
+   {
+      $result = $client->sendEmail($options);
+   }
 }
 

src/writer/FileWriter.php

@@ -3,10 +3,9 @@
 namespace sndsgd\log\writer;
 
 use \Exception;
-use \sndsgd\log\FileInterface;
-use \sndsgd\log\Record;
 use \sndsgd\Config;
-use \sndsgd\Path;
+use \sndsgd\log\File;
+use \sndsgd\log\Record;
 
 
 /**

tests/FileTest.php

@@ -0,0 +1,28 @@
+<?php
+
+namespace sndsgd\log;
+
+use \sndsgd\Config;
+
+
+class FileTest extends \PHPUnit_Framework_TestCase
+{
+
+   public function testCreateFromLogName()
+   {
+      Config::init();
+
+      $name = "test";
+      $expect = File::DEFAULT_DIR."/$name.log";
+      $file = File::createFromLogName($name);
+      $this->assertEquals($expect, $file->getPath());
+
+      Config::set("sndsgd.log.file.dir", __DIR__);
+      $expect = __DIR__."/$name.log";
+      $file = File::createFromLogName($name);
+      $this->assertEquals($expect, $file->getPath());
+
+      Config::init();
+   }
+}
+

tests/RecordTest.php

@@ -14,18 +14,18 @@ public function setUp()
 
    public function testCreate()
    {
-      $name = 'test';
-      $message = 'test message';
+      $name = "test";
+      $message = "test message";
       $r = Record::create($name, $message);
-      $this->assertInstanceOf('sndsgd\\log\\Record', $r);
+      $this->assertInstanceOf("sndsgd\\log\\Record", $r);
       $this->assertEquals($name, $r->getName());
       $this->assertEquals($message, $r->getMessage());
    }
 
    public function testSetGetName()
    {
-      $this->r->setName('test');
-      $this->assertEquals('test', $this->r->getName());
+      $this->r->setName("test");
+      $this->assertEquals("test", $this->r->getName());
    }
 
    /**
@@ -41,7 +41,7 @@ public function testSetNameException()
     */
    public function testSetNameRegexException()
    {
-      $this->r->setName('asdf 09u132487y9 07 o&YI*(&ty');
+      $this->r->setName("asdf 09u132487y9 07 o&YI*(&ty");
    }
 
    public function testDateAndTimestamp()
@@ -50,13 +50,13 @@ public function testDateAndTimestamp()
       $this->assertTrue(is_float($timestamp));
       $this->assertTrue($timestamp < microtime(true));
 
-      $date = $this->r->getDate();
-      $this->assertEquals(date('r', floor($timestamp)), $date);
+      $date = $this->r->getDate("r");
+      $this->assertEquals(date("r", $timestamp), $date);
    }
 
    public function testSetAndGetMessage()
    {
-      $this->assertEquals('', $this->r->getMessage());
+      $this->assertEquals("", $this->r->getMessage());
       $this->r->setMessage("test \n  test ");
       $this->assertEquals("test test", $this->r->getMessage());
 
@@ -76,22 +76,22 @@ public function testData()
    {
       $this->assertEquals([], $this->r->getData());
 
-      $this->r->addData('one', 1);
-      $this->assertEquals(['one' => 1], $this->r->getData());
+      $this->r->addData("one", 1);
+      $this->assertEquals(["one" => 1], $this->r->getData());
 
       $this->r->addData([
-         'two' => 2,
-         'three' => 3
+         "two" => 2,
+         "three" => 3
       ]);
-      $this->assertEquals(['one'=>1,'two'=>2,'three'=>3], $this->r->getData());
+      $this->assertEquals(["one"=>1,"two"=>2,"three"=>3], $this->r->getData());
    }
 
    /**
     * @expectedException InvalidArgumentException
     */
    public function testWriteInvalidStringArgument()
    {
-      $this->r->write('invalid');
+      $this->r->write("invalid");
    }
 
    /**