[pbxProject] enforce quote delimiters to support unsupported chars

enforce quote delimiters on names and paths for PBXFileReference

Author: goya

lib/pbxProject.js

@@ -495,8 +495,10 @@ function pbxFileReferenceObj(file) {
 
     obj.isa = 'PBXFileReference';
     obj.lastKnownFileType = file.lastType;
-    obj.name = file.basename;
-    obj.path = file.path;
+    
+    obj.name = "\"" + file.basename + "\"";
+    obj.path = "\"" + file.path + "\"";
+    
     obj.sourceTree = file.sourceTree;
 
     if (file.fileEncoding)

test/addFramework.js

@@ -52,8 +52,8 @@ exports.addFramework = {
 
         test.equal(fileRefEntry.isa, 'PBXFileReference');
         test.equal(fileRefEntry.lastKnownFileType, '"compiled.mach-o.dylib"');
-        test.equal(fileRefEntry.name, 'libsqlite3.dylib');
-        test.equal(fileRefEntry.path, 'usr/lib/libsqlite3.dylib');
+        test.equal(fileRefEntry.name, '"libsqlite3.dylib"');
+        test.equal(fileRefEntry.path, '"usr/lib/libsqlite3.dylib"');
         test.equal(fileRefEntry.sourceTree, 'SDKROOT');
 
         test.done();

test/addHeaderFile.js

@@ -53,8 +53,8 @@ exports.addHeaderFile = {
         test.equal(fileRefEntry.isa, 'PBXFileReference');
         test.equal(fileRefEntry.fileEncoding, 4);
         test.equal(fileRefEntry.lastKnownFileType, 'sourcecode.c.h');
-        test.equal(fileRefEntry.name, 'file.h');
-        test.equal(fileRefEntry.path, 'file.h');
+        test.equal(fileRefEntry.name, '"file.h"');
+        test.equal(fileRefEntry.path, '"file.h"');
         test.equal(fileRefEntry.sourceTree, '"<group>"');
 
         test.done();

test/addResourceFile.js

@@ -91,8 +91,8 @@ exports.addResourceFile = {
         test.equal(fileRefEntry.isa, 'PBXFileReference');
         test.equal(fileRefEntry.fileEncoding, undefined);
         test.equal(fileRefEntry.lastKnownFileType, '"wrapper.plug-in"');
-        test.equal(fileRefEntry.name, 'assets.bundle');
-        test.equal(fileRefEntry.path, 'Resources/assets.bundle');
+        test.equal(fileRefEntry.name, '"assets.bundle"');
+        test.equal(fileRefEntry.path, '"Resources/assets.bundle"');
         test.equal(fileRefEntry.sourceTree, '"<group>"');
 
         test.done();
@@ -152,8 +152,8 @@ exports.addResourceFile = {
             test.equal(fileRefEntry.isa, 'PBXFileReference');
             test.equal(fileRefEntry.fileEncoding, undefined);
             test.equal(fileRefEntry.lastKnownFileType, '"wrapper.plug-in"');
-            test.equal(fileRefEntry.name, 'assets.bundle');
-            test.equal(fileRefEntry.path, 'Plugins/assets.bundle');
+            test.equal(fileRefEntry.name, '"assets.bundle"');
+            test.equal(fileRefEntry.path, '"Plugins/assets.bundle"');
             test.equal(fileRefEntry.sourceTree, '"<group>"');
             test.done();
         },

test/addSourceFile.js

@@ -89,8 +89,8 @@ exports.addSourceFile = {
         test.equal(fileRefEntry.isa, 'PBXFileReference');
         test.equal(fileRefEntry.fileEncoding, 4);
         test.equal(fileRefEntry.lastKnownFileType, 'sourcecode.c.objc');
-        test.equal(fileRefEntry.name, 'file.m');
-        test.equal(fileRefEntry.path, 'file.m');
+        test.equal(fileRefEntry.name, '"file.m"');
+        test.equal(fileRefEntry.path, '"file.m"');
         test.equal(fileRefEntry.sourceTree, '"<group>"');
 
         test.done();

test/addStaticLibrary.js

@@ -123,8 +123,8 @@ exports.addStaticLibrary = {
 
         test.equal(fileRefEntry.isa, 'PBXFileReference');
         test.equal(fileRefEntry.lastKnownFileType, 'archive.ar');
-        test.equal(fileRefEntry.name, 'libGoogleAnalytics.a');
-        test.equal(fileRefEntry.path, 'libGoogleAnalytics.a');
+        test.equal(fileRefEntry.name, '"libGoogleAnalytics.a"');
+        test.equal(fileRefEntry.path, '"libGoogleAnalytics.a"');
         test.equal(fileRefEntry.sourceTree, '"<group>"');
 
         test.done();

test/removeHeaderFile.js

@@ -77,8 +77,8 @@ exports.removeHeaderFile = {
         test.equal(fileRefEntry.isa, 'PBXFileReference');
         test.equal(fileRefEntry.fileEncoding, 4);
         test.equal(fileRefEntry.lastKnownFileType, 'sourcecode.c.h');
-        test.equal(fileRefEntry.name, 'file.h');
-        test.equal(fileRefEntry.path, 'file.h');
+        test.equal(fileRefEntry.name, '"file.h"');
+        test.equal(fileRefEntry.path, '"file.h"');
         test.equal(fileRefEntry.sourceTree, '"<group>"');
 
         var deletedFile = proj.removeHeaderFile('Plugins/file.h'),

test/removeResourceFile.js

@@ -141,8 +141,8 @@ exports.removeResourceFile = {
         test.equal(fileRefEntry.isa, 'PBXFileReference');
         test.equal(fileRefEntry.fileEncoding, undefined);
         test.equal(fileRefEntry.lastKnownFileType, '"wrapper.plug-in"');
-        test.equal(fileRefEntry.name, 'assets.bundle');
-        test.equal(fileRefEntry.path, 'Resources/assets.bundle');
+        test.equal(fileRefEntry.name, '"assets.bundle"');
+        test.equal(fileRefEntry.path, '"Resources/assets.bundle"');
         test.equal(fileRefEntry.sourceTree, '"<group>"');
 
         var deletedFile = proj.removeResourceFile('Resources/assets.bundle'),