StringsTest.php$1 ➔ compressIgnoresNewLinesByDefault() - Code Metrics - Inspection of "Merge pull request #12 from chadicus/fea/compress" - traderinteractive/filter-strings-php - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 5b69ef...fa3943 )

by Chad

created 2019-03-20 17:33 UTC

php$1 ➔ compressIgnoresNewLinesByDefault() A

↳ Parent: StringsTest

Complexity

Conditions

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
dl	0
loc	6
rs	10
c	0
b	0
f	0

<?php

namespace TraderInteractive\Filter;

use InvalidArgumentException;
use PHPUnit\Framework\TestCase;
use TraderInteractive\Exceptions\FilterException;

/**
 * @coversDefaultClass \TraderInteractive\Filter\Strings
 * @covers ::<private>
 */
final class StringsTest extends TestCase
{
    /**
     * Verify basic use of filter
     *
     * @test
     * @covers ::filter
     * @dataProvider filterData
     *
     * @param mixed $input    The input.
     * @param mixed $expected The expected value(s).
     *
     * @return void
     * @throws FilterException
     */
    public function filter($input, $expected)
    {
        $this->assertSame($expected, Strings::filter($input));
    }

    /**
     * Data provider for basic filter tests
     *
     * @return array
     */
    public function filterData()
    {
        return [
            'string' => ['abc', 'abc'],
            'int' => [1, '1'],
            'float' => [1.1, '1.1'],
            'bool' => [true, '1'],
            'object' => [new \SplFileInfo(__FILE__), __FILE__],
        ];
    }

    /**
     * @test
     * @covers ::filter
     */
    public function filterNullPass()
    {
        $this->assertNull(Strings::filter(null, true));
    }

    /**
     * @test
     * @expectedException \TraderInteractive\Exceptions\FilterException
     * @expectedExceptionMessage Value failed filtering, $allowNull is set to false
     * @covers ::filter
     */
    public function filterNullFail()
    {
        Strings::filter(null);
    }

    /**
     * @test
     * @covers ::filter
     */
    public function filterMinLengthPass()
    {
        $this->assertSame('a', Strings::filter('a'));
    }

    /**
     * @test
     * @expectedException \TraderInteractive\Exceptions\FilterException
     * @covers ::filter
     */
    public function filterMinLengthFail()
    {
        Strings::filter('');
    }

    /**
     * @test
     * @covers ::filter
     */
    public function filterMaxLengthPass()
    {
        $this->assertSame('a', Strings::filter('a', false, 0, 1));
    }

    /**
     * @test
     * @expectedException \TraderInteractive\Exceptions\FilterException
     * @expectedExceptionMessage Value 'a' with length '1' is less than '0' or greater than '0'
     * @covers ::filter
     */
    public function filterMaxLengthFail()
    {
        Strings::filter('a', false, 0, 0);
    }

    /**
     * @test
     * @expectedException InvalidArgumentException
     * @expectedExceptionMessage $minLength was not a positive integer value
     * @covers ::filter
     */
    public function filterMinLengthNotInteger()
    {
        Strings::filter('a', false, -1);
    }

    /**
     * @test
     * @expectedException InvalidArgumentException
     * @expectedExceptionMessage $maxLength was not a positive integer value
     * @covers ::filter
     */
    public function filterMaxLengthNotInteger()
    {
        Strings::filter('a', false, 1, -1);
    }

    /**
     * @test
     * @expectedException InvalidArgumentException
     * @expectedExceptionMessage $minLength was not a positive integer value
     * @covers ::filter
     */
    public function filterMinLengthNegative()
    {
        Strings::filter('a', false, -1);
    }

    /**
     * @test
     * @expectedException InvalidArgumentException
     * @expectedExceptionMessage $maxLength was not a positive integer value
     * @covers ::filter
     */
    public function filterMaxLengthNegative()
    {
        Strings::filter('a', false, 1, -1);
    }

    /**
     * @test
     * @covers ::filter
     */
    public function filterWithScalar()
    {
        $this->assertSame('24141', Strings::filter(24141));
    }

    /**
     * @test
     * @covers ::filter
     */
    public function filterWithObject()
    {
        $testObject = new class() {
            private $data;

            public function __construct()
            {
                $this->data = [1,2,3,4,5];
            }

            public function __toString()
            {
                return implode(',', $this->data);
            }
        };

        $this->assertSame('1,2,3,4,5', Strings::filter(new $testObject));
    }

    /**
     * @test
     * @covers ::filter
     *
     * @expectedException \TraderInteractive\Exceptions\FilterException
     * @expectedExceptionMessage Value 'class@anonymous
     */
    public function filterWithObjectNoToStringMethod()
    {
        $testObject = new class() {
            private $data;

            public function __construct()
            {
                $this->data = [1, 2, 3, 4, 5];
            }
        };

        Strings::filter(new $testObject);
    }

    /**
     * @test
     * @covers ::translate
     */
    public function translateValue()
    {
        $map = ['foo' => '100', 'bar' => '200'];
        $this->assertSame('100', Strings::translate('foo', $map));
    }

    /**
     * @test
     * @covers ::translate
     * @expectedException \TraderInteractive\Exceptions\FilterException
     * @expectedExceptionMessage The value 'baz' was not found in the translation map array.
     */
    public function translateValueNotFoundInMap()
    {
        $map = ['foo' => '100', 'bar' => '200'];
        Strings::translate('baz', $map);
    }

    /**
     * Verifies basic explode functionality.
     *
     * @test
     * @covers ::explode
     */
    public function explode()
    {
        $this->assertSame(['a', 'bcd', 'e'], Strings::explode('a,bcd,e'));
    }

    /**
     * Verifies explode with a custom delimiter.
     *
     * @test
     * @covers ::explode
     */
    public function explodeCustomDelimiter()
    {
        $this->assertSame(['a', 'b', 'c', 'd,e'], Strings::explode('a b c d,e', ' '));
    }

    /**
     * @test
     * @expectedException \TraderInteractive\Exceptions\FilterException
     * @expectedExceptionMessage Value '1234' is not a string
     * @covers ::explode
     */
    public function explodeNonString()
    {
        Strings::explode(1234, '');
    }

    /**
     * Verifies explode filter with an empty delimiter.
     *
     * @test
     * @expectedException \InvalidArgumentException
     * @expectedExceptionMessage Delimiter '''' is not a non-empty string
     * @covers ::explode
     */
    public function explodeEmptyDelimiter()
    {
        Strings::explode('test', '');
    }

    /**
     * @test
     * @covers ::stripTags
     * @dataProvider provideStripTags
     *
     * @param string|null $value
     * @param string      $replacement
     * @param string|null $expected
     */
    public function stripTags($value, string $replacement, $expected)
    {
        $actual = Strings::stripTags($value, $replacement);
        $this->assertSame($expected, $actual);
    }

    /**
     * @return array
     */
    public function provideStripTags()
    {
        return [
            'null returns null' => [
                'value' => null,
                'replacement' => '',
                'expected' => null,
            ],
            'remove html from string' => [
                'value' => 'A string with <p>paragraph</p> tags',
                'replacement' => '',
                'expected' => 'A string with paragraph tags',
            ],
            'remove xml and replace with space' => [
                'value' => '<something>inner value</something>',
                'replacement' => ' ',
                'expected' => ' inner value ',
            ],
            'remove multiline html from string' => [
                'value' => "<p\nclass='something'\nstyle='display:none'></p>",
                'replacement' => ' ',
                'expected' => '  ',
            ],
            'remove php tags' => [
                'value' => '<?php some php code',
                'replacement' => ' ',
                'expected' => '',
            ],
            'remove shorthand php tags' => [
                'value' => '<?= some php code ?> something else',
                'replacement' => ' ',
                'expected' => '  something else',
            ],
            'do not remove unmatched <' => [
                'value' => '1 < 3',
                'replacement' => ' ',
                'expected' => '1 < 3',
            ],
            'do not remove unmatched >' => [
                'value' => '3 > 1',
                'replacement' => ' ',
                'expected' => '3 > 1',
            ],
        ];
    }

    /**
     * @test
     * @covers ::concat
     */
    public function concat()
    {
        $this->assertSame('prefixstringsuffix', Strings::concat('string', 'prefix', 'suffix'));
    }

    /**
     * Verify behavior of concat() when $value is not filterable
     *
     * @test
     * @covers ::concat
     * @expectedException \TraderInteractive\Exceptions\FilterException
     *
     * @return void
     */
    public function concatValueNotFilterable()
    {
        Strings::concat(new \StdClass(), 'prefix', 'suffix');
    }

    /**
     * @test
     * @covers ::concat
     */
    public function concatScalarValue()
    {
        $this->assertSame('prefix123suffix', Strings::concat(123, 'prefix', 'suffix'));
    }

    /**
     * @test
     * @covers ::concat
     */
    public function concatObjectValue()
    {
        $this->assertSame('prefix' . __FILE__ . 'suffix', Strings::concat(new \SplFileInfo(__FILE__), 'prefix', 'suffix'));
    }

    /**
     * @test
     * @covers ::compress
     */
    public function compressRemovesSuperfluousWhitespace()
    {
        $this->assertSame('a compressed string', Strings::compress('  a   compressed     string    '));
    }

    /**
     * @test
     * @covers ::compress
     */
    public function compressReturnsNullIfValueIsNull()
    {
        $this->assertNull(Strings::compress(null));
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {
    }

    /**
     * @test
     * @covers ::compress
     */
    public function compressRemovesNewLines()
    {
        $input = " This string\nhas     superfluous   whitespace  and  \nnewlines\n";
        $this->assertSame(
            'This string has superfluous whitespace and newlines',
            Strings::compress($input, true)
        );
    }

    /**
     * @test
     * @covers ::compress
     */
    public function compressIgnoresNewLinesByDefault()
    {
        $input = " This string\nhas     superfluous   whitespace  and  \nnewlines\n";
        $this->assertSame(
            "This string\nhas superfluous whitespace and \nnewlines",
            Strings::compress($input)
        );
    }

    /**
     * @test
     * @covers ::redact
     * @dataProvider provideRedact
     *
     * @param string|null    $value       The value to pass to the filter.
     * @param array|callable $words       The words to pass to the filter.
     * @param string         $replacement The replacement to pass to the filter.
     * @param string|null    $expected    The expected result.
     */
    public function redact($value, $words, string $replacement, $expected)
    {
        $actual = Strings::redact($value, $words, $replacement);

        $this->assertSame($expected, $actual);
    }

    /**
     * @return array
     */
    public function provideRedact() : array
    {
        return [
            'null value' => [
                'value' => null,
                'words' => [],
                'replacement' => '',
                'expected' => null,
            ],
            'empty string' => [
                'value' => '',
                'words' => [],
                'replacement' => '',
                'expected' => '',
            ],
            'replace with empty' => [
                'value' => 'this message contains something that you want removed',
                'words' => ['something that you want removed'],
                'replacement' => '',
                'expected' => 'this message contains ',
            ],
            'replace with *' => [
                'value' => 'replace certain words that you might want to remove',
                'words' => ['might', 'certain'],
                'replacement' => '*',
                'expected' => 'replace ******* words that you ***** want to remove',
            ],
            'replace with █' => [
                'value' => 'redact specific dates and secret locations',
                'words' => ['secret locations', 'specific dates'],
                'replacement' => '█',
                'expected' => 'redact ██████████████ and ████████████████',
            ],
            'replace with multi-character string uses first character' => [
                'value' => 'replace some particular words',
                'words' => ['particular', 'words', 'some'],
                'replacement' => ' *** ',
                'expected' => 'replace                      ',
            ],
            'no replacements' => [
                'value' => 'some perfectly normal string',
                'words' => ['undesired', 'words'],
                'replacement' => '*',
                'expected' => 'some perfectly normal string',
            ],
            'closure provides words' => [
                'value' => 'doe a deer, a female deer',
                'words' => function () {
                    return ['doe', 'deer'];
                },
                'replacement' => '-',
                'expected' => '--- a ----, a female ----',
            ],
        ];
    }

    /**
     * @test
     * @covers ::redact
     * @dataProvider provideRedactFailsOnBadInput
     *
     * @param mixed  $value       The value to pass to the filter.
     * @param mixed  $words       The words to pass to the filter.
     * @param string $replacement The replacement to pass to the filter.
     * @param string $exception   The exception to expect.
     * @param string $message     The exception message to expect.
     */
    public function redactFailsOnBadInput($value, $words, string $replacement, string $exception, string $message)
    {
        $this->expectException($exception);
        $this->expectExceptionMessage($message);

        Strings::redact($value, $words, $replacement);
    }

    /**
     * @return array
     */
    public function provideRedactFailsOnBadInput() : array
    {
        return [
            'non-string value' => [
                'value' => ['bad', 'input'],
                'words' => [],
                'replacement' => '',
                'exception' => FilterException::class,
                'message' => "Value '" . var_export(['bad', 'input'], true) . "' is not a string",
            ],
            'invalid words argument' => [
                'value' => 'some string',
                'words' => 'this is not valid',
                'replacement' => '',
                'exception' => FilterException::class,
                'message' => 'Words was not an array or a callable that returns an array',
            ],
            'invalid return from callable words argument' => [
                'value' => 'some string',
                'words' => function () {
                    return 'this is also not valid';
                },
                'replacement' => '',
                'exception' => FilterException::class,
                'message' => 'Words was not an array or a callable that returns an array',
            ],
        ];
    }
}


1			<?php
2
3			namespace TraderInteractive\Filter;
4
5			use InvalidArgumentException;
6			use PHPUnit\Framework\TestCase;
7			use TraderInteractive\Exceptions\FilterException;
8
9			/**
10			* @coversDefaultClass \TraderInteractive\Filter\Strings
11			* @covers ::<private>
12			*/
13			final class StringsTest extends TestCase
14			{
15			/**
16			* Verify basic use of filter
17			*
18			* @test
19			* @covers ::filter
20			* @dataProvider filterData
21			*
22			* @param mixed $input The input.
23			* @param mixed $expected The expected value(s).
24			*
25			* @return void
26			* @throws FilterException
27			*/
28			public function filter($input, $expected)
29			{
30			$this->assertSame($expected, Strings::filter($input));
31			}
32
33			/**
34			* Data provider for basic filter tests
35			*
36			* @return array
37			*/
38			public function filterData()
39			{
40			return [
41			'string' => ['abc', 'abc'],
42			'int' => [1, '1'],
43			'float' => [1.1, '1.1'],
44			'bool' => [true, '1'],
45			'object' => [new \SplFileInfo(__FILE__), __FILE__],
46			];
47			}
48
49			/**
50			* @test
51			* @covers ::filter
52			*/
53			public function filterNullPass()
54			{
55			$this->assertNull(Strings::filter(null, true));
56			}
57
58			/**
59			* @test
60			* @expectedException \TraderInteractive\Exceptions\FilterException
61			* @expectedExceptionMessage Value failed filtering, $allowNull is set to false
62			* @covers ::filter
63			*/
64			public function filterNullFail()
65			{
66			Strings::filter(null);
67			}
68
69			/**
70			* @test
71			* @covers ::filter
72			*/
73			public function filterMinLengthPass()
74			{
75			$this->assertSame('a', Strings::filter('a'));
76			}
77
78			/**
79			* @test
80			* @expectedException \TraderInteractive\Exceptions\FilterException
81			* @covers ::filter
82			*/
83			public function filterMinLengthFail()
84			{
85			Strings::filter('');
86			}
87
88			/**
89			* @test
90			* @covers ::filter
91			*/
92			public function filterMaxLengthPass()
93			{
94			$this->assertSame('a', Strings::filter('a', false, 0, 1));
95			}
96
97			/**
98			* @test
99			* @expectedException \TraderInteractive\Exceptions\FilterException
100			* @expectedExceptionMessage Value 'a' with length '1' is less than '0' or greater than '0'
101			* @covers ::filter
102			*/
103			public function filterMaxLengthFail()
104			{
105			Strings::filter('a', false, 0, 0);
106			}
107
108			/**
109			* @test
110			* @expectedException InvalidArgumentException
111			* @expectedExceptionMessage $minLength was not a positive integer value
112			* @covers ::filter
113			*/
114			public function filterMinLengthNotInteger()
115			{
116			Strings::filter('a', false, -1);
117			}
118
119			/**
120			* @test
121			* @expectedException InvalidArgumentException
122			* @expectedExceptionMessage $maxLength was not a positive integer value
123			* @covers ::filter
124			*/
125			public function filterMaxLengthNotInteger()
126			{
127			Strings::filter('a', false, 1, -1);
128			}
129
130			/**
131			* @test
132			* @expectedException InvalidArgumentException
133			* @expectedExceptionMessage $minLength was not a positive integer value
134			* @covers ::filter
135			*/
136			public function filterMinLengthNegative()
137			{
138			Strings::filter('a', false, -1);
139			}
140
141			/**
142			* @test
143			* @expectedException InvalidArgumentException
144			* @expectedExceptionMessage $maxLength was not a positive integer value
145			* @covers ::filter
146			*/
147			public function filterMaxLengthNegative()
148			{
149			Strings::filter('a', false, 1, -1);
150			}
151
152			/**
153			* @test
154			* @covers ::filter
155			*/
156			public function filterWithScalar()
157			{
158			$this->assertSame('24141', Strings::filter(24141));
159			}
160
161			/**
162			* @test
163			* @covers ::filter
164			*/
165			public function filterWithObject()
166			{
167			$testObject = new class() {
168			private $data;
169
170			public function __construct()
171			{
172			$this->data = [1,2,3,4,5];
173			}
174
175			public function __toString()
176			{
177			return implode(',', $this->data);
178			}
179			};
180
181			$this->assertSame('1,2,3,4,5', Strings::filter(new $testObject));
182			}
183
184			/**
185			* @test
186			* @covers ::filter
187			*
188			* @expectedException \TraderInteractive\Exceptions\FilterException
189			* @expectedExceptionMessage Value 'class@anonymous
190			*/
191			public function filterWithObjectNoToStringMethod()
192			{
193			$testObject = new class() {
194			private $data;
195
196			public function __construct()
197			{
198			$this->data = [1, 2, 3, 4, 5];
199			}
200			};
201
202			Strings::filter(new $testObject);
203			}
204
205			/**
206			* @test
207			* @covers ::translate
208			*/
209			public function translateValue()
210			{
211			$map = ['foo' => '100', 'bar' => '200'];
212			$this->assertSame('100', Strings::translate('foo', $map));
213			}
214
215			/**
216			* @test
217			* @covers ::translate
218			* @expectedException \TraderInteractive\Exceptions\FilterException
219			* @expectedExceptionMessage The value 'baz' was not found in the translation map array.
220			*/
221			public function translateValueNotFoundInMap()
222			{
223			$map = ['foo' => '100', 'bar' => '200'];
224			Strings::translate('baz', $map);
225			}
226
227			/**
228			* Verifies basic explode functionality.
229			*
230			* @test
231			* @covers ::explode
232			*/
233			public function explode()
234			{
235			$this->assertSame(['a', 'bcd', 'e'], Strings::explode('a,bcd,e'));
236			}
237
238			/**
239			* Verifies explode with a custom delimiter.
240			*
241			* @test
242			* @covers ::explode
243			*/
244			public function explodeCustomDelimiter()
245			{
246			$this->assertSame(['a', 'b', 'c', 'd,e'], Strings::explode('a b c d,e', ' '));
247			}
248
249			/**
250			* @test
251			* @expectedException \TraderInteractive\Exceptions\FilterException
252			* @expectedExceptionMessage Value '1234' is not a string
253			* @covers ::explode
254			*/
255			public function explodeNonString()
256			{
257			Strings::explode(1234, '');
258			}
259
260			/**
261			* Verifies explode filter with an empty delimiter.
262			*
263			* @test
264			* @expectedException \InvalidArgumentException
265			* @expectedExceptionMessage Delimiter '''' is not a non-empty string
266			* @covers ::explode
267			*/
268			public function explodeEmptyDelimiter()
269			{
270			Strings::explode('test', '');
271			}
272
273			/**
274			* @test
275			* @covers ::stripTags
276			* @dataProvider provideStripTags
277			*
278			* @param string\|null $value
279			* @param string $replacement
280			* @param string\|null $expected
281			*/
282			public function stripTags($value, string $replacement, $expected)
283			{
284			$actual = Strings::stripTags($value, $replacement);
285			$this->assertSame($expected, $actual);
286			}
287
288			/**
289			* @return array
290			*/
291			public function provideStripTags()
292			{
293			return [
294			'null returns null' => [
295			'value' => null,
296			'replacement' => '',
297			'expected' => null,
298			],
299			'remove html from string' => [
300			'value' => 'A string with <p>paragraph</p> tags',
301			'replacement' => '',
302			'expected' => 'A string with paragraph tags',
303			],
304			'remove xml and replace with space' => [
305			'value' => '<something>inner value</something>',
306			'replacement' => ' ',
307			'expected' => ' inner value ',
308			],
309			'remove multiline html from string' => [
310			'value' => "<p\nclass='something'\nstyle='display:none'></p>",
311			'replacement' => ' ',
312			'expected' => ' ',
313			],
314			'remove php tags' => [
315			'value' => '<?php some php code',
316			'replacement' => ' ',
317			'expected' => '',
318			],
319			'remove shorthand php tags' => [
320			'value' => '<?= some php code ?> something else',
321			'replacement' => ' ',
322			'expected' => ' something else',
323			],
324			'do not remove unmatched <' => [
325			'value' => '1 < 3',
326			'replacement' => ' ',
327			'expected' => '1 < 3',
328			],
329			'do not remove unmatched >' => [
330			'value' => '3 > 1',
331			'replacement' => ' ',
332			'expected' => '3 > 1',
333			],
334			];
335			}
336
337			/**
338			* @test
339			* @covers ::concat
340			*/
341			public function concat()
342			{
343			$this->assertSame('prefixstringsuffix', Strings::concat('string', 'prefix', 'suffix'));
344			}
345
346			/**
347			* Verify behavior of concat() when $value is not filterable
348			*
349			* @test
350			* @covers ::concat
351			* @expectedException \TraderInteractive\Exceptions\FilterException
352			*
353			* @return void
354			*/
355			public function concatValueNotFilterable()
356			{
357			Strings::concat(new \StdClass(), 'prefix', 'suffix');
358			}
359
360			/**
361			* @test
362			* @covers ::concat
363			*/
364			public function concatScalarValue()
365			{
366			$this->assertSame('prefix123suffix', Strings::concat(123, 'prefix', 'suffix'));
367			}
368
369			/**
370			* @test
371			* @covers ::concat
372			*/
373			public function concatObjectValue()
374			{
375			$this->assertSame('prefix' . __FILE__ . 'suffix', Strings::concat(new \SplFileInfo(__FILE__), 'prefix', 'suffix'));
376			}
377
378			/**
379			* @test
380			* @covers ::compress
381			*/
382			public function compressRemovesSuperfluousWhitespace()
383			{
384			$this->assertSame('a compressed string', Strings::compress(' a compressed string '));
385			}
386
387			/**
388			* @test
389			* @covers ::compress
390			*/
391			public function compressReturnsNullIfValueIsNull()
392			{
393			$this->assertNull(Strings::compress(null));
			0 ignored issues – show Bug introduced 2019-03-20 15:49 UTC by Report Bug Copy Issue Report Are you sure the usage of `TraderInteractive\Filter\Strings::compress(null)` targeting `TraderInteractive\Filter\Strings::compress()` 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...
394			}
395
396			/**
397			* @test
398			* @covers ::compress
399			*/
400			public function compressRemovesNewLines()
401			{
402			$input = " This string\nhas superfluous whitespace and \nnewlines\n";
403			$this->assertSame(
404			'This string has superfluous whitespace and newlines',
405			Strings::compress($input, true)
406			);
407			}
408
409			/**
410			* @test
411			* @covers ::compress
412			*/
413			public function compressIgnoresNewLinesByDefault()
414			{
415			$input = " This string\nhas superfluous whitespace and \nnewlines\n";
416			$this->assertSame(
417			"This string\nhas superfluous whitespace and \nnewlines",
418			Strings::compress($input)
419			);
420			}
421
422			/**
423			* @test
424			* @covers ::redact
425			* @dataProvider provideRedact
426			*
427			* @param string\|null $value The value to pass to the filter.
428			* @param array\|callable $words The words to pass to the filter.
429			* @param string $replacement The replacement to pass to the filter.
430			* @param string\|null $expected The expected result.
431			*/
432			public function redact($value, $words, string $replacement, $expected)
433			{
434			$actual = Strings::redact($value, $words, $replacement);
435
436			$this->assertSame($expected, $actual);
437			}
438
439			/**
440			* @return array
441			*/
442			public function provideRedact() : array
443			{
444			return [
445			'null value' => [
446			'value' => null,
447			'words' => [],
448			'replacement' => '',
449			'expected' => null,
450			],
451			'empty string' => [
452			'value' => '',
453			'words' => [],
454			'replacement' => '',
455			'expected' => '',
456			],
457			'replace with empty' => [
458			'value' => 'this message contains something that you want removed',
459			'words' => ['something that you want removed'],
460			'replacement' => '',
461			'expected' => 'this message contains ',
462			],
463			'replace with *' => [
464			'value' => 'replace certain words that you might want to remove',
465			'words' => ['might', 'certain'],
466			'replacement' => '*',
467			'expected' => 'replace ***** words that you *** want to remove',
468			],
469			'replace with █' => [
470			'value' => 'redact specific dates and secret locations',
471			'words' => ['secret locations', 'specific dates'],
472			'replacement' => '█',
473			'expected' => 'redact ██████████████ and ████████████████',
474			],
475			'replace with multi-character string uses first character' => [
476			'value' => 'replace some particular words',
477			'words' => ['particular', 'words', 'some'],
478			'replacement' => ' *** ',
479			'expected' => 'replace ',
480			],
481			'no replacements' => [
482			'value' => 'some perfectly normal string',
483			'words' => ['undesired', 'words'],
484			'replacement' => '*',
485			'expected' => 'some perfectly normal string',
486			],
487			'closure provides words' => [
488			'value' => 'doe a deer, a female deer',
489			'words' => function () {
490			return ['doe', 'deer'];
491			},
492			'replacement' => '-',
493			'expected' => '--- a ----, a female ----',
494			],
495			];
496			}
497
498			/**
499			* @test
500			* @covers ::redact
501			* @dataProvider provideRedactFailsOnBadInput
502			*
503			* @param mixed $value The value to pass to the filter.
504			* @param mixed $words The words to pass to the filter.
505			* @param string $replacement The replacement to pass to the filter.
506			* @param string $exception The exception to expect.
507			* @param string $message The exception message to expect.
508			*/
509			public function redactFailsOnBadInput($value, $words, string $replacement, string $exception, string $message)
510			{
511			$this->expectException($exception);
512			$this->expectExceptionMessage($message);
513
514			Strings::redact($value, $words, $replacement);
515			}
516
517			/**
518			* @return array
519			*/
520			public function provideRedactFailsOnBadInput() : array
521			{
522			return [
523			'non-string value' => [
524			'value' => ['bad', 'input'],
525			'words' => [],
526			'replacement' => '',
527			'exception' => FilterException::class,
528			'message' => "Value '" . var_export(['bad', 'input'], true) . "' is not a string",
529			],
530			'invalid words argument' => [
531			'value' => 'some string',
532			'words' => 'this is not valid',
533			'replacement' => '',
534			'exception' => FilterException::class,
535			'message' => 'Words was not an array or a callable that returns an array',
536			],
537			'invalid return from callable words argument' => [
538			'value' => 'some string',
539			'words' => function () {
540			return 'this is also not valid';
541			},
542			'replacement' => '',
543			'exception' => FilterException::class,
544			'message' => 'Words was not an array or a callable that returns an array',
545			],
546			];
547			}
548			}
549

traderinteractive / filter-strings-php

Push — master ( 5b69ef...fa3943 )

php$1 ➔ compressIgnoresNewLinesByDefault() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like