diff -u b/core/modules/migrate/tests/src/Unit/process/CopyFileTest.php b/core/modules/migrate/tests/src/Unit/process/CopyFileTest.php
--- b/core/modules/migrate/tests/src/Unit/process/CopyFileTest.php
+++ b/core/modules/migrate/tests/src/Unit/process/CopyFileTest.php
@@ -37,64 +37,71 @@
 
   /**
    * Test successful imports/copies.
-   * @dataProvider fileDataCopyProvider
    */
-  public function testSuccessfulCopies($data) {
-    list($source_path, $destination_path) = $data;
-    $this->doImport($source_path, $destination_path);
-    $message = sprintf('File %s exists', $destination_path);
-    $this->assertTrue(is_file($destination_path), $message);
-    // Make sure we didn't accidentally do a move.
-    $this->assertTrue(is_file($source_path), $message);
-  }
-
-  /**
-   * The data provider for testing the file destination.
-   *
-   * @return array
-   *   An array of file permutations to test.
-   */
-  public function fileDataCopyProvider() {
+  public function testSuccessfulCopies() {
     $file = $this->createUri(NULL, NULL, 'temporary');
     $file_absolute = $this->fileSystem->realpath($file);
-    return [
-      'local to local copy' => [$this->root . '/core/modules/simpletest/files/image-test.jpg', 'public://file1.jpg'],
-      'Temporary file using an absolute path' => [$file_absolute, 'temporary://test.jpg'],
-      'Temporary file using a relative path' => [$file_absolute, 'temporary://core/modules/simpletest/files/test.jpg'],
+    $data_sets = [
+      // Test a local to local copy.
+      [
+        $this->root . '/core/modules/simpletest/files/image-test.jpg',
+        'public://file1.jpg'
+      ],
+      // Test a temporary file using an absolute path.
+      [
+        $file_absolute,
+        'temporary://test.jpg'
+      ],
+      // Test a temporary file using a relative path.
+      [
+        $file_absolute,
+        'temporary://core/modules/simpletest/files/test.jpg'
+      ],
     ];
+    foreach ($data_sets as $data) {
+      list($source_path, $destination_path) = $data;
+      $this->doImport($source_path, $destination_path);
+      $message = sprintf('File %s exists', $destination_path);
+      $this->assertTrue(is_file($destination_path), $message);
+      // Make sure we didn't accidentally do a move.
+      $this->assertTrue(is_file($source_path), $message);
+    }
   }
 
   /**
    * Test successful moves.
-   * @dataProvider localFileDataMoveProvider
    */
-  public function testSuccessfulMoves($data) {
-    list($source_path, $destination_path) = $data;
-    $this->doImport($source_path, $destination_path, ['move' => TRUE]);
-    $message = sprintf('File %s exists', $destination_path);
-    $this->assertTrue(is_file($destination_path), $message);
-    $message = sprintf('File %s does not exist', $source_path);
-    $this->assertTrue(!is_file($source_path), $message);
-  }
-
-  /**
-   * The data provider for testing the file destination.
-   *
-   * @return array
-   *   An array of file permutations to test.
-   */
-  public function localFileDataMoveProvider() {
+  public function testSuccessfulMoves() {
     $file_1 = $this->createUri(NULL, NULL, 'temporary');
     $file_1_absolute = $this->fileSystem->realpath($file_1);
     $file_2 = $this->createUri(NULL, NULL, 'temporary');
     $file_2_absolute = $this->fileSystem->realpath($file_2);
     $local_file = $this->createUri(NULL, NULL, 'public');
-
-    return [
-      'Local to local copy' => [$local_file, 'public://file1.jpg'],
-      'Temporary file using an absolute path.' => [$file_1_absolute, 'temporary://test.jpg'],
-      'Temporary file using a relative path' => [$file_2_absolute, 'temporary://core/modules/simpletest/files/test.jpg'],
+    $data_sets = [
+      // Test a local to local copy.
+      [
+        $local_file,
+        'public://file1.jpg'
+      ],
+      // Test a temporary file using an absolute path.
+      [
+        $file_1_absolute,
+        'temporary://test.jpg'
+      ],
+      // Test a temporary file using a relative path.
+      [
+        $file_2_absolute,
+        'temporary://core/modules/simpletest/files/test.jpg'
+      ],
     ];
+    foreach ($data_sets as $data) {
+      list($source_path, $destination_path) = $data;
+      $this->doImport($source_path, $destination_path, ['move' => TRUE]);
+      $message = sprintf('File %s exists', $destination_path);
+      $this->assertTrue(is_file($destination_path), $message);
+      $message = sprintf('File %s does not exist', $source_path);
+      $this->assertTrue(!is_file($source_path), $message);
+    }
   }
 
   /**
@@ -133,11 +140,11 @@
    * @throws \Drupal\migrate\MigrateException
    */
   protected function doImport($source_path, $destination_path, $configuration = []) {
-    $this->plugin = FileCopy::create($this->container, $configuration, 'file_copy', []);
+    $plugin = FileCopy::create($this->container, $configuration, 'file_copy', []);
     $executable = $this->prophesize(MigrateExecutableInterface::class)->reveal();
     $row = new Row([], []);
 
-    $result = $this->plugin->transform([$source_path, $destination_path], $executable, $row, 'foobaz');
+    $result = $plugin->transform([$source_path, $destination_path], $executable, $row, 'foobaz');
 
     // The plugin should either throw an exception or return the destination
     // path.