From 12007974bda9df3bbd949aab1aff86209a0b4169 Mon Sep 17 00:00:00 2001
From: Colin Mitchell <muffinista@gmail.com>
Date: Mon, 11 Dec 2017 13:05:49 -0500
Subject: [PATCH] Fix issues with setting folder, creating screensaver

---
 package.json                                  |  3 ++-
 src/lib/savers.js                             |  9 +------
 .../__template/index.html                     |  0
 .../__template/saver.json                     |  0
 src/main/index.js                             | 25 ++++++++++++++++---
 src/renderer/Prefs.vue                        | 17 ++++++++++++-
 src/renderer/components/PrefsForm.vue         | 10 +++++---
 test/lib/savers.js                            | 12 ++++++---
 test/ui/prefs.js                              | 17 +++++++++++++
 test/ui/setup.js                              |  6 +++++
 webpack.main.config.js                        | 11 ++++++--
 yarn.lock                                     |  4 +++
 12 files changed, 91 insertions(+), 23 deletions(-)
 rename src/{system-savers => main}/__template/index.html (100%)
 rename src/{system-savers => main}/__template/saver.json (100%)

diff --git a/package.json b/package.json
index da51b5fe..f05698ce 100644
--- a/package.json
+++ b/package.json
@@ -1,7 +1,7 @@
 {
   "name": "before-dawn",
   "productName": "Before Dawn",
-  "version": "0.9.13",
+  "version": "0.9.14",
   "description": "A desktop screensaver app using web technologies",
   "author": "Colin Mitchell <colin@muffinlabs.com> (http://muffinlabs.com)",
   "license": "MIT",
@@ -111,6 +111,7 @@
     "sass-loader": "^6.0.6",
     "sinon": "^4.0.2",
     "spectron": "^3.7.2",
+    "spectron-fake-dialog": "^0.0.1",
     "style-loader": "^0.18.2",
     "url-loader": "^0.5.9",
     "vue": "^2.5.6",
diff --git a/src/lib/savers.js b/src/lib/savers.js
index c27bead3..706000a2 100644
--- a/src/lib/savers.js
+++ b/src/lib/savers.js
@@ -312,7 +312,6 @@ var systemSource = function() {
 }
 
 
-
 /**
  * set config var k to value v
  */
@@ -566,11 +565,6 @@ var writeSync = function() {
   nconf.save();
 };
 
-var getTemplatePath = function() {
-  var d = systemSource();
-  return path.join(d, "__template");
-};
-
 var getConfig = function(cb) {
   var configPath = path.join(baseDir, config_file);
   fs.readFile(configPath, function(err, data) {
@@ -596,8 +590,7 @@ var getConfigSync = function() {
 /**
  * generate a screensaver template
  */
-var generateScreensaver = function(opts) {
-  var src = getTemplatePath();
+var generateScreensaver = function(src, opts) {
   var destDir = getLocalSource();
 
   var contents = fs.readdirSync(src);
diff --git a/src/system-savers/__template/index.html b/src/main/__template/index.html
similarity index 100%
rename from src/system-savers/__template/index.html
rename to src/main/__template/index.html
diff --git a/src/system-savers/__template/saver.json b/src/main/__template/saver.json
similarity index 100%
rename from src/system-savers/__template/saver.json
rename to src/main/__template/saver.json
diff --git a/src/main/index.js b/src/main/index.js
index a7d7dab8..6cd56151 100644
--- a/src/main/index.js
+++ b/src/main/index.js
@@ -613,6 +613,19 @@ var openPrefsOnFirstLoad = function() {
 };
 
 
+var getSystemDir = function() {
+  if ( process.env.BEFORE_DAWN_SYSTEM_DIR !== undefined ) {
+    return process.env.BEFORE_DAWN_SYSTEM_DIR;
+  }
+  
+  if ( global.IS_DEV ) {
+    return __dirname;
+  }
+
+  return app.getAppPath();
+}
+
+
 /**
  * handle initial startup of app
  */
@@ -644,9 +657,14 @@ var bootApp = function() {
     closeRunningScreensavers();
   });
 
+
+  let saverOpts = {
+    base: global.basePath,
+    systemDir: getSystemDir()
+  };
   
-  log.info("Load config from " + global.basePath);
-  global.savers.init(global.basePath, function() {
+  log.info("Load config with", saverOpts);
+  global.savers.init(saverOpts, function() {
     configLoaded = true;
     updateStateManager();
 
@@ -1215,7 +1233,8 @@ ipcMain.on("set-autostart", (event, value) => {
 // generate screensaver template with specified attributes
 //
 ipcMain.on("generate-screensaver", (event, args) => {
-  var data = global.savers.generateScreensaver(args);
+  var src = path.join(getSystemDir(), "__template");
+  var data = global.savers.generateScreensaver(src, args);
   event.sender.send("generate-screensaver", data);
 
   global.savers.reset();
diff --git a/src/renderer/Prefs.vue b/src/renderer/Prefs.vue
index cbba104c..3691dae3 100644
--- a/src/renderer/Prefs.vue
+++ b/src/renderer/Prefs.vue
@@ -39,7 +39,9 @@
       </b-tab>
       <b-tab title="Preferences">
         <div class="container-fluid">
-          <prefs-form :prefs="prefs"></prefs-form>
+          <prefs-form
+             :prefs="prefs"
+             v-on:localSourceChange="localSourceChange"></prefs-form>
         </div>
         <div class="container-fluid">
           <button class="btn btn-large btn-positive reset-to-defaults"
@@ -287,8 +289,21 @@ export default {
           }
         }
       );
+    },
+    localSourceChange(ls) {
+      console.log("hey", this.prefs);
+      console.log("set local source to", ls);
+      var tmp = {
+        localSource: ls
+      };
+      this.prefs = Object.assign(this.prefs, tmp);
+
+//      this.prefs.localSource = ls;
+
     }
   }
+
+
 };
 
   
diff --git a/src/renderer/components/PrefsForm.vue b/src/renderer/components/PrefsForm.vue
index f2238869..83aec19e 100644
--- a/src/renderer/components/PrefsForm.vue
+++ b/src/renderer/components/PrefsForm.vue
@@ -58,7 +58,7 @@
             instead.
           </small>
         </div>
-
+        <!-- ' -->
         <div class="form-check">
           <label for="auto_start" class="form-check-label">
             <input type="checkbox" class="form-check-input" v-model="prefs.auto_start" />
@@ -121,9 +121,11 @@ export default {
         this.handlePathChoice );
     },
     handlePathChoice(result) {
-      if ( result !== undefined ) {
-        this.prefs.localSource = result[0];
-      }
+      this.$emit("localSourceChange", result[0]);
+
+      // blah this is weird
+      this.prefs.localSource = result[0];
+      document.querySelector("[name=localSource]").value = this.prefs.localSource;
     }
   }
 };
diff --git a/test/lib/savers.js b/test/lib/savers.js
index 1468ce99..8ab87c40 100644
--- a/test/lib/savers.js
+++ b/test/lib/savers.js
@@ -182,12 +182,16 @@ describe('Savers', function() {
   
   describe('generateScreensaver', function() {
     it('works', function(done) {
+
+      // this should be the path to our __template in the main app
+      var src = path.join(__dirname, "..", "..", "src", "main", "__template");
       savers.init(workingDir, function() {
         savers.setLocalSource(saversDir);
-        savers.generateScreensaver({
-          name:"New Screensaver"
-        });
-
+        savers.generateScreensaver(src,
+                                   {
+                                     name:"New Screensaver"
+                                   });
+        
         savers.listAll(function(data) {
           assert.equal(4, data.length);
           done();
diff --git a/test/ui/prefs.js b/test/ui/prefs.js
index 1dcaf746..c2ab6799 100644
--- a/test/ui/prefs.js
+++ b/test/ui/prefs.js
@@ -104,4 +104,21 @@ describe('Prefs', function() {
         }).
         then(() => done());
   });
+
+  it('allows setting path via dialog', function(done) {
+    app.fakeDialog.mock([ { method: 'showOpenDialog', value: ['/not/a/real/path'] } ]);
+    
+    app.client.waitUntilWindowLoaded().click("=Preferences").
+        then(() => app.client.click("button.pick")).
+        then(() => app.client.click("button.save")).
+        then(() => {
+          app.client.getWindowCount().should.eventually.equal(1)
+        }).
+        then(() => {
+          assert.equal("/not/a/real/path", helpers.savedConfig(workingDir).localSource);
+        }).
+        then(() => done());
+  });
+
+
 });
diff --git a/test/ui/setup.js b/test/ui/setup.js
index 2baafe69..3af52248 100644
--- a/test/ui/setup.js
+++ b/test/ui/setup.js
@@ -2,6 +2,8 @@ const tmp = require('tmp');
 const path = require('path');
 const fs = require('fs-extra');
 const Application = require('spectron').Application;
+const fakeDialog = require('spectron-fake-dialog');
+
 const chai = require('chai');
 const chaiAsPromised = require('chai-as-promised');
 
@@ -45,6 +47,10 @@ exports.application = function(workingDir) {
     }
   });
 
+  fakeDialog.apply(a);
+
+  a.fakeDialog = fakeDialog;
+  
   chaiAsPromised.transferPromiseness = a.transferPromiseness;
 
   return a;
diff --git a/webpack.main.config.js b/webpack.main.config.js
index c269fe5b..7e33c70a 100644
--- a/webpack.main.config.js
+++ b/webpack.main.config.js
@@ -50,8 +50,8 @@ let mainConfig = {
     ]
   },
   node: {
-    __dirname: false, //process.env.NODE_ENV !== 'production',
-    __filename: false // process.env.NODE_ENV !== 'production'
+    __dirname: false,
+    __filename: false
   },
   output: {
     filename: '[name].js',
@@ -80,6 +80,13 @@ let mainConfig = {
         ignore: ['.*']
       }
     ]),
+    new CopyWebpackPlugin([
+      {
+        from: path.join(__dirname, 'src', 'main', '__template'),
+        to: path.join(outputDir, '__template'),
+        ignore: ['.*']
+      }
+    ]),
     new webpack.DefinePlugin({
       'process.env.PACKAGE_VERSION': JSON.stringify(packageJSON.version)
     }),
diff --git a/yarn.lock b/yarn.lock
index 3888a5dd..17e9f413 100644
--- a/yarn.lock
+++ b/yarn.lock
@@ -7191,6 +7191,10 @@ spdy@^3.4.1:
     select-hose "^2.0.0"
     spdy-transport "^2.0.18"
 
+spectron-fake-dialog@^0.0.1:
+  version "0.0.1"
+  resolved "https://registry.yarnpkg.com/spectron-fake-dialog/-/spectron-fake-dialog-0.0.1.tgz#ffd5cfadcac64eb957340389f047529c4a68e23a"
+
 spectron@^3.7.2:
   version "3.7.2"
   resolved "https://registry.yarnpkg.com/spectron/-/spectron-3.7.2.tgz#86f41306a9b70ed6ee1500f7f7d3adc389afb446"