Merge branch 'master' into performs_no_expectations

# Conflicts:
#	src/PendingCalls/TestCall.php

tests/Features/Coverage.php

@@ -8,7 +8,7 @@ use Symfony\Component\Console\Output\ConsoleOutput;
 it('has plugin')->assertTrue(class_exists(CoveragePlugin::class));
 
 it('adds coverage if --coverage exist', function () {
-    $plugin = new CoveragePlugin(new ConsoleOutput());
+    $plugin    = new CoveragePlugin(new ConsoleOutput());
     $testSuite = TestSuite::getInstance();
 
     expect($plugin->coverage)->toBeFalse();

tests/Features/Covers.php

@@ -0,0 +1,78 @@
+<?php
+
+use Pest\Factories\Attributes\Covers;
+use Pest\PendingCalls\TestCall;
+use Pest\TestSuite;
+
+$runCounter = 0;
+
+class TestCoversClass1
+{
+}
+class TestCoversClass2
+{
+}
+
+class TestCoversClass3
+{
+}
+
+function testCoversFunction()
+{
+}
+
+it('uses the correct PHPUnit attribute for class', function () {
+    $attributes = (new ReflectionClass($this))->getAttributes();
+
+    expect($attributes[0]->getName())->toBe('PHPUnit\Framework\Attributes\CoversClass');
+    expect($attributes[0]->getArguments()[0])->toBe('P\Tests\Features\TestCoversClass1');
+})->coversClass(TestCoversClass1::class);
+
+it('uses the correct PHPUnit attribute for function', function () {
+    $attributes = (new ReflectionClass($this))->getAttributes();
+
+    expect($attributes[1]->getName())->toBe('PHPUnit\Framework\Attributes\CoversFunction');
+    expect($attributes[1]->getArguments()[0])->toBe('foo');
+})->coversFunction('foo');
+
+it('removes duplicated attributes', function () {
+    $attributes = (new ReflectionClass($this))->getAttributes();
+
+    expect($attributes[2]->getName())->toBe('PHPUnit\Framework\Attributes\CoversClass');
+    expect($attributes[2]->getArguments()[0])->toBe('P\Tests\Features\TestCoversClass2');
+    expect($attributes[3]->getName())->toBe('PHPUnit\Framework\Attributes\CoversClass');
+    expect($attributes[3]->getArguments()[0])->toBe('Pest\Factories\Attributes\Covers');
+    expect($attributes[4]->getName())->toBe('PHPUnit\Framework\Attributes\CoversFunction');
+    expect($attributes[4]->getArguments()[0])->toBe('bar');
+    expect($attributes[5]->getName())->toBe('PHPUnit\Framework\Attributes\CoversFunction');
+    expect($attributes[5]->getArguments()[0])->toBe('baz');
+})
+    ->coversClass(TestCoversClass2::class, TestCoversClass1::class, Covers::class)
+    ->coversFunction('bar', 'foo', 'baz');
+
+it('guesses if the given argument is a class or function', function () {
+    $attributes = (new ReflectionClass($this))->getAttributes();
+
+    expect($attributes[6]->getName())->toBe('PHPUnit\Framework\Attributes\CoversClass');
+    expect($attributes[6]->getArguments()[0])->toBe('P\Tests\Features\TestCoversClass3');
+    expect($attributes[7]->getName())->toBe('PHPUnit\Framework\Attributes\CoversFunction');
+    expect($attributes[7]->getArguments()[0])->toBe('testCoversFunction');
+})->covers(TestCoversClass3::class, 'testCoversFunction');
+
+it('appends CoversNothing to method attributes', function () {
+    $phpDoc = (new ReflectionClass($this))->getMethod($this->getName());
+
+    expect(str_contains($phpDoc->getDocComment(), '* @coversNothing'))->toBeTrue();
+})->coversNothing();
+
+it('does not append CoversNothing to other methods', function () {
+    $phpDoc = (new ReflectionClass($this))->getMethod($this->getName());
+
+    expect(str_contains($phpDoc->getDocComment(), '* @coversNothing'))->toBeFalse();
+});
+
+it('throws exception if no class nor method has been found', function () {
+    $testCall = new TestCall(TestSuite::getInstance(), 'filename', 'description', fn () => 'closure');
+
+    $testCall->covers('fakeName');
+})->throws(InvalidArgumentException::class, 'No class or method named "fakeName" has been found.');

tests/Features/Exceptions.php

@@ -14,10 +14,18 @@ it('catch exceptions and messages', function () {
     throw new Exception('Something bad happened');
 })->throws(Exception::class, 'Something bad happened');
 
+it('catch exceptions, messages and code', function () {
+    throw new Exception('Something bad happened', 1);
+})->throws(Exception::class, 'Something bad happened', 1);
+
 it('can just define the message', function () {
     throw new Exception('Something bad happened');
 })->throws('Something bad happened');
 
+it('can just define the code', function () {
+    throw new Exception('Something bad happened', 1);
+})->throws(1);
+
 it('not catch exceptions if given condition is false', function () {
     $this->assertTrue(true);
 })->throwsIf(false, Exception::class);
@@ -30,10 +38,22 @@ it('catch exceptions and messages if given condition is true', function () {
     throw new Exception('Something bad happened');
 })->throwsIf(true, Exception::class, 'Something bad happened');
 
+it('catch exceptions, messages and code if given condition is true', function () {
+    throw new Exception('Something bad happened', 1);
+})->throwsIf(true, Exception::class, 'Something bad happened', 1);
+
 it('can just define the message if given condition is true', function () {
     throw new Exception('Something bad happened');
 })->throwsIf(true, 'Something bad happened');
 
+it('can just define the code if given condition is true', function () {
+    throw new Exception('Something bad happened', 1);
+})->throwsIf(true, 1);
+
 it('can just define the message if given condition is 1', function () {
     throw new Exception('Something bad happened');
 })->throwsIf(1, 'Something bad happened');
+
+it('can just define the code if given condition is 1', function () {
+    throw new Exception('Something bad happened', 1);
+})->throwsIf(1, 1);

tests/Features/Expect/pipes.php

@@ -12,7 +12,7 @@ class Number
     public function __construct(
         public int $value
     ) {
-        //..
+        // ..
     }
 }
 
@@ -21,7 +21,7 @@ class Char
     public function __construct(
         public string $value
     ) {
-        //..
+        // ..
     }
 }
 
@@ -30,7 +30,7 @@ class Symbol
     public function __construct(
         public string $value
     ) {
-        //..
+        // ..
     }
 }
 
@@ -69,11 +69,11 @@ expect()->pipe('toBe', function ($next, $expected) use ($state) {
         assertInstanceOf(Char::class, $expected);
         assertEquals($this->value->value, $expected->value);
 
-        //returning nothing stops pipeline execution
+        // returning nothing stops pipeline execution
         return;
     }
 
-    //calling $next(); let the pipeline to keep running
+    // calling $next(); let the pipeline to keep running
     $next();
 });
 

tests/Features/Expect/toBe.php

@@ -5,7 +5,7 @@ use PHPUnit\Framework\ExpectationFailedException;
 expect(true)->toBeTrue()->and(false)->toBeFalse();
 
 test('strict comparisons', function () {
-    $nuno = new stdClass();
+    $nuno  = new stdClass();
     $dries = new stdClass();
 
     expect($nuno)->toBe($nuno)->not->toBe($dries);

tests/Features/Expect/toHaveProperties.php

@@ -3,24 +3,24 @@
 use PHPUnit\Framework\ExpectationFailedException;
 
 test('pass', function () {
-    $object = new stdClass();
+    $object       = new stdClass();
     $object->name = 'Jhon';
-    $object->age = 21;
+    $object->age  = 21;
 
     expect($object)->toHaveProperties(['name', 'age']);
 });
 
 test('failures', function () {
-    $object = new stdClass();
+    $object       = new stdClass();
     $object->name = 'Jhon';
 
     expect($object)->toHaveProperties(['name', 'age']);
 })->throws(ExpectationFailedException::class);
 
 test('not failures', function () {
-    $object = new stdClass();
+    $object       = new stdClass();
     $object->name = 'Jhon';
-    $object->age = 21;
+    $object->age  = 21;
 
     expect($object)->not->toHaveProperties(['name', 'age']);
 })->throws(ExpectationFailedException::class);

tests/Features/Expect/toThrow.php

@@ -58,3 +58,19 @@ test('closure missing parameter', function () {
 test('closure missing type-hint', function () {
     expect(function () {})->toThrow(function ($e) {});
 })->throws(InvalidArgumentException::class, 'The given closure\'s parameter must be type-hinted as the class string.');
+
+it('can handle a non-defined exception', function () {
+    expect(function () {
+        throw new NonExistingException();
+    })->toThrow(NonExistingException::class);
+})->throws(Error::class, 'Class "NonExistingException" not found');
+
+it('can handle a class not found Error', function () {
+    expect(function () {
+        throw new NonExistingException();
+    })->toThrow('Class "NonExistingException" not found');
+
+    expect(function () {
+        throw new NonExistingException();
+    })->toThrow(Error::class, 'Class "NonExistingException" not found');
+});

tests/Features/Expect/unless.php

@@ -3,9 +3,9 @@
 use PHPUnit\Framework\ExpectationFailedException;
 
 beforeEach(function () {
-    $this->unlessObject = new stdClass();
+    $this->unlessObject            = new stdClass();
     $this->unlessObject->trueValue = true;
-    $this->unlessObject->foo = 'foo';
+    $this->unlessObject->foo       = 'foo';
 });
 
 it('pass', function () {

tests/Features/Expect/when.php

@@ -3,9 +3,9 @@
 use PHPUnit\Framework\ExpectationFailedException;
 
 beforeEach(function () {
-    $this->whenObject = new stdClass();
+    $this->whenObject            = new stdClass();
     $this->whenObject->trueValue = true;
-    $this->whenObject->foo = 'foo';
+    $this->whenObject->foo       = 'foo';
 });
 
 it('pass', function () {

tests/Unit/Support/Reflection.php

@@ -10,7 +10,7 @@ it('gets file name from closure', function () {
 });
 
 it('gets property values', function () {
-    $class = new class() {
+    $class           = new class() {
         private $foo = 'bar';
     };
 

tests/Unit/TestSuite.php

@@ -8,7 +8,7 @@ use Pest\TestSuite;
 
 it('does not allow to add the same test description twice', function () {
     $testSuite = new TestSuite(getcwd(), 'tests');
-    $method = new TestCaseMethodFactory('foo', 'bar', null);
+    $method    = new TestCaseMethodFactory('foo', 'bar', null);
 
     $testSuite->tests->set($method);
     $testSuite->tests->set($method);
@@ -43,7 +43,7 @@ it('can return an array of all test suite filenames', function () {
 it('can filter the test suite filenames to those with the only method', function () {
     $testSuite = new TestSuite(getcwd(), 'tests');
 
-    $testWithOnly = new TestCaseMethodFactory('a', 'b', null);
+    $testWithOnly       = new TestCaseMethodFactory('a', 'b', null);
     $testWithOnly->only = true;
     $testSuite->tests->set($testWithOnly);
 
@@ -61,7 +61,7 @@ it('does not filter the test suite filenames to those with the only method when
 
     $test = function () {};
 
-    $testWithOnly = new TestCaseMethodFactory('a', 'b', null);
+    $testWithOnly       = new TestCaseMethodFactory('a', 'b', null);
     $testWithOnly->only = true;
     $testSuite->tests->set($testWithOnly);
 

tests/Visual/Help.php

@@ -8,7 +8,7 @@ test('visual snapshot of help command output', function () {
 
     if (getenv('REBUILD_SNAPSHOTS')) {
         $outputBuffer = new BufferedOutput();
-        $plugin = new Help($outputBuffer);
+        $plugin       = new Help($outputBuffer);
 
         $plugin();
 

tests/Visual/Success.php

@@ -2,7 +2,7 @@
 
 test('visual snapshot of test suite on success', function () {
     $testsPath = dirname(__DIR__);
-    $snapshot = implode(DIRECTORY_SEPARATOR, [
+    $snapshot  = implode(DIRECTORY_SEPARATOR, [
         $testsPath,
         '.snapshots',
         'success.txt',