StopwordAnalyzerTest::testToLoadFromCache() - Code Metrics - Inspection of "Tidy" - onoi/tesa - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 83b1b2...0f3bb4 )

by mw

created 2015-12-08 21:18 UTC

StopwordAnalyzerTest::testToLoadFromCache() A

↳ Parent: StopwordAnalyzerTest

Complexity

Conditions	1
Paths	1

Size

Total Lines	18
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	18
rs	9.4286
cc	1
eloc	13
nc	1
nop	0

<?php

namespace Onoi\Tesa\Tests;

use Onoi\Tesa\StopwordAnalyzer;

/**
 * @covers \Onoi\Tesa\StopwordAnalyzer
 * @group onoi-tesa
 *
 * @license GNU GPL v2+
 * @since 0.1
 *
 * @author mwjames
 */
class StopwordAnalyzerTest extends \PHPUnit_Framework_TestCase {

	public function testCanConstruct() {

		$this->assertInstanceOf(
			'\Onoi\Tesa\StopwordAnalyzer',
			new StopwordAnalyzer()
		);
	}

	public function testCustomStopwordList() {

		$instance = new StopwordAnalyzer();

		$instance->setCustomStopwordList(
			array(
				'zoo' => array( 'Foo' ),
				'doo' => array( 'bar' )
			)
		);

		$this->assertTrue(
			$instance->isStopWord( 'Foo' )
		);

		$this->assertEquals(
			array( 'zoo', 'doo' ),
			$instance->getLanguageList()
		);
	}

	/**
	 * @dataProvider defaultListStopWordsProvider
	 */
	public function testUncachedIsWordCheckForDefaultList( $word, $expected ) {

		$instance = new StopwordAnalyzer();
		$instance->loadListByDefaultLanguages();

		$this->assertEquals(
			$expected,
			$instance->isStopWord( $word )
		);
	}

	/**
	 * @dataProvider listByLanguageStopWordsProvider
	 */
	public function testListByLanguage( $languageCode, $word, $expected ) {

		$instance = new StopwordAnalyzer();
		$instance->loadListByLanguage( $languageCode );

		$this->assertEquals(
			$expected,
			$instance->isStopWord( $word )
		);

		$this->assertEquals(
			(array)$languageCode,
			$instance->getLanguageList()
		);
	}

	/**
	 * @dataProvider listByLanguageStopWordsProvider
	 */
	public function testListFromCustomLocation( $languageCode, $word, $expected ) {

		$instance = new StopwordAnalyzer();
		$instance->loadListFromCustomLocation( __DIR__ . '/../../../data/stopwords/', $languageCode );

		$this->assertEquals(
			$expected,
			$instance->isStopWord( $word )
		);
	}

	public function testTryToLoadUnlistedLanguage() {

		$instance = new StopwordAnalyzer();
		$instance->loadListByLanguage( 'foo' );

		$this->assertFalse(
			$instance->isStopWord( 'Bar' )
		);
	}

	public function testToLoadFromCache() {

		$cache = $this->getMockBuilder( '\Onoi\Cache\Cache' )
			->disableOriginalConstructor()
			->setMethods( array( 'contains', 'fetch' ) )
			->getMockForAbstractClass();

		$cache->expects( $this->once() )
			->method( 'contains' )
			->with( $this->stringContains( 'onoi:tesa:stopword' ) )
			->will( $this->returnValue( true ) );

		$cache->expects( $this->once() )
			->method( 'fetch' );

		$instance = new StopwordAnalyzer( $cache );
		$instance->loadListByLanguage( 'foo' );
	}

	public function defaultListStopWordsProvider() {

		$provider[] = array(
foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

    if ($item->hasBar()) {
        $myArray['bar'] = $item->getBar();
    }

    // do something with $myArray
}
			'Foo',
			false
		);

		// en-list
		$provider[] = array(
			'about',
			true
		);

		// de-list
		$provider[] = array(
			'über',
			true
		);

		// fr-list
		$provider[] = array(
			'avoir',
			true
		);

		// es-list
		$provider[] = array(
			'aquellos',
			true
		);

		// ja-list (not default)
		$provider[] = array(
			'かつて',
			false
		);

		return $provider;
	}

	public function listByLanguageStopWordsProvider() {

		// ja-list
		$provider[] = array(
foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

    if ($item->hasBar()) {
        $myArray['bar'] = $item->getBar();
    }

    // do something with $myArray
}
			'ja',
			'かつて',
			true
		);

		return $provider;
	}

}


1			<?php
2
3			namespace Onoi\Tesa\Tests;
4
5			use Onoi\Tesa\StopwordAnalyzer;
6
7			/**
8			* @covers \Onoi\Tesa\StopwordAnalyzer
9			* @group onoi-tesa
10			*
11			* @license GNU GPL v2+
12			* @since 0.1
13			*
14			* @author mwjames
15			*/
16			class StopwordAnalyzerTest extends \PHPUnit_Framework_TestCase {
17
18			public function testCanConstruct() {
19
20			$this->assertInstanceOf(
21			'\Onoi\Tesa\StopwordAnalyzer',
22			new StopwordAnalyzer()
23			);
24			}
25
26			public function testCustomStopwordList() {
27
28			$instance = new StopwordAnalyzer();
29
30			$instance->setCustomStopwordList(
31			array(
32			'zoo' => array( 'Foo' ),
33			'doo' => array( 'bar' )
34			)
35			);
36
37			$this->assertTrue(
38			$instance->isStopWord( 'Foo' )
39			);
40
41			$this->assertEquals(
42			array( 'zoo', 'doo' ),
43			$instance->getLanguageList()
44			);
45			}
46
47			/**
48			* @dataProvider defaultListStopWordsProvider
49			*/
50			public function testUncachedIsWordCheckForDefaultList( $word, $expected ) {
51
52			$instance = new StopwordAnalyzer();
53			$instance->loadListByDefaultLanguages();
54
55			$this->assertEquals(
56			$expected,
57			$instance->isStopWord( $word )
58			);
59			}
60
61			/**
62			* @dataProvider listByLanguageStopWordsProvider
63			*/
64			public function testListByLanguage( $languageCode, $word, $expected ) {
65
66			$instance = new StopwordAnalyzer();
67			$instance->loadListByLanguage( $languageCode );
68
69			$this->assertEquals(
70			$expected,
71			$instance->isStopWord( $word )
72			);
73
74			$this->assertEquals(
75			(array)$languageCode,
76			$instance->getLanguageList()
77			);
78			}
79
80			/**
81			* @dataProvider listByLanguageStopWordsProvider
82			*/
83			public function testListFromCustomLocation( $languageCode, $word, $expected ) {
84
85			$instance = new StopwordAnalyzer();
86			$instance->loadListFromCustomLocation( __DIR__ . '/../../../data/stopwords/', $languageCode );
87
88			$this->assertEquals(
89			$expected,
90			$instance->isStopWord( $word )
91			);
92			}
93
94			public function testTryToLoadUnlistedLanguage() {
95
96			$instance = new StopwordAnalyzer();
97			$instance->loadListByLanguage( 'foo' );
98
99			$this->assertFalse(
100			$instance->isStopWord( 'Bar' )
101			);
102			}
103
104			public function testToLoadFromCache() {
105
106			$cache = $this->getMockBuilder( '\Onoi\Cache\Cache' )
107			->disableOriginalConstructor()
108			->setMethods( array( 'contains', 'fetch' ) )
109			->getMockForAbstractClass();
110
111			$cache->expects( $this->once() )
112			->method( 'contains' )
113			->with( $this->stringContains( 'onoi:tesa:stopword' ) )
114			->will( $this->returnValue( true ) );
115
116			$cache->expects( $this->once() )
117			->method( 'fetch' );
118
119			$instance = new StopwordAnalyzer( $cache );
120			$instance->loadListByLanguage( 'foo' );
121			}
122
123			public function defaultListStopWordsProvider() {
124
125			$provider[] = array(
			0 ignored issues – show Coding Style Comprehensibility introduced 2015-12-01 19:40 UTC by Report Bug Copy Issue Report `$provider` was never initialized. Although not strictly required by PHP, it is generally a good practice to add `$provider = array();` before regardless. Adding an explicit array definition is generally preferable to implicit array definition as it guarantees a stable state of the code. Let’s take a look at an example: foreach ($collection as $item) { $myArray['foo'] = $item->getFoo(); if ($item->hasBar()) { $myArray['bar'] = $item->getBar(); } // do something with $myArray } As you can see in this example, the array `$myArray` is initialized the first time when the foreach loop is entered. You can also see that the value of the `bar` key is only written conditionally; thus, its value might result from a previous iteration. This might or might not be intended. To make your intention clear, your code more readible and to avoid accidental bugs, we recommend to add an explicit initialization `$myArray = array()` either outside or inside the foreach loop. Loading history...
126			'Foo',
127			false
128			);
129
130			// en-list
131			$provider[] = array(
132			'about',
133			true
134			);
135
136			// de-list
137			$provider[] = array(
138			'über',
139			true
140			);
141
142			// fr-list
143			$provider[] = array(
144			'avoir',
145			true
146			);
147
148			// es-list
149			$provider[] = array(
150			'aquellos',
151			true
152			);
153
154			// ja-list (not default)
155			$provider[] = array(
156			'かつて',
157			false
158			);
159
160			return $provider;
161			}
162
163			public function listByLanguageStopWordsProvider() {
164
165			// ja-list
166			$provider[] = array(
			0 ignored issues – show Coding Style Comprehensibility introduced 2015-12-01 19:40 UTC by Report Bug Copy Issue Report `$provider` was never initialized. Although not strictly required by PHP, it is generally a good practice to add `$provider = array();` before regardless. Adding an explicit array definition is generally preferable to implicit array definition as it guarantees a stable state of the code. Let’s take a look at an example: foreach ($collection as $item) { $myArray['foo'] = $item->getFoo(); if ($item->hasBar()) { $myArray['bar'] = $item->getBar(); } // do something with $myArray } As you can see in this example, the array `$myArray` is initialized the first time when the foreach loop is entered. You can also see that the value of the `bar` key is only written conditionally; thus, its value might result from a previous iteration. This might or might not be intended. To make your intention clear, your code more readible and to avoid accidental bugs, we recommend to add an explicit initialization `$myArray = array()` either outside or inside the foreach loop. Loading history...
167			'ja',
168			'かつて',
169			true
170			);
171
172			return $provider;
173			}
174
175			}
176

onoi / tesa

Push — master ( 83b1b2...0f3bb4 )

StopwordAnalyzerTest::testToLoadFromCache() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like