Merge pull request #624 from lightninglabs/safe-lnd-shutdown

Safely shutdown lnd with SIGINT.

Author: tanx

public/electron.js

@@ -195,8 +195,8 @@ app.on('activate', () => {
 });
 
 app.on('quit', () => {
-  lndProcess && lndProcess.kill();
-  btcdProcess && btcdProcess.kill();
+  lndProcess && lndProcess.kill('SIGINT');
+  btcdProcess && btcdProcess.kill('SIGINT');
 });
 
 app.setAsDefaultProtocolClient(PREFIX_NAME);

test/integration/action/action-integration.spec.js

@@ -169,9 +169,9 @@ describe('Action Integration Tests', function() {
 
   after(async () => {
     await Promise.all([grpc1.closeLnd(), grpc2.closeLnd()]);
-    lndProcess1.kill();
-    lndProcess2.kill();
-    btcdProcess.kill();
+    lndProcess1.kill('SIGINT');
+    lndProcess2.kill('SIGINT');
+    btcdProcess.kill('SIGINT');
   });
 
   describe('Generate seed and unlock wallet', () => {
@@ -223,7 +223,7 @@ describe('Action Integration Tests', function() {
     });
 
     it('should fund wallet for node1', async () => {
-      btcdProcess.kill();
+      btcdProcess.kill('SIGINT');
       btcdArgs.miningAddress = store1.walletAddress;
       btcdProcess = await startBtcdProcess(btcdArgs);
       await nap(NAP_TIME);