fix: add and remove pbxGroup

Author: Kristian D. Dimitrov

lib/pbxProject.js

@@ -523,12 +523,16 @@ pbxProject.prototype.addPbxGroup = function (filePathsArray, name, path, sourceT
             isa: 'PBXGroup',
             children: [],
             name: name,
-            path: path,
             sourceTree: sourceTree ? sourceTree : '"<group>"'
-        },//path is mandatory only for the main group
+        },
         fileReferenceSection = this.pbxFileReferenceSection(),
         filePathToReference = {};
 
+    //path is mandatory only for the main group
+    if(!opt.filesRelativeToProject) {
+        pbxGroup.path = path;
+    }
+
     for (var key in fileReferenceSection) {
         // only look for comments
         if (!COMMENT_KEY.test(key)) continue;
@@ -602,7 +606,7 @@ pbxProject.prototype.removePbxGroup = function(groupName, path) {
         return;
     }
 
-    path = path || $path.dirname(this.filepath);
+    path = path || "";
 
     var children = group.children;
 
@@ -620,7 +624,7 @@ pbxProject.prototype.removePbxGroup = function(groupName, path) {
     if(mainGroup) {
         var mainGroupChildren = this.findMainPbxGroup().children, i;
         for(i in mainGroupChildren) {
-            if (mainGroupChildren[i].comment == name) {
+            if (mainGroupChildren[i].comment == groupName) {
                 mainGroupChildren.splice(i, 1);
             }
         }

test/parser/projects/expected/with_omit_empty_values_disabled_expected.pbxproj

@@ -260,8 +260,8 @@
 			children = (
 			);
 			name = CustomGroup;
-			path = undefined;
 			sourceTree = "<group>";
+			path = undefined;
 		};
 /* End PBXGroup section */