Deactivate auditing for time entries in importer

solidtime-io · Sep 16, 2024 · 5c6b32d · 5c6b32d
1 parent 37400d2
commit 5c6b32d
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 2 deletions.
diff --git a/app/Service/Import/Importers/ClockifyTimeEntriesImporter.php b/app/Service/Import/Importers/ClockifyTimeEntriesImporter.php
@@ -99,6 +99,7 @@ public function importData(string $data, string $timezone): void
                     ]);
                 }
                 $timeEntry = new TimeEntry;
+                $timeEntry->disableAuditing();
                 $timeEntry->user_id = $userId;
                 $timeEntry->member_id = $memberId;
                 $timeEntry->task_id = $taskId;

diff --git a/app/Service/Import/Importers/SolidtimeImporter.php b/app/Service/Import/Importers/SolidtimeImporter.php
@@ -221,6 +221,7 @@ public function importData(string $data, string $timezone): void
                     $taskId = $this->taskImportHelper->getKeyByExternalIdentifier($timeEntryRow['task_id']);
                 }
                 $timeEntry = new TimeEntry;
+                $timeEntry->disableAuditing();
                 $timeEntry->user_id = $userId;
                 $timeEntry->member_id = $memberId;
                 $timeEntry->task_id = $taskId;

diff --git a/app/Service/Import/Importers/TogglTimeEntriesImporter.php b/app/Service/Import/Importers/TogglTimeEntriesImporter.php
@@ -99,6 +99,7 @@ public function importData(string $data, string $timezone): void
                     ]);
                 }
                 $timeEntry = new TimeEntry;
+                $timeEntry->disableAuditing();
                 $timeEntry->user_id = $userId;
                 $timeEntry->member_id = $memberId;
                 $timeEntry->task_id = $taskId;

diff --git a/tests/Unit/Service/Import/Importers/TogglTimeEntriesImporterTest.php b/tests/Unit/Service/Import/Importers/TogglTimeEntriesImporterTest.php
@@ -36,7 +36,7 @@ public function test_import_of_test_file_succeeds(): void
         $queryLog = DB::getQueryLog();
 
         // Assert
-        $this->assertCount(23, $queryLog);
+        $this->assertCount(21, $queryLog);
         $testScenario = $this->checkTestScenarioAfterImportExcludingTimeEntries();
         $this->checkTimeEntries($testScenario);
         $this->assertSame(2, $report->timeEntriesCreated);
@@ -67,7 +67,7 @@ public function test_import_of_test_file_twice_succeeds(): void
         $queryLog = DB::getQueryLog();
 
         // Assert
-        $this->assertCount(15, $queryLog);
+        $this->assertCount(13, $queryLog);
         $testScenario = $this->checkTestScenarioAfterImportExcludingTimeEntries();
         $this->checkTimeEntries($testScenario, true);
         $this->assertSame(2, $report->timeEntriesCreated);