Merge pull request #76 from RallyApps/stop_saving_config

Stop trying to resave the config file all the time

Author: krmorse

lib/config.coffee

@@ -4,21 +4,14 @@ pathUtils = require('path')
 
 configFileName = "config.json"
 
-_updateConfig = (config)->
-  config.server = config.server || "https://rally1.rallydev.com"
-  config
-
 saveConfig = ({path, config}, callback)->
   configPath = pathUtils.join(path, configFileName)
   fs.writeFile(configPath, JSON.stringify(config, null, '    '), callback)
 
 getConfig = (path, callback) ->
   convertToJson = (error, file)->
     if !error
-      config = JSON.parse(file)
-      _updateConfig(config)
-      saveConfig({config, path})
-      callback(null, config)
+      callback(null, JSON.parse(file))
     else
       callback(error)
 
@@ -39,5 +32,5 @@ getAppSourceRoot = (path, callback) ->
       common = pathUtils.resolve common, '..'
     callback null, common
 
-module.exports = {_updateConfig,getConfig,saveConfig,getAppSourceRoot}
+module.exports = {getConfig,saveConfig,getAppSourceRoot}
 _.defaults module.exports, {configFileName}

package.json

@@ -2,7 +2,7 @@
   "author": "Kyle Morse <morky01@ca.com>",
   "name": "rally-app-builder",
   "description": "A node module that assists in the building of Rally Apps",
-  "version": "1.6.1",
+  "version": "1.6.2",
   "homepage": "https://github.com/rallyapps/rally-app-builder",
   "repository": {
     "type": "git",

test/config.test.coffee

@@ -3,30 +3,6 @@ config = require '../lib/config'
 path = require 'path'
 
 describe 'Config', ()->
-  describe('Updates Config', ()->
-    testConfig = {
-      "name": "CardboardCustomCard",
-      "className": "CustomApp",
-      "sdk": "2.0p5",
-      "javascript": [
-        "CustomCard.js",
-        "App.js",
-        "TestCoffee.coffee"
-      ],
-      "css": [
-        "app.css"
-      ],
-      "parents": [
-        "ferentchak/teamboard",
-        "rallyapps/ninjas"
-      ]
-    }
-
-    updatedConfig = config._updateConfig(testConfig)
-    it "should add a server to the config file if one is not present",
-    ()->
-      assert(updatedConfig.server == "https://rally1.rallydev.com")
-  )
 
   describe '#getAppSourceRoot', () ->