remoção de anotações do phpunit

lidani · lidani · commit f64abd471dfb · 2022-02-15T17:01:36.000-03:00
diff --git a/src/Exception/ExceptionCollection.php b/src/Exception/ExceptionCollection.php
@@ -13,7 +13,6 @@ class ExceptionCollection extends \Exception implements ExceptionInterface, Iter
      */
     protected $exceptions = [];
     /**
-     * @var string
      * stan: Property NFePHP\Common\Exception\ExceptionCollection::$shortMessage is never read, only written.
      */
     // private $shortMessage;
diff --git a/tests/Certificate/CertificateTest.php b/tests/Certificate/CertificateTest.php
@@ -3,7 +3,6 @@
 namespace NFePHP\Common\Tests\Certificate;
 
 use NFePHP\Common\Certificate;
-use NFePHP\Common\Exception\CertificateException;
 
 class CertificateTest extends \PHPUnit\Framework\TestCase
 {
@@ -42,9 +41,6 @@ public function testShouldLoadCertificate()
         $this->assertTrue($certificate->verify('nfe', $dataSigned));
     }
 
-    /**
-     * @expectedException NFePHP\Common\Exception\CertificateException
-     */
     public function testShouldGetExceptionWhenLoadPfxCertificate()
     {
         $this->expectException(\NFePHP\Common\Exception\CertificateException::class);
diff --git a/tests/SignerTest.php b/tests/SignerTest.php
@@ -4,7 +4,6 @@
 
 use NFePHP\Common\Signer;
 use NFePHP\Common\Certificate;
-use NFePHP\Common\Exception\SignerException;
 
 class SignerTest extends \PHPUnit\Framework\TestCase
 {
@@ -44,12 +43,10 @@ public function testIsSigned()
 
     /**
      * @covers Signer::existsSignature
-     * @expectedException NFePHP\Common\Exception\SignerException
      */
     public function testSignFailNotXML()
     {
-        $this->expectException(SignerException::class);
-
+        $this->expectException(\NFePHP\Common\Exception\SignerException::class);
         $pfx = file_get_contents(__DIR__ . '/fixtures/certs/certificado_teste.pfx');
         $certificate = Certificate::readPfx($pfx, 'associacao');
         $content = "<html><body></body></html>";
@@ -59,12 +56,10 @@ public function testSignFailNotXML()
     /**
      * @covers Signer::existsSignature
      * @covers Signer::digestCheck
-     * @expectedException NFePHP\Common\Exception\SignerException
      */
     public function testIsSignedFailTagNotFound()
     {
-        $this->expectException(SignerException::class);
-
+        $this->expectException(\NFePHP\Common\Exception\SignerException::class);
         $file = __DIR__ . '/fixtures/xml/NFe/2017signed.xml';
         $xml = file_get_contents($file);
         Signer::isSigned($xml, 'infCTe');
@@ -75,12 +70,10 @@ public function testIsSignedFailTagNotFound()
      * @covers Signer::digestCheck
      * @covers Signer::canonize
      * @covers Signer::makeDigest
-     * @expectedException NFePHP\Common\Exception\SignerException
      */
     public function testIsSignedFailDigest()
     {
-        $this->expectException(SignerException::class);
-
+        $this->expectException(\NFePHP\Common\Exception\SignerException::class);
         $file = __DIR__ . '/fixtures/xml/NFe/2017signedDigestFail.xml';
         $xml = file_get_contents($file);
         Signer::isSigned($xml);
@@ -92,12 +85,10 @@ public function testIsSignedFailDigest()
      * @covers Signer::signatureCheck
      * @covers Signer::canonize
      * @covers Signer::makeDigest
-     * @expectedException NFePHP\Common\Exception\SignerException
      */
     public function testIsSignedFailSignature()
     {
-        $this->expectException(SignerException::class);
-
+        $this->expectException(\NFePHP\Common\Exception\SignerException::class);
         $file = __DIR__ . '/fixtures/xml/NFe/2017signedSignatureFail.xml';
         $xml = file_get_contents($file);
         Signer::isSigned($xml);
diff --git a/tests/Soap/SoapFakeTest.php b/tests/Soap/SoapFakeTest.php
@@ -91,12 +91,10 @@ public function testDisableCertValidation()
     }
 
     /**
-     * @expectedException NFePHP\Common\Exception\RuntimeException
      */
     public function testDisableCertValidationFail()
     {
-        $this->expectException(\NFePHP\Common\Exception\RuntimeException::class);
-
+        $this->expectException(\NFePHP\Common\Certificate\Exception\Expired::class);
         $certificate = Certificate::readPfx(file_get_contents(__DIR__ . self::TEST_PFX_FILE), 'associacao');
         new SoapFake($certificate);
     }
diff --git a/tests/UFListTest.php b/tests/UFListTest.php
@@ -46,12 +46,9 @@ public function testgetUFByCode()
         $this->assertEquals('SP', $uf);
     }
 
-    /**
-     * @expectedException InvalidArgumentException
-     */
     public function testgetUFByCodeFail()
     {
-        $this->expectException(\InvalidArgumentException::class);
+        $this->expectException(\NFePHP\Common\Exception\InvalidArgumentException::class);
 
         UFList::getUFByCode(77);
     }
@@ -62,12 +59,9 @@ public function testgetUFByUF()
         $this->assertEquals(35, $code);
     }
 
-    /**
-     * @expectedException InvalidArgumentException
-     */
     public function testgetUFByUFFail()
     {
-        $this->expectException(\InvalidArgumentException::class);
+        $this->expectException(\NFePHP\Common\Exception\InvalidArgumentException::class);
 
         UFList::getCodeByUF('aa');
     }
diff --git a/tests/ValidatorTest.php b/tests/ValidatorTest.php
@@ -17,12 +17,9 @@ public function testIsValidTrue()
         $this->assertTrue($actual);
     }
 
-    /**
-     * @expectedException RuntimeException
-     */
     public function testIsValidWithErrors()
     {
-        $this->expectException(\RuntimeException::class);
+        $this->expectException(\NFePHP\Common\Exception\ValidatorException::class);
 
         $xml = file_get_contents(__DIR__ . self::TEST_XML_PATH . 'NFe/' .
             '35101158716523000119550010000000011003000000-nfeSigned.xml');
@@ -31,12 +28,9 @@ public function testIsValidWithErrors()
         $this->assertFalse($actual);
     }
 
-    /**
-     * @expectedException RuntimeException
-     */
     public function testIsValidWithNoXML()
     {
-        $this->expectException(\RuntimeException::class);
+        $this->expectException(\NFePHP\Common\Exception\ValidatorException::class);
 
         $xml = 'alkjdkjhdshkjshsjhskjshksjh';
         $xsd = __DIR__ . self::TEST_XSD_PATH . 'nfe_v3.10.xsd';

Original file line number	Diff line number	Diff line change
`@@ -3,7 +3,6 @@`
`3`	`3`	`namespace NFePHP\Common\Tests\Certificate;`
`4`	`4`
`5`	`5`	`use NFePHP\Common\Certificate;`
`6`		`-use NFePHP\Common\Exception\CertificateException;`
`7`	`6`
`8`	`7`	`class CertificateTest extends \PHPUnit\Framework\TestCase`
`9`	`8`	`{`
`@@ -42,9 +41,6 @@ public function testShouldLoadCertificate()`
`42`	`41`	`$this->assertTrue($certificate->verify('nfe', $dataSigned));`
`43`	`42`	`}`
`44`	`43`
`45`		`- /**`
`46`		`- * @expectedException NFePHP\Common\Exception\CertificateException`
`47`		`- */`
`48`	`44`	`public function testShouldGetExceptionWhenLoadPfxCertificate()`
`49`	`45`	`{`
`50`	`46`	`$this->expectException(\NFePHP\Common\Exception\CertificateException::class);`
Original file line number	Diff line number	Diff line change
`@@ -91,12 +91,10 @@ public function testDisableCertValidation()`
`91`	`91`	`}`
`92`	`92`
`93`	`93`	`/**`
`94`		`- * @expectedException NFePHP\Common\Exception\RuntimeException`
`95`	`94`	`*/`
`96`	`95`	`public function testDisableCertValidationFail()`
`97`	`96`	`{`
`98`		`- $this->expectException(\NFePHP\Common\Exception\RuntimeException::class);`
`99`		`-`
	`97`	`+ $this->expectException(\NFePHP\Common\Certificate\Exception\Expired::class);`
`100`	`98`	`$certificate = Certificate::readPfx(file_get_contents(__DIR__ . self::TEST_PFX_FILE), 'associacao');`
`101`	`99`	`new SoapFake($certificate);`
`102`	`100`	`}`
Original file line number	Diff line number	Diff line change
`@@ -46,12 +46,9 @@ public function testgetUFByCode()`
`46`	`46`	`$this->assertEquals('SP', $uf);`
`47`	`47`	`}`
`48`	`48`
`49`		`- /**`
`50`		`- * @expectedException InvalidArgumentException`
`51`		`- */`
`52`	`49`	`public function testgetUFByCodeFail()`
`53`	`50`	`{`
`54`		`- $this->expectException(\InvalidArgumentException::class);`
	`51`	`+ $this->expectException(\NFePHP\Common\Exception\InvalidArgumentException::class);`
`55`	`52`
`56`	`53`	`UFList::getUFByCode(77);`
`57`	`54`	`}`
`@@ -62,12 +59,9 @@ public function testgetUFByUF()`
`62`	`59`	`$this->assertEquals(35, $code);`
`63`	`60`	`}`
`64`	`61`
`65`		`- /**`
`66`		`- * @expectedException InvalidArgumentException`
`67`		`- */`
`68`	`62`	`public function testgetUFByUFFail()`
`69`	`63`	`{`
`70`		`- $this->expectException(\InvalidArgumentException::class);`
	`64`	`+ $this->expectException(\NFePHP\Common\Exception\InvalidArgumentException::class);`
`71`	`65`
`72`	`66`	`UFList::getCodeByUF('aa');`
`73`	`67`	`}`
Original file line number	Diff line number	Diff line change
`@@ -17,12 +17,9 @@ public function testIsValidTrue()`
`17`	`17`	`$this->assertTrue($actual);`
`18`	`18`	`}`
`19`	`19`
`20`		`- /**`
`21`		`- * @expectedException RuntimeException`
`22`		`- */`
`23`	`20`	`public function testIsValidWithErrors()`
`24`	`21`	`{`
`25`		`- $this->expectException(\RuntimeException::class);`
	`22`	`+ $this->expectException(\NFePHP\Common\Exception\ValidatorException::class);`
`26`	`23`
`27`	`24`	`$xml = file_get_contents(__DIR__ . self::TEST_XML_PATH . 'NFe/' .`
`28`	`25`	`'35101158716523000119550010000000011003000000-nfeSigned.xml');`
`@@ -31,12 +28,9 @@ public function testIsValidWithErrors()`
`31`	`28`	`$this->assertFalse($actual);`
`32`	`29`	`}`
`33`	`30`
`34`		`- /**`
`35`		`- * @expectedException RuntimeException`
`36`		`- */`
`37`	`31`	`public function testIsValidWithNoXML()`
`38`	`32`	`{`
`39`		`- $this->expectException(\RuntimeException::class);`
	`33`	`+ $this->expectException(\NFePHP\Common\Exception\ValidatorException::class);`
`40`	`34`
`41`	`35`	`$xml = 'alkjdkjhdshkjshsjhskjshksjh';`
`42`	`36`	`$xsd = __DIR__ . self::TEST_XSD_PATH . 'nfe_v3.10.xsd';`