Code styling fixes (#26)

* Fix: all strings to use single quotes

https://github.com/airbnb/javascript\?tab\=readme-ov-file\#strings from Airbnb style guide. We can choose double but we need to be consistent

* Fix: object property direct access

* Fix: formatting spaces and consistent semicolons

Author: Bharath Raja

index.js

@@ -1,9 +1,9 @@
 'use strict';
 
-require("dotenv").config();
-const app = require("./server");
+require('dotenv').config();
+const app = require('./server');
 const port = process.env.BACKEND_PORT || 8080;
 
 app.listen(port, () => {
-    console.log(`Example app listening at http://localhost:${port}`)
-});
+    console.log(`Example app listening at http://localhost:${port}`);
+});

jest.config.js

@@ -21,10 +21,10 @@ module.exports = {
         '!**/test/**',
         '!**/jest.config.js',
         '!**/index.js',
-        "!**/coverage/**",
-        "!**/coverage_output.js/**",
-        "!**/coverage_output.json/**",
-        "!**/start-dev.js"
+        '!**/coverage/**',
+        '!**/coverage_output.js/**',
+        '!**/coverage_output.json/**',
+        '!**/start-dev.js'
     ],
     bail: true
-};
+};

jest.init.js

@@ -1,19 +1,19 @@
 const express = require('express');
-jest.mock("@gofynd/fdk-extension-javascript/express/storage", () => {
+jest.mock('@gofynd/fdk-extension-javascript/express/storage', () => {
     return {
-      SQLiteStorage: jest.fn().mockImplementation(() => ({})),
+        SQLiteStorage: jest.fn().mockImplementation(() => ({})),
     };
-  });
+});
 // Write your own jest init
-jest.mock("@gofynd/fdk-extension-javascript/express", jest.fn(() => {
+jest.mock('@gofynd/fdk-extension-javascript/express', jest.fn(() => {
     return {
         setupFdk: function () {
             return {
                 fdkHandler: (req, res, next) => {
                     next();
                 },
                 platformApiRoutes: express.Router()
-            }
+            };
         }
-    }
-}))
+    };
+}));

server.js

@@ -1,16 +1,15 @@
 const express = require('express');
 const cookieParser = require('cookie-parser');
 const bodyParser = require('body-parser');
-const path = require("path");
+const path = require('path');
 const sqlite3 = require('sqlite3').verbose();
-const serveStatic = require("serve-static");
+const serveStatic = require('serve-static');
 const { readFileSync } = require('fs');
-const { setupFdk } = require("@gofynd/fdk-extension-javascript/express");
-const { SQLiteStorage } = require("@gofynd/fdk-extension-javascript/express/storage");
+const { setupFdk } = require('@gofynd/fdk-extension-javascript/express');
+const { SQLiteStorage } = require('@gofynd/fdk-extension-javascript/express/storage');
 const sqliteInstance = new sqlite3.Database('session_storage.db');
 const productRouter = express.Router();
 
-
 const fdkExtension = setupFdk({
     api_key: process.env.EXTENSION_API_KEY,
     api_secret: process.env.EXTENSION_API_SECRET,
@@ -20,25 +19,25 @@ const fdkExtension = setupFdk({
         auth: async (req) => {
             // Write you code here to return initial launch url after auth process complete
             if (req.query.application_id)
-                return `${req.extension.base_url}/company/${req.query['company_id']}/application/${req.query.application_id}`;
+                return `${req.extension.base_url}/company/${req.query.company_id}/application/${req.query.application_id}`;
             else
-                return `${req.extension.base_url}/company/${req.query['company_id']}`;
+                return `${req.extension.base_url}/company/${req.query.company_id}`;
         },
-        
+
         uninstall: async (req) => {
             // Write your code here to cleanup data related to extension
             // If task is time taking then process it async on other process.
         }
     },
-    storage: new SQLiteStorage(sqliteInstance,"exapmple-fynd-platform-extension"), // add your prefix
-    access_mode: "online",
+    storage: new SQLiteStorage(sqliteInstance, 'example-fynd-platform-extension'), // add your prefix
+    access_mode: 'online',
     webhook_config: {
-        api_path: "/api/webhook-events",
-        notification_email: "useremail@example.com",
+        api_path: '/api/webhook-events',
+        notification_email: 'useremail@example.com',
         event_map: {
-            "company/product/delete": {
-                "handler": (eventName) => {  console.log(eventName)},
-                "version": '1'
+            'company/product/delete': {
+                'handler': (eventName) => { console.log(eventName); },
+                'version': '1'
             }
         }
     },
@@ -47,12 +46,12 @@ const fdkExtension = setupFdk({
 const STATIC_PATH = process.env.NODE_ENV === 'production'
     ? path.join(process.cwd(), 'frontend', 'public', 'dist')
     : path.join(process.cwd(), 'frontend');
-    
+
 const app = express();
 const platformApiRoutes = fdkExtension.platformApiRoutes;
 
 // Middleware to parse cookies with a secret key
-app.use(cookieParser("ext.session"));
+app.use(cookieParser('ext.session'));
 
 // Middleware to parse JSON bodies with a size limit of 2mb
 app.use(bodyParser.json({
@@ -63,26 +62,26 @@ app.use(bodyParser.json({
 app.use(serveStatic(STATIC_PATH, { index: false }));
 
 // FDK extension handler and API routes (extension launch routes)
-app.use("/", fdkExtension.fdkHandler);
+app.use('/', fdkExtension.fdkHandler);
 
 // Route to handle webhook events and process it.
-app.post('/api/webhook-events', async function(req, res) {
+app.post('/api/webhook-events', async function (req, res) {
     try {
-      console.log(`Webhook Event: ${req.body.event} received`)
-      await fdkExtension.webhookRegistry.processWebhook(req);
-      return res.status(200).json({"success": true});
-    } catch(err) {
-      console.log(`Error Processing ${req.body.event} Webhook`);
-      return res.status(500).json({"success": false});
+        console.log(`Webhook Event: ${req.body.event} received`);
+        await fdkExtension.webhookRegistry.processWebhook(req);
+        return res.status(200).json({ 'success': true });
+    } catch (err) {
+        console.log(`Error Processing ${req.body.event} Webhook`);
+        return res.status(500).json({ 'success': false });
     }
-})
+});
 
 productRouter.get('/', async function view(req, res, next) {
     try {
         const {
             platformClient
         } = req;
-        const data = await platformClient.catalog.getProducts()
+        const data = await platformClient.catalog.getProducts();
         return res.json(data);
     } catch (err) {
         next(err);
@@ -96,7 +95,7 @@ productRouter.get('/application/:application_id', async function view(req, res,
             platformClient
         } = req;
         const { application_id } = req.params;
-        const data = await platformClient.application(application_id).catalog.getAppProducts()
+        const data = await platformClient.application(application_id).catalog.getAppProducts();
         return res.json(data);
     } catch (err) {
         next(err);
@@ -106,16 +105,16 @@ productRouter.get('/application/:application_id', async function view(req, res,
 // FDK extension api route which has auth middleware and FDK client instance attached to it.
 platformApiRoutes.use('/products', productRouter);
 
-// If you are adding routes outside of the /api path, 
+// If you are adding routes outside of the /api path,
 // remember to also add a proxy rule for them in /frontend/vite.config.js
 app.use('/api', platformApiRoutes);
 
 // Serve the Vue app for all other routes
 app.get('*', (req, res) => {
     return res
-    .status(200)
-    .set("Content-Type", "text/html")
-    .send(readFileSync(path.join(STATIC_PATH, "index.html")));
+        .status(200)
+        .set('Content-Type', 'text/html')
+        .send(readFileSync(path.join(STATIC_PATH, 'index.html')));
 });
 
 module.exports = app;

test/index.js

@@ -6,7 +6,7 @@ describe('Test server', () => {
         process.env.PORT = mockPort;
         const app = server();
         app.listen = jest.fn();
-        app.listen(mockPort, () => {});
+        app.listen(mockPort, () => { });
         expect(app.listen).toHaveBeenCalledWith(mockPort, expect.any(Function));
     });
-});
+});

test/products.route.spec.js

@@ -20,7 +20,7 @@ describe('Product Routes', () => {
 
   it('GET /*: Fallback route', async () => {
     const res = await request.get('/test');
-    expect(res.headers['content-type']).toContain("text/html");
+    expect(res.headers['content-type']).toContain('text/html');
   });
 
   it('GET /api/products: should handle errors in the product list route', async () => {

test/utils/server.js

@@ -38,5 +38,5 @@ module.exports = () => {
   if (!request) {
     request = supertest(createTestApp());
   }
-  return { request, mockPlatformClient }
-}
+  return { request, mockPlatformClient };
+};