fix: replace exec with execsync for immediate process

getulysse · Apr 11, 2024 · d083813 · d083813
1 parent ba0d820
commit d083813
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/__tests__/daemon.spec.js b/__tests__/daemon.spec.js
@@ -24,7 +24,7 @@ jest.mock('socket.io-client', () => ({
 
 beforeEach(() => {
     jest.spyOn(console, 'log').mockImplementation(() => {});
-    jest.spyOn(childProcess, 'exec').mockImplementation(() => {});
+    jest.spyOn(childProcess, 'execSync').mockImplementation(() => {});
     jest.spyOn(Utils, 'isSudo').mockReturnValue(true);
     jest.spyOn(Utils, 'getRunningBlockedApps').mockReturnValue([{ name: 'chromium', pid: 123 }]);
     jest.spyOn(Utils, 'updateResolvConf').mockImplementation(() => {

diff --git a/src/daemon.js b/src/daemon.js
@@ -1,4 +1,4 @@
-import { exec } from 'child_process';
+import { execSync } from 'child_process';
 import {
     config,
     isSudo,
@@ -17,7 +17,7 @@ const handleAppBlocking = () => {
     const blockedApps = getRunningBlockedApps();
 
     for (const app of blockedApps) {
-        exec(`kill -9 ${app.pid}`);
+        execSync(`kill -9 ${app.pid}`);
         console.log(`Blocking ${app.name}`);
         sendNotification('Ulysse', `Blocking ${app.name}`);
     }