Issues in StringWrapperTraitTest.php (master) - Issues in master - webeweb/jquery-datatables-bundle - Measure and Improve Code Quality continuously with Scrutinizer

Issues (117)

Tests/Renderer/Strings/StringWrapperTraitTest.php (1 issue)

Labels

Bug 1

Severity

Minor 1

<?php

/*
 * This file is part of the jquery-datatables-bundle package.
 *
 * (c) 2021 WEBEWEB
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace WBW\Bundle\JQuery\DataTablesBundle\Tests\Renderer\Strings;

use WBW\Bundle\JQuery\DataTablesBundle\Tests\AbstractTestCase;
use WBW\Bundle\JQuery\DataTablesBundle\Tests\Fixtures\Renderer\Strings\TestStringWrapperTrait;

/**
 * String wrapper trait test.
 *
 * @author webeweb <https://github.com/webeweb>
 * @package WBW\Bundle\JQuery\DataTablesBundle\Tests\Renderer\Strings
 */
class StringWrapperTraitTest extends AbstractTestCase {

    /**
     * Test wrapString()
     *
     * @return void
     */
    public function testWrapString(): void {

        $obj = new TestStringWrapperTrait();

        $this->assertNull($obj->wrapString(null, "prefix-", "-suffix"));
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {
        $this->assertEquals("content", $obj->wrapString("content", null, null));
        $this->assertEquals("prefix-content", $obj->wrapString("content", "prefix-", null));
        $this->assertEquals("prefix-content-suffix", $obj->wrapString("content", "prefix-", "-suffix"));
        $this->assertEquals("content-suffix", $obj->wrapString("content", null, "-suffix"));
    }
}


1			<?php
2
3			/*
4			* This file is part of the jquery-datatables-bundle package.
5			*
6			* (c) 2021 WEBEWEB
7			*
8			* For the full copyright and license information, please view the LICENSE
9			* file that was distributed with this source code.
10			*/
11
12			namespace WBW\Bundle\JQuery\DataTablesBundle\Tests\Renderer\Strings;
13
14			use WBW\Bundle\JQuery\DataTablesBundle\Tests\AbstractTestCase;
15			use WBW\Bundle\JQuery\DataTablesBundle\Tests\Fixtures\Renderer\Strings\TestStringWrapperTrait;
16
17			/**
18			* String wrapper trait test.
19			*
20			* @author webeweb <https://github.com/webeweb>
21			* @package WBW\Bundle\JQuery\DataTablesBundle\Tests\Renderer\Strings
22			*/
23			class StringWrapperTraitTest extends AbstractTestCase {
24
25			/**
26			* Test wrapString()
27			*
28			* @return void
29			*/
30			public function testWrapString(): void {
31
32			$obj = new TestStringWrapperTrait();
33
34			$this->assertNull($obj->wrapString(null, "prefix-", "-suffix"));
			0 ignored issues – show Bug introduced 2023-01-02 11:49 UTC by Report Bug Copy Issue Report Show Similar Issues like this Are you sure the usage of `$obj->wrapString(null, 'prefix-', '-suffix')` targeting `WBW\Bundle\JQuery\DataTa...pperTrait::wrapString()` seems to always return null. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
35			$this->assertEquals("content", $obj->wrapString("content", null, null));
36			$this->assertEquals("prefix-content", $obj->wrapString("content", "prefix-", null));
37			$this->assertEquals("prefix-content-suffix", $obj->wrapString("content", "prefix-", "-suffix"));
38			$this->assertEquals("content-suffix", $obj->wrapString("content", null, "-suffix"));
39			}
40			}
41

webeweb / jquery-datatables-bundle

Issues (117)

Tests/Renderer/Strings/StringWrapperTraitTest.php (1 issue)

Labels

Severity

Introduced By

Duplication Side-by-Side

Filter issues like