Skip to content

Commit 9e4f4dd

Browse files
authored
test(dataprovider): Rename dataprovider functions to not be accidentally tested (#1266)
1 parent 09c5478 commit 9e4f4dd

4 files changed

Lines changed: 8 additions & 8 deletions

File tree

tests/src/Kernel/DataProducer/Images/ImageResourceUrlTest.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ class ImageResourceUrlTest extends GraphQLTestBase {
1414
/**
1515
* @covers \Drupal\graphql\Plugin\GraphQL\DataProducer\Images\ImageResourceUrl::resolve
1616
*
17-
* @dataProvider testImageResourceUrlProvider
17+
* @dataProvider imageResourceUrlProvider
1818
*/
1919
public function testImageResourceUrl(array $input, string $expected): void {
2020
$result = $this->executeDataProducer('image_style_url', [
@@ -27,7 +27,7 @@ public function testImageResourceUrl(array $input, string $expected): void {
2727
/**
2828
* Provider for testImageResourceUrl().
2929
*/
30-
public function testImageResourceUrlProvider(): array {
30+
public function imageResourceUrlProvider(): array {
3131
return [
3232
[
3333
['url' => 'http://localhost/test_image.jpg'],

tests/src/Kernel/DataProducer/SeekTest.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ class SeekTest extends GraphQLTestBase {
1414
/**
1515
* @covers \Drupal\graphql\Plugin\GraphQL\DataProducer\Utility\Seek::resolve
1616
*
17-
* @dataProvider testSeekProvider
17+
* @dataProvider seekProvider
1818
*
1919
* @param array $input
2020
* The test list to seek in.
@@ -37,7 +37,7 @@ public function testSeek(array $input, int $position, $expected): void {
3737
*
3838
* @return array
3939
*/
40-
public function testSeekProvider() {
40+
public function seekProvider(): array {
4141
return [
4242
[
4343
[1, 2, 3],

tests/src/Kernel/DataProducer/StringTest.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ class StringTest extends GraphQLTestBase {
1414
/**
1515
* @covers \Drupal\graphql\Plugin\GraphQL\DataProducer\String\Uppercase::resolve
1616
*
17-
* @dataProvider testUppercaseProvider
17+
* @dataProvider uppercaseProvider
1818
*/
1919
public function testUppercase(string $input, string $expected): void {
2020
$result = $this->executeDataProducer('uppercase', [
@@ -27,7 +27,7 @@ public function testUppercase(string $input, string $expected): void {
2727
/**
2828
* Tests the upper case data producer.
2929
*/
30-
public function testUppercaseProvider(): array {
30+
public function uppercaseProvider(): array {
3131
return [
3232
['test', 'TEST'],
3333
['123 ..!!', '123 ..!!'],

tests/src/Kernel/ResolverBuilderTest.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ public function setUp(): void {
4949
/**
5050
* @covers ::produce
5151
*
52-
* @dataProvider testBuilderProducingProvider
52+
* @dataProvider builderProducingProvider
5353
*
5454
* @param string $input
5555
* @param string $expected
@@ -62,7 +62,7 @@ public function testBuilderProducing($input, $expected): void {
6262
/**
6363
* @return array
6464
*/
65-
public function testBuilderProducingProvider(): array {
65+
public function builderProducingProvider(): array {
6666
return [
6767
['entity_load', ResolverInterface::class],
6868
['entity_id', ResolverInterface::class],

0 commit comments

Comments
 (0)