comment delete service

Author: jenaepen

Main.js

@@ -78,33 +78,33 @@ ipcMain.on('setup', (message) => {
   message.returnValue = setupRequired;
 });
 
-// Loads existing settings JSON and update settings to include new services entered by the user.
-//on ipc 'submit' request --> Ousman
+// Adds a new service to the services array within the user/setting.json file
 ipcMain.on('submit', (message, newService) => {
-  //assigning state to the parsed return of setting
+  // Declares a variable state and initialize it to the returned parsed json object from the user/settings.json file
   const state = JSON.parse(
     fs.readFileSync(path.resolve(__dirname, './user/settings.json'), {
       encoding: 'UTF-8',
     }),
   );
-  //  if statement is used to replace hard coded data. Hard coded data and the michelleWasHere key is needed to avoid a load error caused by Electron querying the database before a user has added or selected a database.
 
-  //*** What is happening here --> Ousman */
+  // Checks if setup is required by checking if the value for the state key 'setupRequired' is true 
   if (state.setupRequired) {
+    // If setup is required, the value for key 'setupRequired' is reassign to false and the value for key 'services' is reassign to an array with newService as its only element
     state.setupRequired = false;
     state.services = [JSON.parse(newService)];
-    fs.writeFileSync(path.resolve(__dirname, './user/settings.json'), JSON.stringify(state));
   } else {
-    state.setupRequired = false;
+    // Else the newService is pushed into the services array
     state.services.push(JSON.parse(newService));
-    fs.writeFileSync(path.resolve(__dirname, './user/settings.json'), JSON.stringify(state));
-  }
+  } 
+
+  // Rewrites user/settings.json to show state
+  fs.writeFileSync(path.resolve(__dirname, './user/settings.json'), JSON.stringify(state));
 });
 
 // Load settings JSON and returns updated state back to the render process.
 //on ipc 'dashboard' request --> Ousman
 ipcMain.on('dashboard', (message) => {
-  //assign state to the parsed return of setting --> Ousman 
+  // Declares a variable state and initialize it to the returned parse json object from the user/settings.json file
   const state = JSON.parse(
     fs.readFileSync(path.resolve(__dirname, './user/settings.json'), {
       encoding: 'UTF-8',
@@ -119,27 +119,33 @@ ipcMain.on('dashboard', (message) => {
   message.returnValue = dashboardList;
 });
 
-//ipc 'deleteService' route
+// Deletes the service at position 'index' from the services array within the user/setting.json file,
+// resets the user/setting.json file to what it was originally if all of the services are deleted,
+// and sends the remaining services back to onDelete function within DeleteService as a response
 ipcMain.on('deleteService', (message, index) => {
-  //assigns state to the returned the object returned from settings.json
-    let state = JSON.parse(
-      //read json from settings.json
-      fs.readFileSync(path.resolve(__dirname, './user/settings.json'), {
-        encoding: 'UTF-8',
-      }),
-    );
-    // updating the services
-    if(state.services.length > 1){
-      state.services.splice(index,1); 
-    }else{
-      state= {"setupRequired":true, "services":["hard","coded","in"] }
-    }
+  // Declares a variable state and initialize it to the returned parse json object from the user/settings.json file
+  let state = JSON.parse(
+    fs.readFileSync(path.resolve(__dirname, './user/settings.json'), {
+      encoding: 'UTF-8',
+    }),
+  );
+
+  // Checks if there is more than one services in the services array
+  if(state.services.length > 1){
+    // If true, removes the service at position 'index'
+    state.services.splice(index,1); 
+  }else{
+     // Else reassign state to what the user/setting.json file was originally before any database was save
+    state= {"setupRequired":true, "services":["hard","coded","in"] }
+  }
 
-    //write json from settings.json
-    fs.writeFileSync(path.resolve(__dirname, './user/settings.json'), JSON.stringify(state), {
-        encoding: 'UTF-8'}) 
-    message.sender.send('deleteResponse', state.services);
-  });
+  // Rewrites user/settings.json to show state
+  fs.writeFileSync(path.resolve(__dirname, './user/settings.json'), JSON.stringify(state), {
+    encoding: 'UTF-8'}) 
+
+  // Send a response back with the updated services
+  message.sender.send('deleteResponse', state.services);
+});
 
 
 // Queries the database for communications information and returns it back to the render process.

__tests__/enzyme.test.js

@@ -0,0 +1,32 @@
+import React from 'react';
+import { configure, shallow } from 'enzyme';
+import Adapter from 'enzyme-adapter-react-16';
+const { JSDOM } = require('jsdom');
+const jsdom = new JSDOM('<!doctype html><html><body></body></html>');
+const { window } = jsdom;
+
+import DeleteService from '../app/components/DeleteService.jsx'
+
+
+configure({ adapter: new Adapter() });
+
+describe('Delete a Service', () => {
+    beforeAll(() => {
+        let wrapper = shallow(<DeleteService/>);
+      });
+
+    xit('returns a div',()=>{
+        expect(wrapper.type()).toEqual('div')
+    })
+    it('Delete a service with more than one services within user/setting.json',()=>{
+        expect(1).toEqual(1)
+    })
+    describe('Delete a service with only one services within user/setting.json', () => {
+        it('Delete the service',()=>{
+            expect(1).toEqual(1)
+        })
+        it('Render the AddService Component',()=>{
+            expect(1).toEqual(1)
+        })
+    })
+})

app/components/DeleteService.jsx

@@ -2,43 +2,47 @@ import React, { useContext } from 'react';
 import DashboardContext from '../context/DashboardContext';
 import SetupContext from '../context/SetupContext';
 
-
+// Declare a constant ipcRenderer by deconstructing window.require('electron') so that the onDelete function can initialize an IPC Communication 
 const { ipcRenderer } = window.require('electron');
 
-
+// Deletes a Service
 const DeleteService = (props) => {
-  console.log('in the DeleteService')
-  
-  // Used to toggle setup required if user wants delete all of the databases and the user needs to be sent to the front page.
+  // Declares a constant setup and initialize it to the SetupContext. 
+  // SetupContext indicates whether or not an initial setup is necessary.
+  // An initial setup is necessary when the user has not saved any database to chronos frontend application.   
   const setup = useContext(SetupContext);
 
-  // List of the databases saved by users to track microservices. 
+  // Declares a variable serviceList and initialize it to the DashboardContext. 
+  // DashboardContext lists the databases saved by users. 
   let serviceList = useContext(DashboardContext);
 
-  // Only happens when a DB button is clicked
+  /* 
+   Sends a deleteService request with an index to an ipcMain.on function within the Main.js 
+   On return, the function reassigns the serviceList variable to the updated services provided within the deleteResponse. If the serviceList equals ["hard","coded","in"], then the user has no database saved to the chronos frontend application. If this is true, the function reassigns setup.setupRequired to true by invoking setup.toggleSetup with the argument 'false'. Then the function reloads the application to show the changes made to the services.
+  */
   const onDelete= (index) => {
-    // IPC communication used to delete the button pressed.
     ipcRenderer.send('deleteService', index);
     ipcRenderer.on('deleteResponse', (event, services) => {
         serviceList = services;
         if(serviceList === ["hard","coded","in"]){
-          setup.setupRequired =setup.toggleSetup(false);
+          setup.setupRequired = setup.toggleSetup(false);
         }
         document.location.reload();
-      
     }); 
   }
 
+  // Declares a constant databaseButtons and initialize it an empty array
   const databaseButtons =[];
+
+  /* Iterates over the serviceList to create a button for each service. Each button is pushed into the databaseButtons array as the button is created. Each button has an onclick function that invokes the window confirm function with a warning message (ex:'Are you sure you want to delete this service?') and stores the result of invoking confirm into a constant moveForward. If the moveForward is true, then onDelete function is invoked with the index of where the service is stored within the serviceList*/
   for(let i = 0; i<serviceList.length; i++){
     databaseButtons.push(<button className="microserviceBtn deleteMicroservice" key ={"delete"+i} onClick={()=>{
-      const moveForward = confirm(`Are you sure you want to delete ${serviceList[i]}? \n If "YES" press the "OK" button, \n else press the "Cancel" button`);
-      // if yes run the function 
+      const moveForward = confirm(`Are you sure you want to delete ${serviceList[i]}? \n If "YES" press the "OK" button, else press the "Cancel" button`);
       if (moveForward){onDelete(i)}
 
     }}>{serviceList[i]}</button>)
   }
-
+  // returns the title of the page with all of the services that can be deleted as buttons 
   return (
     <div className="mainContainer">
         <h1 className='overviewTitle'>Press on the Database You Want to Delete</h1>
@@ -47,4 +51,5 @@ const DeleteService = (props) => {
   );
 };
 
+// export the DeleteService function so that it can imported anywhere
 export default DeleteService;