Merge pull request #121 from open-source-labs/revert-119-v13-launch

Revert "V13 launch"
open-source-labs · Feb 8, 2024 · f2678fa · f2678fa
2 parents 277c728 + c8f5375
commit f2678fa
Show file tree

Hide file tree

Showing 7 changed files with 174 additions and 26,113 deletions.
diff --git a/DEV_README.md b/DEV_README.md
@@ -31,8 +31,6 @@ Run npm run dev twice if you do not manually run tsc to compile the files first.
 
 - Migrated ERTabling to tableTabBar component to access the ERD because it lacked a parent compartment for prop drilling, hindering the addition of new features. Going forward, a more maintainable solution should be implemented like Redux or Zustand.
 
-- Migrated ERTabling to tableTabBar component to access the ERD because it lacked a parent compartment for prop drilling, hindering the addition of new features. Going forward, a more maintainable solution should be implemented like Redux or Zustand.
-
 <p><b>2. Isolating Database</b> <br> One of the biggest tasks that we tried but <b>did not finish</b> is isolating the concerns of each database type (DBType). The current application has multiple</p>
 <code>if (database === DBType.postgres) {}<br>
 else if (database === DBType.mysql) {}<br>

diff --git a/__tests__/frontend/lib/unittesting.spec.ts b/__tests__/frontend/lib/unittesting.spec.ts
@@ -46,7 +46,7 @@ describe('runSelectAllQuery', () => {
       { sqlString, selectedDb },
       curDBType,
     );
-
+    console.log(result);
 
     expect(mockConnectionModel.connectToDB).toHaveBeenCalledWith('test', 'pg');
     expect(mockConnectionModel.connectToDB).toHaveBeenCalledTimes(1);

diff --git a/backend/src/ipcHandlers/handlers/queryHandler.ts b/backend/src/ipcHandlers/handlers/queryHandler.ts
@@ -264,6 +264,7 @@ export async function runSelectAllQuery(event, {sqlString, selectedDb}:SelectAll
   try {
     await connectionModel.connectToDB(selectedDb, curDBType);
     const results = await queryModel.query(sqlString, [], curDBType);
+    console.log('good',results.rows)
     return results?.rows
   } catch (error) {
     console.log(error, 'in runSelectAllQuery')

diff --git a/backend/src/utils/dummyData/dummyDataMain.ts b/backend/src/utils/dummyData/dummyDataMain.ts
@@ -26,6 +26,7 @@ export const getRandomInt = (min: number, max: number) => {
 // helper function to generate random data based on a given column's data type
 const generateDataByType = (columnObj: ColumnObj): string | number => {
   let length;
+  console.log(columnObj.data_type);
   // updated the new faker package so updated to follow proper documentation.
   switch (columnObj.data_type) {
     case 'smallint':