more fixes

antibody.go

@@ -25,9 +25,9 @@ func New(home string, lines []string) *Antibody {
 
 // Bundle processes all given lines and returns the shell content to execute
 func (a *Antibody) Bundle() (string, error) {
-	var shs []string
-	var total = len(a.Lines)
 	var count int
+	var total = len(a.Lines)
+	var shs []string
 	done := make(chan bool)
 
 	for _, line := range a.Lines {
@@ -38,6 +38,7 @@ func (a *Antibody) Bundle() (string, error) {
 			done <- true
 		}(line)
 	}
+
 	for {
 		select {
 		case event := <-a.Events:

cmd/antibody/command/list.go

@@ -1,16 +1,25 @@
 package command
 
-import "github.com/urfave/cli"
+import (
+	"fmt"
+
+	"github.com/caarlos0/gohome"
+	"github.com/getantibody/antibody/project"
+	"github.com/urfave/cli"
+)
 
 // List all downloaded bundles
 var List = cli.Command{
 	Name:  "list",
 	Usage: "list all currently downloaded bundles",
 	Action: func(ctx *cli.Context) error {
-		// TODO
-		// for _, b := range bundle.List(antibody.Home()) {
-		// 	fmt.Println(b.Name())
-		// }
+		projects, err := project.List(gohome.Cache("antibody"))
+		if err != nil {
+			return err
+		}
+		for _, b := range projects {
+			fmt.Println(b)
+		}
 		return nil
 	},
 }

cmd/antibody/command/update.go

@@ -1,14 +1,16 @@
 package command
 
-import "github.com/urfave/cli"
+import (
+	"github.com/caarlos0/gohome"
+	"github.com/getantibody/antibody/project"
+	"github.com/urfave/cli"
+)
 
 // Update all previously bundled bundles
 var Update = cli.Command{
 	Name:  "update",
 	Usage: "updates all previously bundled commands",
 	Action: func(ctx *cli.Context) error {
-		// TODO
-		// antibody.New(bundle.List(antibody.Home())).Update()
-		return nil
+		return project.Update(gohome.Cache("antibody"))
 	},
 }

project/project.go

@@ -1,8 +1,30 @@
 package project
 
+import "io/ioutil"
+
 // Project is basically any kind of project (git, local, svn, bzr, nfs...)
 type Project interface {
 	Download() error
 	Update() error
 	Folder() string
 }
+
+// List all projects in the given folder
+func List(home string) ([]string, error) {
+	var result []string
+	entries, err := ioutil.ReadDir(home)
+	if err != nil {
+		return result, err
+	}
+	for _, entry := range entries {
+		if entry.Mode().IsDir() && entry.Name()[0] != '.' {
+			result = append(result, entry.Name())
+		}
+	}
+	return result, nil
+}
+
+// Update all projects in the given folder
+func Update(home string) error {
+	return nil
+}

project/project_test.go

@@ -0,0 +1,39 @@
+package project_test
+
+import (
+	"os"
+	"testing"
+
+	"github.com/getantibody/antibody/project"
+	"github.com/stretchr/testify/assert"
+)
+
+func TestList(t *testing.T) {
+	assert := assert.New(t)
+	home := home()
+	defer os.RemoveAll(home)
+	assert.NoError(project.NewGit(home, "caarlos0/jvm", "gh-pages").Download())
+	list, err := project.List(home)
+	assert.NoError(err)
+	assert.Len(list, 1)
+}
+
+func TestListEmptyFolder(t *testing.T) {
+	assert := assert.New(t)
+	home := home()
+	defer os.RemoveAll(home)
+	list, err := project.List(home)
+	assert.NoError(err)
+	assert.Len(list, 0)
+}
+
+func TestListNonExistentFolder(t *testing.T) {
+	assert := assert.New(t)
+	list, err := project.List("/tmp/asdasdadadwhateverwtff")
+	assert.Error(err)
+	assert.Len(list, 0)
+}
+
+func TestUpdateUpdate(t *testing.T) {
+	assert.Nil(t, project.Update(""))
+}