Tests\PHPSA\Compiler\Statement\DoTest

Complexity

Total Complexity

2

Size/Duplication

Total Lines	50
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	7

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	testDoConditionCreatesVar()	0	19	1
A	testDoStatementCreatesVar()	0	21	1

<?php

namespace Tests\PHPSA\Compiler\Statement;

use PhpParser\Node;
use PHPSA\Variable;

class DoTest extends \Tests\PHPSA\TestCase
{
    /**
     * Tests do {} while ($stmtTest = 2) creates the variable
     */
    public function testDoConditionCreatesVar()
    {
        $context = $this->getContext();

        $statement = new Node\Stmt\Do_(
            new Node\Expr\Assign(
                new Node\Expr\Variable(
                    new Node\Name("stmtTest")
                ),
                $this->newScalarExpr(2)
            )
        );
        
        \PHPSA\nodeVisitorFactory($statement, $context);

        $variable = $context->getSymbol("stmtTest");

        parent::assertTrue($variable instanceof Variable);

It seems like you call parent on a different method (assertTrue() instead of testDoConditionCreatesVar()). Are you sure this is correct? If so, you might want to change this to $this->assertTrue().

    }

    /**
     * Tests do { $stmtTest = 2; } while (1 == 1) creates the variable
     */
    public function testDoStatementCreatesVar()
    {
        $context = $this->getContext();

        $statement = new Node\Stmt\Do_(
            new Node\Expr\BinaryOp\Equal($this->newScalarExpr(1), $this->newScalarExpr(1)),
            ["stmts" =>
                new Node\Expr\Assign(
                    new Node\Expr\Variable(
                        new Node\Name("stmtTest")
                    ),
                    $this->newScalarExpr(2)
                )]
        );
        
        \PHPSA\nodeVisitorFactory($statement, $context);

        $variable = $context->getSymbol("stmtTest");

        parent::assertTrue($variable instanceof Variable);

It seems like you call parent on a different method (assertTrue() instead of testDoStatementCreatesVar()). Are you sure this is correct? If so, you might want to change this to $this->assertTrue().

    }
}