Merge branch 'Mitko-Kerezov-master'

Author: Anis Kadri

lib/pbxProject.js

@@ -398,6 +398,37 @@ pbxProject.prototype.removeFromPbxFrameworksBuildPhase = function (file) {
     }
 }
 
+pbxProject.prototype.addBuildPhase = function (filePathsArray, isa, comment) {
+    var section = this.hash.project.objects[isa],
+        buildPhaseUuid = this.generateUuid(),
+        commentKey = f("%s_comment", buildPhaseUuid),
+        buildPhase = {
+            isa: isa,
+            buildActionMask: 2147483647,
+            files: [],
+            runOnlyForDeploymentPostprocessing: 0
+        };
+
+    for (var index = 0; index < filePathsArray.length; index++) {
+        var filePath = filePathsArray[index],
+            file = new pbxFile(filePath);
+
+        file.uuid = this.generateUuid();
+        file.fileRef = this.generateUuid();
+        this.addToPbxFileReferenceSection(file);    // PBXFileReference
+        this.addToPbxBuildFileSection(file);        // PBXBuildFile
+        buildPhase.files.push(pbxBuildPhaseObj(file));
+    }
+    
+    if (section) {
+        section[buildPhaseUuid] = buildPhase;
+        section[commentKey] = comment;
+    }
+    
+    buildPhase.uuid = buildPhaseUuid;
+    return buildPhase;
+}
+
 // helper access functions
 pbxProject.prototype.pbxBuildFileSection = function () {
     return this.hash.project.objects['PBXBuildFile'];
@@ -471,18 +502,16 @@ pbxProject.prototype.buildPhase = function (group,target) {
 pbxProject.prototype.buildPhaseObject = function (name, group,target) {
     var section = this.hash.project.objects[name],
         obj, sectionKey, key;
-
     var buildPhase = this.buildPhase(group,target);
 
     for (key in section) {
-       
+
         // only look for comments
         if (!COMMENT_KEY.test(key)) continue;
 
         // select the proper buildPhase
         if (buildPhase && buildPhase!=key)
             continue;
-        
         if (section[key] == group) {
             sectionKey = key.split(COMMENT_KEY)[0];  
             return section[sectionKey];

test/addBuildPhase.js

@@ -0,0 +1,76 @@
+var fullProject = require('./fixtures/full-project')
+    fullProjectStr = JSON.stringify(fullProject),
+    pbx = require('../lib/pbxProject'),
+    proj = new pbx('.');
+
+function cleanHash() {
+    return JSON.parse(fullProjectStr);
+}
+
+exports.setUp = function (callback) {
+    proj.hash = cleanHash();
+    callback();
+}
+
+exports.addBuildPhase = {
+    'should return a pbxBuildPhase': function (test) {
+        var buildPhase = proj.addBuildPhase(['file.m'], 'PBXSourcesBuildPhase', 'My build phase');
+        
+        test.ok(typeof buildPhase === 'object');
+        test.done()
+    },
+    'should set a uuid on the pbxBuildPhase': function (test) {
+        var buildPhase = proj.addBuildPhase(['file.m'], 'PBXSourcesBuildPhase', 'My build phase');
+
+        test.ok(buildPhase.uuid);
+        test.done()
+    },
+    'should add all files to build phase': function (test) {
+        var buildPhase = proj.addBuildPhase(['file.m', 'assets.bundle'], 'PBXResourcesBuildPhase', 'My build phase');
+        for (var index = 0; index < buildPhase.files.length; index++) {
+            var file = buildPhase.files[index];
+            test.ok(file.value);
+        }
+        
+        test.done()
+    },
+    'should add the PBXBuildPhase object correctly': function (test) {
+        var buildPhase = proj.addBuildPhase(['file.m', 'assets.bundle'], 'PBXResourcesBuildPhase', 'My build phase'),
+            buildPhaseInPbx = proj.buildPhaseObject('PBXResourcesBuildPhase', 'My build phase');
+
+        test.equal(buildPhaseInPbx, buildPhase);
+        test.equal(buildPhaseInPbx.isa, 'PBXResourcesBuildPhase');
+        test.equal(buildPhaseInPbx.buildActionMask, 2147483647);
+        test.equal(buildPhaseInPbx.runOnlyForDeploymentPostprocessing, 0);
+        test.done();
+    },
+    'should add each of the files to PBXBuildFile section': function (test) {
+        var buildPhase = proj.addBuildPhase(['file.m', 'assets.bundle'], 'PBXResourcesBuildPhase', 'My build phase'),
+            buildFileSection = proj.pbxBuildFileSection();
+        
+        for (var index = 0; index < buildPhase.files.length; index++) {
+            var file = buildPhase.files[index];
+            test.ok(buildFileSection[file.value]);
+        }  
+
+        test.done();
+    },
+    'should add each of the files to PBXFileReference section': function (test) {
+        var buildPhase = proj.addBuildPhase(['file.m', 'assets.bundle'], 'PBXResourcesBuildPhase', 'My build phase'),
+            fileRefSection = proj.pbxFileReferenceSection(),
+            buildFileSection = proj.pbxBuildFileSection(),
+            fileRefs = [];
+        
+        for (var index = 0; index < buildPhase.files.length; index++) {
+            var file = buildPhase.files[index];
+            fileRefs.push(buildFileSection[file.value].fileRef);
+        }
+        
+        for (var index = 0; index < fileRefs.length; index++) {
+            var fileRef = fileRefs[index];
+            test.ok(fileRefSection[fileRef]);            
+        }
+
+        test.done();
+    }
+}