Merge pull request #3 from keboola/webrouse-COM-487-fix-name

Fix missing name + test
keboola · Oct 27, 2020 · f3abf0d · f3abf0d
2 parents 0ad746e + 30c70d3
commit f3abf0d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/src/Keboola/CsvTable/Table.php b/src/Keboola/CsvTable/Table.php
@@ -30,6 +30,7 @@ class Table extends CsvWriter {
 
 	public function __construct(string $name, array $header = [], bool $writeHeader = true, Temp $temp = null, $delimiter = CsvOptions::DEFAULT_DELIMITER, $enclosure = CsvOptions::DEFAULT_ENCLOSURE, $lineBreak = "\n")
     {
+        $this->name = $name;
         $this->temp = $temp ? $temp : new Temp('csv-table');
         $this->header = $header;
         $tmpFile = $this->temp ->createTmpFile($name);

diff --git a/tests/Keboola/CsvTable/TableTest.php b/tests/Keboola/CsvTable/TableTest.php
@@ -8,7 +8,7 @@ class TempTest extends TestCase
 {
 	public function testCreate()
 	{
-		$table = new Table('filename_suffix', ['first_col', 'second_col']);
+		$table = new Table('filename', ['first_col', 'second_col']);
 		$this->assertFileExists($table->getPathName());
 		$this->assertEquals(
 			'"first_col","second_col"
@@ -22,6 +22,7 @@ public function testCreate()
 			),
 			$table->getHeader()
 		);
+        $this->assertEquals('filename', $table->getName());
 	}
 
     public function testDontWriteHeader()