Skip to content

Commit

Permalink
Merge pull request #214 from thegreenter/fix-phpunit-warn
Browse files Browse the repository at this point in the history
Fix version 5
  • Loading branch information
giansalex authored Jan 11, 2023
2 parents b4d1bab + 341b661 commit 77a3974
Show file tree
Hide file tree
Showing 19 changed files with 44 additions and 40 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/php.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ jobs:
runs-on: ubuntu-20.04
strategy:
matrix:
php: ['8.0']
php: ['8.2']
report-coverage: [false]
include:
- php: '7.4'
Expand Down
4 changes: 2 additions & 2 deletions monorepo-builder.php
Original file line number Diff line number Diff line change
Expand Up @@ -22,8 +22,8 @@

$mbConfig->dataToRemove([
'require-dev' => [
'phpstan/phpstan' => '^1.8',
'vimeo/psalm' => '^4.4',
'phpstan/phpstan' => '^1.9',
'vimeo/psalm' => '^5.4',
],
]);

Expand Down
2 changes: 1 addition & 1 deletion packages/core/composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@
},
"extra": {
"branch-alias": {
"dev-master": "4.4-dev"
"dev-master": "5.0-dev"
}
}
}
2 changes: 1 addition & 1 deletion packages/cpe-validator/composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@
},
"extra": {
"branch-alias": {
"dev-master": "4.4-dev"
"dev-master": "5.0-dev"
}
}
}
4 changes: 2 additions & 2 deletions packages/data/composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@
"type": "library",
"require": {
"php": ">=7.4",
"greenter/core": "^4.4"
"greenter/core": "^5.0"
},
"autoload": {
"psr-4": {
Expand All @@ -25,7 +25,7 @@
},
"extra": {
"branch-alias": {
"dev-master": "4.4-dev"
"dev-master": "5.0-dev"
}
}
}
4 changes: 2 additions & 2 deletions packages/htmltopdf/composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@
"type": "library",
"require": {
"php": ">=7.4",
"greenter/core": "^4.4",
"greenter/core": "^5.0",
"mikehaertl/phpwkhtmltopdf": "^2.4"
},
"require-dev": {
Expand All @@ -37,7 +37,7 @@
},
"extra": {
"branch-alias": {
"dev-master": "4.4-dev"
"dev-master": "5.0-dev"
}
}
}
8 changes: 4 additions & 4 deletions packages/lite/composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -18,9 +18,9 @@
"php": ">=7.4",
"ext-dom": "*",
"greenter/xmldsig": "^5.0",
"greenter/core": "^4.4",
"greenter/ws": "^4.4",
"greenter/xml": "^4.4"
"greenter/core": "^5.0",
"greenter/ws": "^5.0",
"greenter/xml": "^5.0"
},
"require-dev": {
"phpunit/phpunit": "^9"
Expand All @@ -37,7 +37,7 @@
},
"extra": {
"branch-alias": {
"dev-master": "4.4-dev"
"dev-master": "5.0-dev"
}
}
}
4 changes: 4 additions & 0 deletions packages/lite/tests/Greenter/ApiTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,10 @@
use Greenter\Model\Response\SummaryResult;
use PHPUnit\Framework\TestCase;

/**
* Class ApiTest
* @group integration
*/
class ApiTest extends TestCase
{
public function testSend(): void
Expand Down
6 changes: 3 additions & 3 deletions packages/report/composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -17,12 +17,12 @@
"require": {
"php": ">=7.4",
"twig/twig": "~3.0",
"greenter/core": "^4.4",
"greenter/core": "^5.0",
"bacon/bacon-qr-code": "^2.0"
},
"require-dev": {
"phpunit/phpunit": "^9",
"greenter/data": "^4.4"
"greenter/data": "^5.0"
},
"autoload": {
"psr-4": {
Expand All @@ -36,7 +36,7 @@
},
"extra": {
"branch-alias": {
"dev-master": "4.4-dev"
"dev-master": "5.0-dev"
}
}
}
4 changes: 2 additions & 2 deletions packages/validator/composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@
"type": "library",
"require": {
"php": ">=7.4",
"greenter/core": "^4.4",
"greenter/core": "^5.0",
"symfony/validator": "^5.0 || ^6.0"
},
"require-dev": {
Expand All @@ -34,7 +34,7 @@
},
"extra": {
"branch-alias": {
"dev-master": "4.4-dev"
"dev-master": "5.0-dev"
}
}
}
4 changes: 2 additions & 2 deletions packages/ws/composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@
"ext-zlib": "*",
"ext-dom": "*",
"ext-soap": "*",
"greenter/core": "^4.4",
"greenter/core": "^5.0",
"greenter/gre-api": "^1.0",
"nelexa/zip": "^4.0"
},
Expand All @@ -42,7 +42,7 @@
},
"extra": {
"branch-alias": {
"dev-master": "4.4-dev"
"dev-master": "5.0-dev"
}
}
}
4 changes: 2 additions & 2 deletions packages/xcodes/composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@
"type": "library",
"require": {
"php": ">=7.4",
"greenter/core": "^4.4"
"greenter/core": "^5.0"
},
"require-dev": {
"phpunit/phpunit": "^9"
Expand All @@ -33,7 +33,7 @@
},
"extra": {
"branch-alias": {
"dev-master": "4.4-dev"
"dev-master": "5.0-dev"
}
}
}
4 changes: 2 additions & 2 deletions packages/xml-parser/composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@
"require": {
"php": ">=7.4",
"ext-dom": "*",
"greenter/core": "^4.4"
"greenter/core": "^5.0"
},
"autoload": {
"psr-4": {
Expand All @@ -31,7 +31,7 @@
},
"extra": {
"branch-alias": {
"dev-master": "4.4-dev"
"dev-master": "5.0-dev"
}
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ public function testParseDoc($filename)
$obj = $this->getParser()->parse($xml);

$this->assertEquals('09', $obj->getTipoDoc());
$this->assertRegExp('/^T\w{3}/', $obj->getSerie());
$this->assertMatchesRegularExpression('/^T\w{3}/', $obj->getSerie());
$this->assertNotNull($obj->getCompany());
$this->assertNotNull($obj->getDestinatario());
$this->assertLessThanOrEqual(8, strlen($obj->getCorrelativo()));
Expand Down
12 changes: 6 additions & 6 deletions packages/xml-parser/tests/Xml/Parser/InvoiceParserTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -28,8 +28,8 @@ public function testParseSunatSOL($filename)
/**@var $obj Invoice */
$obj = $parser->parse($xml);

$this->assertRegExp('/^0\d{1}/', $obj->getTipoDoc());
$this->assertRegExp('/^E\w{3}/', $obj->getSerie());
$this->assertMatchesRegularExpression('/^0\d{1}/', $obj->getTipoDoc());
$this->assertMatchesRegularExpression('/^E\w{3}/', $obj->getSerie());
$this->assertLessThanOrEqual(8, strlen($obj->getCorrelativo()));
$this->assertNotEmpty($obj->getFechaEmision());
$this->assertGreaterThanOrEqual(1, count($obj->getDetails()));
Expand All @@ -52,8 +52,8 @@ public function testFacParse($filename)
/**@var $obj Invoice */
$obj = $parser->parse($xml);

$this->assertRegExp('/^0\d{1}/', $obj->getTipoDoc());
$this->assertRegExp('/^[FB]\w{3}/', $obj->getSerie());
$this->assertMatchesRegularExpression('/^0\d{1}/', $obj->getTipoDoc());
$this->assertMatchesRegularExpression('/^[FB]\w{3}/', $obj->getSerie());
$this->assertLessThanOrEqual(8, strlen($obj->getCorrelativo()));
$this->assertNotEmpty($obj->getFechaEmision());
$this->assertGreaterThanOrEqual(1, count($obj->getDetails()));
Expand All @@ -71,8 +71,8 @@ public function testFacParseFromDoc()
/**@var $obj Invoice */
$obj = $parser->parse($doc);

$this->assertRegExp('/^0\d{1}/', $obj->getTipoDoc());
$this->assertRegExp('/^[FB]\w{3}/', $obj->getSerie());
$this->assertMatchesRegularExpression('/^0\d{1}/', $obj->getTipoDoc());
$this->assertMatchesRegularExpression('/^[FB]\w{3}/', $obj->getSerie());
$this->assertLessThanOrEqual(8, strlen($obj->getCorrelativo()));
$this->assertNotEmpty($obj->getFechaEmision());
$this->assertGreaterThanOrEqual(1, count($obj->getDetails()));
Expand Down
4 changes: 2 additions & 2 deletions packages/xml-parser/tests/Xml/Parser/PerceptionParserTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -26,8 +26,8 @@ public function testParseDoc($filename)
/**@var $obj Perception */
$obj = $this->getParser()->parse($xml);

$this->assertRegExp('/^[P][0-9A-Z]{3}$/', $obj->getSerie());
$this->assertRegExp('/^\d+$/', $obj->getCorrelativo());
$this->assertMatchesRegularExpression('/^[P][0-9A-Z]{3}$/', $obj->getSerie());
$this->assertMatchesRegularExpression('/^\d+$/', $obj->getCorrelativo());
$this->assertNotNull($obj->getCompany());
$this->assertNotNull($obj->getProveedor());
$this->assertGreaterThan(0, count($obj->getDetails()));
Expand Down
4 changes: 2 additions & 2 deletions packages/xml-parser/tests/Xml/Parser/RetentionParserTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -26,8 +26,8 @@ public function testParseDoc($filename)
/**@var $obj Retention */
$obj = $this->getParser()->parse($xml);

$this->assertRegExp('/^[R][0-9A-Z]{3}$/', $obj->getSerie());
$this->assertRegExp('/^\d+$/', $obj->getCorrelativo());
$this->assertMatchesRegularExpression('/^[R][0-9A-Z]{3}$/', $obj->getSerie());
$this->assertMatchesRegularExpression('/^\d+$/', $obj->getCorrelativo());
$this->assertNotNull($obj->getCompany());
$this->assertNotNull($obj->getProveedor());
$this->assertGreaterThan(0, count($obj->getDetails()));
Expand Down
4 changes: 2 additions & 2 deletions packages/xml-parser/tests/Xml/Parser/SummaryParserTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -27,14 +27,14 @@ public function testParseDoc($filename)
$obj = $this->getParser()->parse($xml);

$this->assertStringContainsString('RC', $obj->getName());
$this->assertRegExp('/^\d+$/', $obj->getCorrelativo());
$this->assertMatchesRegularExpression('/^\d+$/', $obj->getCorrelativo());
$this->assertNotNull($obj->getCompany());
$this->assertGreaterThan(0, count($obj->getDetails()));

foreach ($obj->getDetails() as $detail) {
$this->assertTrue(in_array($detail->getTipoDoc(), ['03', '07', '08']));
$this->assertTrue(in_array($detail->getEstado(), ['1', '2', '3']));
$this->assertRegExp('/^B\d{3}-\d{1,8}$/', $detail->getSerieNro());
$this->assertMatchesRegularExpression('/^B\d{3}-\d{1,8}$/', $detail->getSerieNro());
$this->assertTrue(is_float($detail->getTotal()));
if ($detail->getDocReferencia()) {
$ref = $detail->getDocReferencia();
Expand Down
6 changes: 3 additions & 3 deletions packages/xml/composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -16,11 +16,11 @@
"type": "library",
"require": {
"php": ">=7.4",
"greenter/core": "^4.4",
"greenter/core": "^5.0",
"twig/twig": "~3.0"
},
"require-dev": {
"greenter/data": "^4.4",
"greenter/data": "^5.0",
"greenter/ubl-validator": "^2.0",
"phpunit/phpunit": "^9"
},
Expand All @@ -36,7 +36,7 @@
},
"extra": {
"branch-alias": {
"dev-master": "4.4-dev"
"dev-master": "5.0-dev"
}
}
}

0 comments on commit 77a3974

Please sign in to comment.