Skip to content

Commit

Permalink
Merge pull request #35 from xefi/fix/namespaces
Browse files Browse the repository at this point in the history
Fix/namespaces
  • Loading branch information
GautierDele authored Nov 17, 2024
2 parents b151350 + e1b6987 commit 9de9e16
Show file tree
Hide file tree
Showing 9 changed files with 9 additions and 11 deletions.
2 changes: 1 addition & 1 deletion composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@
],
"require": {
"php": "^8.3",
"phpdocumentor/reflection-docblock": "^5.4",
"phpdocumentor/reflection-docblock": "^5.6",
"psr/container": "^2.0"
},
"require-dev": {
Expand Down
2 changes: 1 addition & 1 deletion src/Manifests/ContainerMixinManifest.php
Original file line number Diff line number Diff line change
Expand Up @@ -114,7 +114,7 @@ public function build(array $extensionMethods, array $extensions)
$parameters[] = new DocBlock\Tags\MethodParameter(
name: $parameter->getName(),
type: $type,
defaultValue: $parameter->isDefaultValueAvailable() ? $parameter->getDefaultValue() : null
defaultValue: $parameter->isDefaultValueAvailable() ? $parameter->getDefaultValue() : DocBlock\Tags\MethodParameter::NO_DEFAULT_VALUE
);
}

Expand Down
2 changes: 1 addition & 1 deletion tests/Unit/ContainerMixinManifestTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ public function testContainerMixinBuild()
);

$this->assertStringContainsString(
'@method string withNullableParameter(?string $one)',
'@method string withNullableParameter(?string $one = \'default value\')',
$containerMixinContent
);

Expand Down
3 changes: 1 addition & 2 deletions tests/Unit/Extensions/FinancialExtensionTest.php
Original file line number Diff line number Diff line change
@@ -1,9 +1,8 @@
<?php

namespace Extensions;
namespace Xefi\Faker\Tests\Unit\Extensions;

use Xefi\Faker\Calculators\Iban;
use Xefi\Faker\Tests\Unit\Extensions\TestCase;

final class FinancialExtensionTest extends TestCase
{
Expand Down
3 changes: 1 addition & 2 deletions tests/Unit/Extensions/InternetExtensionTest.php
Original file line number Diff line number Diff line change
@@ -1,10 +1,9 @@
<?php

namespace Extensions;
namespace Xefi\Faker\Tests\Unit\Extensions;

use ReflectionClass;
use Xefi\Faker\Container\Container;
use Xefi\Faker\Tests\Unit\Extensions\TestCase;

final class InternetExtensionTest extends TestCase
{
Expand Down
2 changes: 1 addition & 1 deletion tests/Unit/Modifiers/LowercaseModifierTest.php
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
<?php

namespace Modifiers;
namespace Xefi\Faker\Tests\Unit\Modifiers;

use Xefi\Faker\Faker;
use Xefi\Faker\Modifiers\LowercaseModifier;
Expand Down
2 changes: 1 addition & 1 deletion tests/Unit/Modifiers/OptionalModifierTest.php
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
<?php

namespace Modifiers;
namespace Xefi\Faker\Tests\Unit\Modifiers;

use Random\Randomizer;
use Xefi\Faker\Faker;
Expand Down
2 changes: 1 addition & 1 deletion tests/Unit/Modifiers/UppercaseModifierTest.php
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
<?php

namespace Modifiers;
namespace Xefi\Faker\Tests\Unit\Modifiers;

use Xefi\Faker\Faker;
use Xefi\Faker\Modifiers\UppercaseModifier;
Expand Down
2 changes: 1 addition & 1 deletion tests/Unit/Strategies/RegexStrategyTest.php
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
<?php

namespace Strategies;
namespace Xefi\Faker\Tests\Unit\Strategies;

use PHPUnit\Framework\Attributes\DataProvider;
use Xefi\Faker\Exceptions\MaximumTriesReached;
Expand Down

0 comments on commit 9de9e16

Please sign in to comment.