Merge pull request #15 from lyrixx/feat-admin-logger

Alexandre Salomé · Alexandre Salomé · commit d2b4b06b0f32 · 2013-01-17T08:47:04.000-08:00
Feat admin logger
diff --git a/src/Gitonomy/Git/Admin.php b/src/Gitonomy/Git/Admin.php
@@ -12,6 +12,7 @@
 
 namespace Gitonomy\Git;
 
+use Psr\Log\LoggerInterface;
 use Symfony\Component\Process\ProcessBuilder;
 
 /**
@@ -31,7 +32,7 @@ class Admin
      *
      * @throws RuntimeException Directory exists or not writable
      */
-    public static function init($path, $bare = true)
+    public static function init($path, $bare = true, LoggerInterface $logger = null)
     {
         $builder = ProcessBuilder::create(array('git', 'init', '-q'));
 
@@ -48,6 +49,6 @@ public static function init($path, $bare = true)
             throw new \RuntimeException(sprintf('Error while initializing repository: %s', $process->getErrorOutput()));
         }
 
-        return new Repository($path);
+        return new Repository($path, null, $logger);
     }
 }
diff --git a/tests/Gitonomy/Git/Tests/AbstractTest.php b/tests/Gitonomy/Git/Tests/AbstractTest.php
@@ -15,7 +15,7 @@
 use Gitonomy\Git\Admin;
 use Gitonomy\Git\Repository;
 
-abstract class AbtractTest extends \PHPUnit_Framework_TestCase
+abstract class AbstractTest extends \PHPUnit_Framework_TestCase
 {
     private static $libRepo;
     private static $testRepo;
diff --git a/tests/Gitonomy/Git/Tests/AdminTest.php b/tests/Gitonomy/Git/Tests/AdminTest.php
@@ -14,7 +14,7 @@
 
 use Gitonomy\Git\Admin;
 
-class AdminTest extends AbtractTest
+class AdminTest extends AbstractTest
 {
     private $tmpDir;
 
diff --git a/tests/Gitonomy/Git/Tests/BlameTest.php b/tests/Gitonomy/Git/Tests/BlameTest.php
@@ -12,7 +12,7 @@
 
 namespace Gitonomy\Git\Tests;
 
-class BlameTest extends AbtractTest
+class BlameTest extends AbstractTest
 {
     public function testBlame()
     {
diff --git a/tests/Gitonomy/Git/Tests/BlobTest.php b/tests/Gitonomy/Git/Tests/BlobTest.php
@@ -14,7 +14,7 @@
 
 use Gitonomy\Git\Blob;
 
-class BlobTest extends AbtractTest
+class BlobTest extends AbstractTest
 {
     public function testGetContent()
     {
diff --git a/tests/Gitonomy/Git/Tests/CommitTest.php b/tests/Gitonomy/Git/Tests/CommitTest.php
@@ -15,7 +15,7 @@
 use Gitonomy\Git\Commit;
 use Gitonomy\Git\Diff;
 
-class CommitTest extends AbtractTest
+class CommitTest extends AbstractTest
 {
     public function testGetDiff()
     {
diff --git a/tests/Gitonomy/Git/Tests/DiffTest.php b/tests/Gitonomy/Git/Tests/DiffTest.php
@@ -14,7 +14,7 @@
 
 use Gitonomy\Git\Diff;
 
-class DiffTest extends AbtractTest
+class DiffTest extends AbstractTest
 {
     public function testGetRevisions()
     {
diff --git a/tests/Gitonomy/Git/Tests/HooksTest.php b/tests/Gitonomy/Git/Tests/HooksTest.php
@@ -14,7 +14,7 @@
 
 use Gitonomy\Git\Admin;
 
-class HooksTest extends AbtractTest
+class HooksTest extends AbstractTest
 {
     private $tmpDir;
     private $hooksDir;
diff --git a/tests/Gitonomy/Git/Tests/LogTest.php b/tests/Gitonomy/Git/Tests/LogTest.php
@@ -14,7 +14,7 @@
 
 use Gitonomy\Git\Log;
 
-class LogTest extends AbtractTest
+class LogTest extends AbstractTest
 {
     public function testRevisionAndPath()
     {
diff --git a/tests/Gitonomy/Git/Tests/PushReferenceTest.php b/tests/Gitonomy/Git/Tests/PushReferenceTest.php
@@ -14,7 +14,7 @@
 
 use Gitonomy\Git\PushReference;
 
-class PushReferenceTest extends AbtractTest
+class PushReferenceTest extends AbstractTest
 {
     const CREATE = 1;
     const DELETE = 2;
diff --git a/tests/Gitonomy/Git/Tests/ReferenceTest.php b/tests/Gitonomy/Git/Tests/ReferenceTest.php
@@ -15,7 +15,7 @@
 use Gitonomy\Git\Reference\Branch;
 use Gitonomy\Git\Reference\Tag;
 
-class ReferenceTest extends AbtractTest
+class ReferenceTest extends AbstractTest
 {
     private $references;
 
diff --git a/tests/Gitonomy/Git/Tests/RepositoryTest.php b/tests/Gitonomy/Git/Tests/RepositoryTest.php
@@ -18,7 +18,7 @@
 use Gitonomy\Git\Event\PreCommandEvent;
 use Gitonomy\Git\Event\PostCommandEvent;
 
-class RepositoryTest extends AbtractTest
+class RepositoryTest extends AbstractTest
 {
     public function testGetBlob_WithExisting_Works()
     {
diff --git a/tests/Gitonomy/Git/Tests/RevisionTest.php b/tests/Gitonomy/Git/Tests/RevisionTest.php
@@ -16,7 +16,7 @@
 use Gitonomy\Git\Log;
 use Gitonomy\Git\Revision;
 
-class RevisionTest extends AbtractTest
+class RevisionTest extends AbstractTest
 {
     public function testGetCommit()
     {
diff --git a/tests/Gitonomy/Git/Tests/TreeTest.php b/tests/Gitonomy/Git/Tests/TreeTest.php
@@ -14,7 +14,7 @@
 
 use Gitonomy\Git\Blob;
 
-class TreeTest extends AbtractTest
+class TreeTest extends AbstractTest
 {
     public function testCase()
     {
diff --git a/tests/Gitonomy/Git/Tests/WorkingCopyTest.php b/tests/Gitonomy/Git/Tests/WorkingCopyTest.php
@@ -15,7 +15,7 @@
 use Gitonomy\Git\Admin;
 use Gitonomy\Git\Reference\Branch;
 
-class WorkingCopyTest extends AbtractTest
+class WorkingCopyTest extends AbstractTest
 {
     protected $tempDir;
     protected $repo;

Original file line number	Diff line number	Diff line change
`@@ -12,6 +12,7 @@`
`12`	`12`
`13`	`13`	`namespace Gitonomy\Git;`
`14`	`14`
	`15`	`+use Psr\Log\LoggerInterface;`
`15`	`16`	`use Symfony\Component\Process\ProcessBuilder;`
`16`	`17`
`17`	`18`	`/**`
`@@ -31,7 +32,7 @@ class Admin`
`31`	`32`	`*`
`32`	`33`	`* @throws RuntimeException Directory exists or not writable`
`33`	`34`	`*/`
`34`		`- public static function init($path, $bare = true)`
	`35`	`+ public static function init($path, $bare = true, LoggerInterface $logger = null)`
`35`	`36`	`{`
`36`	`37`	`$builder = ProcessBuilder::create(array('git', 'init', '-q'));`
`37`	`38`
`@@ -48,6 +49,6 @@ public static function init($path, $bare = true)`
`48`	`49`	`throw new \RuntimeException(sprintf('Error while initializing repository: %s', $process->getErrorOutput()));`
`49`	`50`	`}`
`50`	`51`
`51`		`- return new Repository($path);`
	`52`	`+ return new Repository($path, null, $logger);`
`52`	`53`	`}`
`53`	`54`	`}`
Original file line number	Diff line number	Diff line change
`@@ -15,7 +15,7 @@`
`15`	`15`	`use Gitonomy\Git\Admin;`
`16`	`16`	`use Gitonomy\Git\Repository;`
`17`	`17`
`18`		`-abstract class AbtractTest extends \PHPUnit_Framework_TestCase`
	`18`	`+abstract class AbstractTest extends \PHPUnit_Framework_TestCase`
`19`	`19`	`{`
`20`	`20`	`private static $libRepo;`
`21`	`21`	`private static $testRepo;`
Original file line number	Diff line number	Diff line change
`@@ -14,7 +14,7 @@`
`14`	`14`
`15`	`15`	`use Gitonomy\Git\Admin;`
`16`	`16`
`17`		`-class AdminTest extends AbtractTest`
	`17`	`+class AdminTest extends AbstractTest`
`18`	`18`	`{`
`19`	`19`	`private $tmpDir;`
`20`	`20`
Original file line number	Diff line number	Diff line change
`@@ -12,7 +12,7 @@`
`12`	`12`
`13`	`13`	`namespace Gitonomy\Git\Tests;`
`14`	`14`
`15`		`-class BlameTest extends AbtractTest`
	`15`	`+class BlameTest extends AbstractTest`
`16`	`16`	`{`
`17`	`17`	`public function testBlame()`
`18`	`18`	`{`