DEV - clean console log

ecotaxa · Jan 2, 2025 · 026f05f · 026f05f
1 parent d0ebe0e
commit 026f05f
Show file tree

Hide file tree

Showing 6 changed files with 10 additions and 22 deletions.
diff --git a/src/data/data-sources/sqlite/sqlite-sample-data-source.ts b/src/data/data-sources/sqlite/sqlite-sample-data-source.ts
@@ -188,8 +188,7 @@ export class SQLiteSampleDataSource implements SampleDataSource {
             // Begin transaction
             this.db.run('BEGIN TRANSACTION', (beginErr: Error) => {
                 if (beginErr) {
-                    console.log("Failed to begin transaction:", beginErr);
-                    return reject(beginErr);
+                    return reject("Failed to begin transaction:" + beginErr);
                 }
 
                 const insertPromises = samples.map((sample) => {
@@ -238,8 +237,7 @@ export class SQLiteSampleDataSource implements SampleDataSource {
 
                         this.db.run(sql, params, function (err) {
                             if (err) {
-                                console.log("Failed to insert sample:", err);
-                                rejectInsert(err);
+                                rejectInsert("Failed to insert sample:" + err);
                             } else {
                                 insertedIds.push(this.lastID);
                                 resolveInsert(this.lastID);
@@ -253,8 +251,7 @@ export class SQLiteSampleDataSource implements SampleDataSource {
                         // Commit transaction if all inserts are successful
                         this.db.run('COMMIT', (commitErr: Error) => {
                             if (commitErr) {
-                                console.log("Failed to commit transaction:", commitErr);
-                                return reject(commitErr);
+                                return reject("Failed to commit transaction:" + commitErr);
                             }
                             resolve(insertedIds);
                         });
@@ -263,11 +260,11 @@ export class SQLiteSampleDataSource implements SampleDataSource {
                         // Rollback transaction if any insert fails
                         this.db.run('ROLLBACK', (rollbackErr: Error) => {
                             if (rollbackErr) {
-                                console.log("Failed to rollback transaction:", rollbackErr);
+                                reject("Failed to rollback transaction:" + rollbackErr);
                             } else {
-                                console.log("Transaction rolled back due to error:", error);
+                                reject("Transaction rolled back due to error:" + error);
                             }
-                            reject(error);
+
                         });
                     });
             });

diff --git a/src/domain/repositories/project-repository.ts b/src/domain/repositories/project-repository.ts
@@ -342,11 +342,11 @@ export class ProjectRepositoryImpl implements ProjectRepository {
                 const zipExists = await this.checkFileExists(destSubfolder);
                 if (!zipExists) {
                     // If not, copy and zip the folder
-                    //console.log(`Copying new folder: ${entry.name}`);
+                    // TODO LOG IN TASK LOG FILE `Copying new folder: ${entry.name}`
                     await this.zipFolder(sourceSubfolder, destSubfolder);
                 }
                 // else {
-                //     console.log(`Skipping already existing folder: ${entry.name}`);
+                //     TODO LOG IN TASK LOG FILE `Skipping already existing folder: ${entry.name}`);
                 // }
             }
         }

diff --git a/src/domain/repositories/search-repository.ts b/src/domain/repositories/search-repository.ts
@@ -33,7 +33,6 @@ export class SearchRepositoryImpl implements SearchRepository {
             if (!sort_by || sort_by.slice(-1) !== ")") {
                 // Add an error message to the errors array
                 errors.push(statement);
-                console.log("error" + statement)
                 return null;
             }
 
@@ -44,16 +43,13 @@ export class SearchRepositoryImpl implements SearchRepository {
             if (!clean_sort_by || !order_by) {
                 // Add an error message to the errors array
                 errors.push(statement);
-                console.log("error" + statement)
-
                 return null;
             }
 
             // Check that order_by is either asc or desc
             if (!this.order_by_allow_params.includes(order_by.toLowerCase())) {
                 // Add an error message to the errors array
                 errors.push(order_by);
-                console.log("error" + order_by)
                 return null;
             }
 

diff --git a/src/domain/repositories/task-repository.ts b/src/domain/repositories/task-repository.ts
@@ -72,7 +72,6 @@ export class TaskRepositoryImpl implements TaskRepository {
             await this.logMessage(log_file_path, "Task created for " + task.task_type)
             await this.logMessage(log_file_path, "Task params: " + JSON.stringify(task.task_params))
         } catch (err) {
-            console.log(err);
             throw new Error("Cannot create log file");
         }
         // update task with log file path
@@ -349,7 +348,6 @@ export class TaskRepositoryImpl implements TaskRepository {
             return data;
         } catch (err) {
             // if error, throw an error
-            console.log(err);
             throw new Error("Cannot read log file");
         }
     }

diff --git a/src/domain/repositories/user-repository.ts b/src/domain/repositories/user-repository.ts
@@ -193,7 +193,6 @@ export class UserRepositoryImpl implements UserRepository {
             return false;
         } catch (error) {
             // An error occurred while fetching or comparing, log the error and return false
-            console.log(error);
             return false;
         }
     }
@@ -218,8 +217,6 @@ export class UserRepositoryImpl implements UserRepository {
             return decoded_token
         } catch (error) {
             // An error occurred while fetching or comparing, log the error and return null
-            console.log(error);
-            console.log("Token invalid or expired.");
             return null;
         }
     }

diff --git a/src/presentation/routers/tasks-router.ts b/src/presentation/routers/tasks-router.ts
@@ -86,7 +86,7 @@ export default function TaskRouter(
             const logs = await getLogFileTask.execute((req as CustomRequest).token, taskId);
             res.status(200).send(logs);
         } catch (err) {
-            console.log(err);
+            console.log(err)
             if (err.message === "User cannot be used") res.status(403).send({ errors: [err.message] })
             else if (err.message === "Cannot find task") res.status(404).send({ errors: [err.message] });
             else if (err.message === "User does not have the necessary permissions to access this task.") res.status(403).send({ errors: ["Cannot get task log"] });
@@ -100,7 +100,7 @@ export default function TaskRouter(
             const taskId = parseInt(req.params.task_id);
             await streamZipFileUseCase.execute((req as CustomRequest).token, taskId, res);
         } catch (err) {
-            console.log(err);
+            console.log(err)
             if (err.message === "User cannot be used") res.status(403).send({ errors: [err.message] })
             else if (err.message === "Cannot find task") res.status(404).send({ errors: [err.message] });
             else if (err.message === "Cannot find task file") res.status(404).send({ errors: [err.message] });