CB-14099 osx: Fixed Resolve Config Path for OSX
authorエリス <ellis.bryan@gmail.com>
Tue, 26 Jun 2018 09:57:41 +0000 (18:57 +0900)
committerTobias Bocanegra <tripod@adobe.com>
Wed, 27 Jun 2018 04:50:49 +0000 (13:50 +0900)
 This closes #32

spec/ConfigChanges/ConfigFile.spec.js
src/ConfigChanges/ConfigFile.js

index 98b1d98..c693420 100644 (file)
@@ -73,6 +73,12 @@ describe('ConfigFile tests', function () {
     });
 
     it('resolveConfigFilePath should return file path', function () {
+        spyOn(configFile, 'getIOSProjectname').and.returnValue('osxpath');
+        var configPath = path.join('project_dir', 'osxpath', 'config.xml');
+        expect(configFile.resolveConfigFilePath('project_dir', 'osx', 'config.xml')).toBe(configPath);
+    });
+
+    it('resolveConfigFilePath should return file path', function () {
         var configPath = path.join('project_dir', 'config.xml');
         expect(configFile.resolveConfigFilePath('project_dir', 'ubuntu', 'config.xml')).toBe(configPath);
     });
index 0b2a23b..a42c06b 100644 (file)
@@ -211,9 +211,12 @@ function resolveConfigFilePath (project_dir, platform, file) {
     if (file === 'config.xml') {
         if (platform === 'ubuntu') {
             filepath = path.join(project_dir, 'config.xml');
-        } else if (platform === 'ios') {
-            var iospath = module.exports.getIOSProjectname(project_dir);
-            filepath = path.join(project_dir, iospath, 'config.xml');
+        } else if (platform === 'ios' || platform === 'osx') {
+            filepath = path.join(
+                project_dir,
+                module.exports.getIOSProjectname(project_dir),
+                'config.xml'
+            );
         } else {
             matches = modules.glob.sync(path.join(project_dir, '**', 'config.xml'));
             if (matches.length) filepath = matches[0];
@@ -225,7 +228,7 @@ function resolveConfigFilePath (project_dir, platform, file) {
     return filepath;
 }
 
-// Find out the real name of an iOS project
+// Find out the real name of an iOS or OSX project
 // TODO: glob is slow, need a better way or caching, or avoid using more than once.
 function getIOSProjectname (project_dir) {
     var matches = modules.glob.sync(path.join(project_dir, '*.xcodeproj'));