Warn on mix of different data providers

Author: staabm

src/Metadata/Api/DataProvider.php

@@ -52,6 +52,25 @@ public function providedData(string $className, string $methodName): ?array
         }
 
         if ($dataProvider->isNotEmpty()) {
+            if ($testWith->isNotEmpty()) {
+                $method = new ReflectionMethod($className, $methodName);
+
+                Event\Facade::emitter()->testTriggeredPhpunitWarning(
+                    new TestMethod(
+                        $className,
+                        $methodName,
+                        $method->getFileName(),
+                        $method->getStartLine(),
+                        Event\Code\TestDoxBuilder::fromClassNameAndMethodName(
+                            $className,
+                            $methodName,
+                        ),
+                        MetadataCollection::fromArray([]),
+                        Event\TestData\TestDataCollection::fromArray([]),
+                    ),
+                    'Mixing #[DataProvider] and #[TestWith] attributes is not supported, only the data provided by #[DataProvider] will be used',
+                );
+            }
             $data = $this->dataProvidedByMethods($className, $methodName, $dataProvider);
         } else {
             $data = $this->dataProvidedByMetadata($testWith);

tests/end-to-end/_files/TestWithAttributeAndDataProviderTest.php

@@ -0,0 +1,30 @@
+<?php declare(strict_types=1);
+
+/*
+ * This file is part of PHPUnit.
+ *
+ * (c) Sebastian Bergmann <[email protected]>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+namespace PHPUnit\TestFixture;
+
+use PHPUnit\Framework\Attributes\DataProvider;
+use PHPUnit\Framework\Attributes\TestWith;
+use PHPUnit\Framework\TestCase;
+
+final class TestWithAttributeAndDataProviderTest extends TestCase
+{
+    public static function provider(): iterable
+    {
+        yield 'foo' => ['bar', 'baz'];
+    }
+
+    #[TestWith(['a', 'b'], 'foo')]
+    #[DataProvider('provider')]
+    public function testWithDifferentProviderTypes($one, $two): void
+    {
+        $this->assertTrue(true);
+    }
+}

tests/end-to-end/metadata/warning-mix-dataprovider-types.phpt

@@ -0,0 +1,29 @@
+--TEST--
+phpunit ../_files/TestWithAttributeAndDataProviderTest.php
+--FILE--
+<?php declare(strict_types=1);
+$_SERVER['argv'][] = '--do-not-cache-result';
+$_SERVER['argv'][] = '--no-configuration';
+$_SERVER['argv'][] = __DIR__ . '/../_files/TestWithAttributeAndDataProviderTest.php';
+
+require __DIR__ . '/../../bootstrap.php';
+
+(new PHPUnit\TextUI\Application)->run($_SERVER['argv']);
+--EXPECTF--
+PHPUnit %s by Sebastian Bergmann and contributors.
+
+Runtime: %s
+
+W                                                                   1 / 1 (100%)
+
+Time: %s, Memory: %s
+
+1 test triggered 1 PHPUnit warning:
+
+1) PHPUnit\TestFixture\TestWithAttributeAndDataProviderTest::testWithDifferentProviderTypes
+Mixing #[DataProvider] and #[TestWith] attributes is not supported, only the data provided by #[DataProvider] will be used
+
+%sTestWithAttributeAndDataProviderTest.php:%d
+
+OK, but there were issues!
+Tests: 1, Assertions: 1, PHPUnit Warnings: 1.