Skip to content

Commit

Permalink
Improve comments
Browse files Browse the repository at this point in the history
  • Loading branch information
jeffersoncasimir committed Nov 18, 2024
1 parent f1e3607 commit ce409a2
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions test/unittests/CSVParserTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ protected function tearDown(): void
}

/**
* DataProviders for constructor valid CSV files
* DataProviders with valid CSV files
*
* @return array[]
*/
Expand All @@ -60,7 +60,8 @@ public function validCSVFiles(): array
value1,"value2",value3\n
value4,value5,value6
CSV; // Empty line above was left intentionally
CSV; // The empty row is intentional
// Tests: All flags + Field enclosure character (")
file_put_contents($filePath, $csvData);
$filePaths[] = [$filePath];

Expand All @@ -82,7 +83,6 @@ public function testCSVParser(string $validCSVPath): void
$csvParser = new \LORIS\CSVParser(new SplFileInfo($validCSVPath));
$expectedHeaders = ['header1', 'header2', 'header3'];
$data = $csvParser->parse($expectedHeaders);
// Tests: All flags and Field enclosure character (")
$this->assertEquals(
[
[
Expand All @@ -101,7 +101,7 @@ public function testCSVParser(string $validCSVPath): void
}

/**
* DataProviders for constructor valid CSV files
* DataProviders with invalid CSV files
*
* @return array[]
*/
Expand Down

0 comments on commit ce409a2

Please sign in to comment.