Merge pull request #1181 from rcjsuen/branch-tests

johnhaley81 · web-flow · commit 0cff089f3645 · 2017-01-28T09:32:06.000-07:00
Added tests for Branch
diff --git a/generate/input/descriptor.json b/generate/input/descriptor.json
@@ -183,6 +183,7 @@
           }
         },
         "git_branch_create_from_annotated": {
+          "isAsync": true,
           "args": {
             "ref_out": {
               "isReturn": true
@@ -193,6 +194,9 @@
             "force": {
               "isOptional": true
             }
+          },
+          "return": {
+            "isErrorCode": true
           }
         },
         "git_branch_next": {
diff --git a/test/tests/branch.js b/test/tests/branch.js
@@ -6,9 +6,14 @@ describe("Branch", function() {
   var NodeGit = require("../../");
   var Repository = NodeGit.Repository;
   var Branch = NodeGit.Branch;
+  var AnnotatedCommit = NodeGit.AnnotatedCommit;
   var branchName = "test-branch";
+  var branchName2 = "test-branch2";
   var fullBranchName = "refs/heads/" + branchName;
+  var fullBranchName2 = "refs/heads/" + branchName2;
   var upstreamName = "origin/master";
+  var fullUpstreamName = "refs/remotes/origin/master";
+  var nonHeadCommit = "c82fb078a192ea221c9f1093c64321c60d64aa0d";
 
   var reposPath = local("../repos/workdir");
 
@@ -27,6 +32,8 @@ describe("Branch", function() {
       })
       .then(function(branch) {
         test.branch = branch;
+        return test.repository.createBranch(
+          branchName2, test.masterCommit, true);
       });
   });
 
@@ -77,4 +84,63 @@ describe("Branch", function() {
         assert.equal(branchNameToTest, branchName);
       });
   });
+
+  it("can rename a branch", function() {
+    var branch = this.branch;
+
+    // don't force the move
+    return Branch.move(branch, branchName2, 0)
+      .then(function(branch) {
+        return Promise.reject(new Error(
+          "should not be able to rename the branch"));
+      }, function(error) {
+        return Promise.resolve()
+          .then(function() {
+            // force the move
+            return Branch.move(branch, branchName2, 1);
+          })
+          .then(function(branch) {
+            assert.equal(branch.name(), fullBranchName2);
+          });
+      });
+  });
+
+  it("can lookup a branch", function() {
+    var repo = this.repository;
+
+    return Branch.lookup(repo, branchName, Branch.BRANCH.LOCAL)
+      .then(function(branch) {
+        assert.equal(branch.name(), fullBranchName);
+        return Branch.lookup(repo, upstreamName, Branch.BRANCH.REMOTE);
+      })
+      .then(function(branch) {
+        assert.equal(branch.name(), fullUpstreamName);
+      });
+  });
+
+  it("can create branch from annotated commit", function() {
+    var repo = this.repository;
+    var annotatedCommit = null;
+
+    return AnnotatedCommit.fromRevspec(repo, nonHeadCommit)
+      .then(function(theAnnotatedCommit) {
+        annotatedCommit = theAnnotatedCommit;
+        return Branch.createFromAnnotated(
+          repo, branchName, annotatedCommit, 0);
+      })
+      .then(function(branch) {
+        return Promise.reject(new Error(
+          "should not be able to create the branch"));
+      }, function(error) {
+        return Promise.resolve()
+          .then(function() {
+            // force the branch creation
+            return Branch.createFromAnnotated(
+              repo, branchName, annotatedCommit, 1);
+          })
+          .then(function(branch) {
+            assert.equal(branch.name(), fullBranchName);
+          });
+      });
+  });
 });

Original file line number	Diff line number	Diff line change
`@@ -183,6 +183,7 @@`
`183`	`183`	`}`
`184`	`184`	`},`
`185`	`185`	`"git_branch_create_from_annotated": {`
	`186`	`+ "isAsync": true,`
`186`	`187`	`"args": {`
`187`	`188`	`"ref_out": {`
`188`	`189`	`"isReturn": true`
`@@ -193,6 +194,9 @@`
`193`	`194`	`"force": {`
`194`	`195`	`"isOptional": true`
`195`	`196`	`}`
	`197`	`+ },`
	`198`	`+ "return": {`
	`199`	`+ "isErrorCode": true`
`196`	`200`	`}`
`197`	`201`	`},`
`198`	`202`	`"git_branch_next": {`