You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cordova.apache.org by er...@apache.org on 2020/03/15 05:46:26 UTC
[cordova-common] branch master updated: fix(ConfigFile): correctly
resolve *-Info.plist file path (#124)
This is an automated email from the ASF dual-hosted git repository.
erisu pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/cordova-common.git
The following commit(s) were added to refs/heads/master by this push:
new 3a0afd3 fix(ConfigFile): correctly resolve *-Info.plist file path (#124)
3a0afd3 is described below
commit 3a0afd3902fff344d479de1022633253ab8bbce1
Author: Arman <ma...@arman.io>
AuthorDate: Sun Mar 15 08:46:15 2020 +0300
fix(ConfigFile): correctly resolve *-Info.plist file path (#124)
* fix(ConfigParser): correctly resolve *-Info.plist file path
* test(ConfigFile): test Info.plist file resolve
Co-authored-by: Raphael von der GrĂ¼n <ra...@gmail.com>
---
spec/ConfigChanges/ConfigFile.spec.js | 15 +++++++++++++++
src/ConfigChanges/ConfigFile.js | 3 ++-
2 files changed, 17 insertions(+), 1 deletion(-)
diff --git a/spec/ConfigChanges/ConfigFile.spec.js b/spec/ConfigChanges/ConfigFile.spec.js
index ad35c19..e47f7e0 100644
--- a/spec/ConfigChanges/ConfigFile.spec.js
+++ b/spec/ConfigChanges/ConfigFile.spec.js
@@ -101,6 +101,21 @@ describe('ConfigFile tests', function () {
var configPath = path.join('project_dir', 'app', 'src', 'main', file, 'xml');
expect(ConfigFile.resolveConfigFilePath('project_dir', 'android', file)).toBe(configPath);
});
+
+ it('should return *-Info.plist file', function () {
+ const projName = 'XXX';
+ const expectedPlistPath = `${projName}-Info.plist`;
+
+ ConfigFile.__set__('getIOSProjectname', () => projName);
+ spyOn(require('glob'), 'sync').and.returnValue([
+ `AAA/${projName}-Info.plist`,
+ `Pods/Target Support Files/Pods-${projName}/Info.plist`,
+ `Pods/Target Support Files/Pods-${projName}/Pods-${projName}-Info.plist`,
+ expectedPlistPath
+ ]);
+
+ expect(ConfigFile.resolveConfigFilePath('', 'ios', '*-Info.plist')).toBe(expectedPlistPath);
+ });
});
});
});
diff --git a/src/ConfigChanges/ConfigFile.js b/src/ConfigChanges/ConfigFile.js
index 6b476d6..4e83119 100644
--- a/src/ConfigChanges/ConfigFile.js
+++ b/src/ConfigChanges/ConfigFile.js
@@ -175,7 +175,8 @@ function resolveConfigFilePath (project_dir, platform, file) {
// [CB-5989] multiple Info.plist files may exist. default to $PROJECT_NAME-Info.plist
if (matches.length > 1 && file.includes('-Info.plist')) {
const plistName = `${getIOSProjectname(project_dir)}-Info.plist`;
- filepath = matches.find(m => m.includes(plistName)) || filepath;
+ const plistPath = path.join(project_dir, plistName);
+ if (matches.includes(plistPath)) return plistPath;
}
return filepath;
}
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@cordova.apache.org
For additional commands, e-mail: commits-help@cordova.apache.org