Skip to content

Commit 1e4c9ac

Browse files
Make code styling more consistent between tests
1 parent 4607ef9 commit 1e4c9ac

File tree

3 files changed

+12
-15
lines changed

3 files changed

+12
-15
lines changed

tests/Unit/ExpressionListTest.php

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,6 @@
2222
namespace WikibaseSolutions\CypherDSL\Tests\Unit;
2323

2424
use PHPUnit\Framework\TestCase;
25-
use TypeError;
2625
use WikibaseSolutions\CypherDSL\ExpressionList;
2726
use WikibaseSolutions\CypherDSL\Query;
2827

@@ -33,7 +32,7 @@ class ExpressionListTest extends TestCase
3332
{
3433
use TestHelper;
3534

36-
public function testEmpty(): void
35+
public function testEmpty()
3736
{
3837
$expressionList = new ExpressionList([]);
3938

@@ -45,7 +44,7 @@ public function testEmpty(): void
4544
* @param array $expressions
4645
* @param string $expected
4746
*/
48-
public function testOneDimensional(array $expressions, string $expected): void
47+
public function testOneDimensional(array $expressions, string $expected)
4948
{
5049
$expressionList = new ExpressionList($expressions);
5150

@@ -57,7 +56,7 @@ public function testOneDimensional(array $expressions, string $expected): void
5756
* @param array $expressions
5857
* @param string $expected
5958
*/
60-
public function testMultidimensional(array $expressions, string $expected): void
59+
public function testMultidimensional(array $expressions, string $expected)
6160
{
6261
$expressionList = new ExpressionList($expressions);
6362

tests/Unit/InTest.php

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -21,13 +21,11 @@
2121

2222
namespace WikibaseSolutions\CypherDSL\Tests\Unit;
2323

24-
use phpDocumentor\Reflection\PseudoTypes\LiteralString;
2524
use PHPUnit\Framework\TestCase;
2625
use TypeError;
2726
use WikibaseSolutions\CypherDSL\ExpressionList;
2827
use WikibaseSolutions\CypherDSL\In;
2928
use WikibaseSolutions\CypherDSL\Literals\StringLiteral;
30-
use WikibaseSolutions\CypherDSL\RawExpression;
3129
use WikibaseSolutions\CypherDSL\Types\AnyType;
3230
use WikibaseSolutions\CypherDSL\Types\CompositeTypes\ListType;
3331
use WikibaseSolutions\CypherDSL\Types\PropertyTypes\PropertyType;
@@ -39,7 +37,7 @@ class InTest extends TestCase
3937
{
4038
use TestHelper;
4139

42-
public function testToQuery(): void
40+
public function testToQuery()
4341
{
4442
$inequality = new In($this->getQueryConvertableMock(PropertyType::class, "a"), $this->getQueryConvertableMock(ListType::class, "b"));
4543

@@ -50,7 +48,7 @@ public function testToQuery(): void
5048
$this->assertSame("((a IN b) IN c)", $inequality->toQuery());
5149
}
5250

53-
public function testInExpressionList(): void
51+
public function testInExpressionList()
5452
{
5553
$inequality = new In($this->getQueryConvertableMock(PropertyType::class, "a"), new ExpressionList([new StringLiteral('a'), new StringLiteral('b')]));
5654

@@ -61,7 +59,7 @@ public function testInExpressionList(): void
6159
$this->assertSame("((a IN ['a', 'b']) IN c)", $inequality->toQuery());
6260
}
6361

64-
public function testDoesNotAcceptAnyTypeAsOperands(): void
62+
public function testDoesNotAcceptAnyTypeAsOperands()
6563
{
6664
$this->expectException(TypeError::class);
6765

tests/Unit/QueryTest.php

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -445,28 +445,28 @@ public function testBuildEmpty()
445445
$this->assertSame("", $query->build());
446446
}
447447

448-
public function testInt(): void
448+
public function testInt()
449449
{
450450
$literal = Query::literal(1);
451451
self::assertInstanceOf(Decimal::class, $literal);
452452
self::assertEquals('1', $literal->toQuery());
453453
}
454454

455-
public function testFloat(): void
455+
public function testFloat()
456456
{
457457
$literal = Query::literal(1.2);
458458
self::assertInstanceOf(Decimal::class, $literal);
459459
self::assertEquals('1.2', $literal->toQuery());
460460
}
461461

462-
public function testString(): void
462+
public function testString()
463463
{
464464
$literal = Query::literal('abc');
465465
self::assertInstanceOf(StringLiteral::class, $literal);
466466
self::assertEquals("'abc'", $literal->toQuery());
467467
}
468468

469-
public function testStringAble(): void
469+
public function testStringAble()
470470
{
471471
$literal = Query::literal(new class () {
472472
public function __toString(): string
@@ -478,14 +478,14 @@ public function __toString(): string
478478
self::assertEquals("'stringable abc'", $literal->toQuery());
479479
}
480480

481-
public function testBool(): void
481+
public function testBool()
482482
{
483483
$literal = Query::literal(true);
484484
self::assertInstanceOf(Boolean::class, $literal);
485485
self::assertEquals("true", $literal->toQuery());
486486
}
487487

488-
public function testInvalidLiteral(): void
488+
public function testInvalidLiteral()
489489
{
490490
$literal = Query::literal(true);
491491
$this->expectException(InvalidArgumentException::class);

0 commit comments

Comments
 (0)