AzineGenderGuesserTest - Code Metrics - Inspection of "Issue/upgrade phpunit to version 7.2" - azine/AzineHybridAuthBundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#11)

by Eugene

created 2018-11-05 22:20 UTC

AzineGenderGuesserTest A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	161
Duplicated Lines	21.12 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

Metric	Value
wmc	8
lcom	1
cbo	2
dl	34
loc	161
rs	10
c	0
b	0
f	0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	testGuess()	0	15	1
A	testGender()	0	9	1
A	testGuesserWithOptions()	0	47	1
A	testGuesserWithEitherWeight()	0	12	1
A	testGuesserWithRules1()	0	15	1
A	testGuesserWithRules2()	0	19	1
A	testGuesserWithMetaPhone()	17	17	1
A	testGuesserWithMetaPhone2()	17	17	1

How to fix Duplicated Code

<?php

namespace Azine\HybridAuthBundle\Tests\Services;

use Azine\HybridAuthBundle\Services\AzineGenderGuesser;
use Azine\HybridAuthBundle\Tests\AzineTestCase;

class AzineGenderGuesserTest extends AzineTestCase
{
    public function testGuess()
    {
        $guesser = new AzineGenderGuesser();
        $result = $guesser->guess('Patrick');
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->assertSame('m', $result['gender']);
        $result = $guesser->guess('Tom');
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->assertSame('m', $result['gender']);

        $result = $guesser->guess('Sonja');
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->assertSame('f', $result['gender']);
        $result = $guesser->guess('Petra');
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->assertSame('f', $result['gender']);

        $this->assertNull($guesser->guess(null));
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
    }

    public function testGender()
    {
        $guesser = new AzineGenderGuesser();
        $this->assertSame('m', $guesser->gender('Patrick'));
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->assertSame('m', $guesser->gender('Tom'));
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->assertSame('f', $guesser->gender('Sonja'));
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->assertSame('f', $guesser->gender('Petra'));
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->assertSame('', $guesser->gender('ertwerqwrfas'));
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
    }

    public function testGuesserWithOptions()
    {
        $females = array('lkjlkjlkj' => 1);
        $males = array('oiuoiuoui' => 1);
        $matchList = array('one_only',
                            'either_weight',
                            'one_only_metaphone',
                            'either_weight_metaphone',
                            'v2_rules',
                            'v1_rules', );

        // use with extra names supplied via constructor
        $guesser = new AzineGenderGuesser($females, $males, true, $matchList);

        $result = $guesser->guess('oiuoiuoui');
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->assertSame('m', $result['gender']);
        $result = $guesser->guess('Patrick');
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->assertSame('m', $result['gender']);
        $result = $guesser->guess('Tom');
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->assertSame('m', $result['gender']);

        $result = $guesser->guess('lkjlkjlkj');
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->assertSame('f', $result['gender']);
        $result = $guesser->guess('Sonja');
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->assertSame('f', $result['gender']);
        $result = $guesser->guess('Petra');
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->assertSame('f', $result['gender']);

        // use only names supplied via constructor
        $guesser = new AzineGenderGuesser($females, $males, false, array('one_only'));

        $result = $guesser->guess('oiuoiuoui');
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->assertSame('m', $result['gender']);

        $result = $guesser->guess('Patrick');
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->assertNull($result);
        $result = $guesser->guess('Tom');
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->assertNull($result);

        $result = $guesser->guess('lkjlkjlkj');
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->assertSame('f', $result['gender']);

        $result = $guesser->guess('Sonja');
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->assertNull($result);
        $result = $guesser->guess('Petra');
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->assertNull($result);
    }

    public function testGuesserWithEitherWeight()
    {
        $females = array();
        $males = array();
        $matchList = array('either_weight');

        // use with extra names supplied via constructor
        $guesser = new AzineGenderGuesser($females, $males, false, $matchList);

        $result = $guesser->guess('Randy');
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->assertSame('m', $result['gender']);
    }

    public function testGuesserWithRules1()
    {
        $females = array();
        $males = array();
        $matchList = array('v1_rules');

        // use with extra names supplied via constructor
        $guesser = new AzineGenderGuesser($females, $males, false, $matchList);

        $result = $guesser->guess('Randy');
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->assertSame('m', $result['gender']);

        $result = $guesser->guess('Lois');
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->assertSame('f', $result['gender']);
    }

    public function testGuesserWithRules2()
    {
        $females = array();
        $males = array();
        $matchList = array('v2_rules');

        // use with extra names supplied via constructor
        $guesser = new AzineGenderGuesser($females, $males, false, $matchList);

        $result = $guesser->guess('John');
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->assertSame('m', $result['gender']);
        $result = $guesser->guess('Anfernee');
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->assertSame('m', $result['gender']);

        $result = $guesser->guess('Trish');
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->assertSame('f', $result['gender']);
        $result = $guesser->guess('Ellie');
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->assertSame('f', $result['gender']);
    }

    public function testGuesserWithMetaPhone()
    {
        $females = array();
        $males = array();
        $matchList = array('one_only_metaphone');

        // use with extra names supplied via constructor
        $guesser = new AzineGenderGuesser($females, $males, true, $matchList);

        $result = $guesser->guess('Patric');
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->assertSame('m', $result['gender']);
        $result = $guesser->guess('Tomàs');
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->assertSame('m', $result['gender']);

        $result = $guesser->guess('Sonja');
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->assertSame('f', $result['gender']);
    }

    public function testGuesserWithMetaPhone2()
    {
        $females = array();
        $males = array();
        $matchList = array('either_weight_metaphone');

        // use with extra names supplied via constructor
        $guesser = new AzineGenderGuesser($females, $males, true, $matchList);

        $result = $guesser->guess('Patric');
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->assertSame('m', $result['gender']);
        $result = $guesser->guess('Tomàs');
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->assertSame('m', $result['gender']);

        $result = $guesser->guess('Sonja');
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->assertSame('f', $result['gender']);
    }
}


1			<?php
2
3			namespace Azine\HybridAuthBundle\Tests\Services;
4
5			use Azine\HybridAuthBundle\Services\AzineGenderGuesser;
6			use Azine\HybridAuthBundle\Tests\AzineTestCase;
7
8			class AzineGenderGuesserTest extends AzineTestCase
9			{
10			public function testGuess()
11			{
12			$guesser = new AzineGenderGuesser();
13			$result = $guesser->guess('Patrick');
			0 ignored issues – show Documentation introduced 2015-11-24 12:21 UTC by Report Bug Copy Issue Report `'Patrick'` is of type `string`, but the function expects a `object<string>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
14			$this->assertSame('m', $result['gender']);
15			$result = $guesser->guess('Tom');
			0 ignored issues – show Documentation introduced 2015-11-24 12:21 UTC by Report Bug Copy Issue Report `'Tom'` is of type `string`, but the function expects a `object<string>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
16			$this->assertSame('m', $result['gender']);
17
18			$result = $guesser->guess('Sonja');
			0 ignored issues – show Documentation introduced 2015-11-24 12:21 UTC by Report Bug Copy Issue Report `'Sonja'` is of type `string`, but the function expects a `object<string>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
19			$this->assertSame('f', $result['gender']);
20			$result = $guesser->guess('Petra');
			0 ignored issues – show Documentation introduced 2015-11-24 12:21 UTC by Report Bug Copy Issue Report `'Petra'` is of type `string`, but the function expects a `object<string>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
21			$this->assertSame('f', $result['gender']);
22
23			$this->assertNull($guesser->guess(null));
			0 ignored issues – show Documentation introduced 2015-11-24 12:21 UTC by Report Bug Copy Issue Report `null` is of type `null`, but the function expects a `object<string>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
24			}
25
26			public function testGender()
27			{
28			$guesser = new AzineGenderGuesser();
29			$this->assertSame('m', $guesser->gender('Patrick'));
			0 ignored issues – show Documentation introduced 2018-04-05 07:37 UTC by Report Bug Copy Issue Report `'Patrick'` is of type `string`, but the function expects a `object<string>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
30			$this->assertSame('m', $guesser->gender('Tom'));
			0 ignored issues – show Documentation introduced 2018-04-05 07:37 UTC by Report Bug Copy Issue Report `'Tom'` is of type `string`, but the function expects a `object<string>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
31			$this->assertSame('f', $guesser->gender('Sonja'));
			0 ignored issues – show Documentation introduced 2018-04-05 07:37 UTC by Report Bug Copy Issue Report `'Sonja'` is of type `string`, but the function expects a `object<string>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
32			$this->assertSame('f', $guesser->gender('Petra'));

azine / AzineHybridAuthBundle

Pull Request — master (#11)

AzineGenderGuesserTest A

Complexity

Size/Duplication

Coupling/Cohesion

Importance

8 Methods

How to fix Duplicated Code

Duplicated Code

Duplication Side-by-Side

Filter issues like