Skip to content

Commit ba40e28

Browse files
Tests: Move wp_parse_list() tests to their own file.
This aims to make the tests more discoverable and easier to expand. Follow-up to [44546]. See #59647. git-svn-id: https://develop.svn.wordpress.org/trunk@57284 602fd350-edb4-49c9-b593-d223f7449a82
1 parent 3d19b28 commit ba40e28

2 files changed

Lines changed: 35 additions & 24 deletions

File tree

tests/phpunit/tests/functions.php

Lines changed: 0 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -726,30 +726,6 @@ public function test_canonical_charset() {
726726
update_option( 'blog_charset', $orig_blog_charset );
727727
}
728728

729-
/**
730-
* @ticket 43977
731-
* @dataProvider data_wp_parse_list
732-
*/
733-
public function test_wp_parse_list( $expected, $actual ) {
734-
$this->assertSame( $expected, array_values( wp_parse_list( $actual ) ) );
735-
}
736-
737-
public function data_wp_parse_list() {
738-
return array(
739-
array( array( '1', '2', '3', '4' ), '1,2,3,4' ),
740-
array( array( 'apple', 'banana', 'carrot', 'dog' ), 'apple,banana,carrot,dog' ),
741-
array( array( '1', '2', 'apple', 'banana' ), '1,2,apple,banana' ),
742-
array( array( '1', '2', 'apple', 'banana' ), '1, 2,apple,banana' ),
743-
array( array( '1', '2', 'apple', 'banana' ), '1,2,apple,,banana' ),
744-
array( array( '1', '2', 'apple', 'banana' ), ',1,2,apple,banana' ),
745-
array( array( '1', '2', 'apple', 'banana' ), '1,2,apple,banana,' ),
746-
array( array( '1', '2', 'apple', 'banana' ), '1,2 ,apple,banana' ),
747-
array( array(), '' ),
748-
array( array(), ',' ),
749-
array( array(), ',,' ),
750-
);
751-
}
752-
753729
/**
754730
* @dataProvider data_wp_parse_id_list
755731
*/
Lines changed: 35 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,35 @@
1+
<?php
2+
3+
/**
4+
* Tests for the wp_parse_list() function.
5+
*
6+
* @group functions
7+
*
8+
* @covers ::wp_parse_list
9+
*/
10+
class Tests_Functions_wpParseList extends WP_UnitTestCase {
11+
12+
/**
13+
* @ticket 43977
14+
* @dataProvider data_wp_parse_list
15+
*/
16+
public function test_wp_parse_list( $expected, $actual ) {
17+
$this->assertSame( $expected, array_values( wp_parse_list( $actual ) ) );
18+
}
19+
20+
public function data_wp_parse_list() {
21+
return array(
22+
array( array( '1', '2', '3', '4' ), '1,2,3,4' ),
23+
array( array( 'apple', 'banana', 'carrot', 'dog' ), 'apple,banana,carrot,dog' ),
24+
array( array( '1', '2', 'apple', 'banana' ), '1,2,apple,banana' ),
25+
array( array( '1', '2', 'apple', 'banana' ), '1, 2,apple,banana' ),
26+
array( array( '1', '2', 'apple', 'banana' ), '1,2,apple,,banana' ),
27+
array( array( '1', '2', 'apple', 'banana' ), ',1,2,apple,banana' ),
28+
array( array( '1', '2', 'apple', 'banana' ), '1,2,apple,banana,' ),
29+
array( array( '1', '2', 'apple', 'banana' ), '1,2 ,apple,banana' ),
30+
array( array(), '' ),
31+
array( array(), ',' ),
32+
array( array(), ',,' ),
33+
);
34+
}
35+
}

0 commit comments

Comments
 (0)