Merge pull request #986 from oliveiradan/bolt
Unit Tests for: repo_test.go, task_test.go
This commit is contained in:
Brad Rydzewski
commit
f0ec8c33b6
4 changed files with 219 additions and 19 deletions
|
|
@ -1,24 +1,130 @@
|
|||
package bolt
|
||||
|
||||
import (
|
||||
"testing"
|
||||
|
||||
"github.com/drone/drone/common"
|
||||
. "github.com/franela/goblin"
|
||||
"os"
|
||||
"testing"
|
||||
)
|
||||
|
||||
func TestRepo(t *testing.T) {
|
||||
g := Goblin(t)
|
||||
g.Describe("Repos", func() {
|
||||
g.Describe("Repo", func() {
|
||||
testUser := "octocat"
|
||||
testRepo := "github.com/octopod/hq"
|
||||
testRepo2 := "github.com/octopod/avengers"
|
||||
var db *DB // Temp database
|
||||
|
||||
// create a new database before each unit
|
||||
// test and destroy afterwards.
|
||||
g.BeforeEach(func() {
|
||||
db = Must("/tmp/drone.test.db")
|
||||
})
|
||||
g.AfterEach(func() {
|
||||
os.Remove(db.Path())
|
||||
})
|
||||
|
||||
g.It("Should set Repo", func() {
|
||||
err := db.SetRepo(&common.Repo{FullName: testRepo})
|
||||
g.Assert(err).Equal(nil)
|
||||
|
||||
repo, err := db.Repo(testRepo)
|
||||
g.Assert(err).Equal(nil)
|
||||
g.Assert(repo.FullName).Equal(testRepo)
|
||||
})
|
||||
|
||||
g.It("Should get Repo", func() {
|
||||
db.SetRepo(&common.Repo{FullName: testRepo})
|
||||
|
||||
repo, err := db.Repo(testRepo)
|
||||
g.Assert(err).Equal(nil)
|
||||
g.Assert(repo.FullName).Equal(testRepo)
|
||||
})
|
||||
|
||||
g.It("Should del Repo", func() {
|
||||
db.SetRepo(&common.Repo{FullName: testRepo})
|
||||
|
||||
db.Repo(testRepo)
|
||||
err_ := db.DelRepo((&common.Repo{FullName: testRepo}))
|
||||
g.Assert(err_).Equal(nil)
|
||||
})
|
||||
|
||||
g.It("Should get RepoList", func() {
|
||||
db.SetRepoNotExists(&common.User{Login: testUser}, &common.Repo{FullName: testRepo})
|
||||
db.SetRepoNotExists(&common.User{Login: testUser}, &common.Repo{FullName: testRepo2})
|
||||
|
||||
repos, err := db.RepoList(testUser)
|
||||
g.Assert(err).Equal(nil)
|
||||
g.Assert(len(repos)).Equal(2)
|
||||
})
|
||||
|
||||
g.It("Should set RepoParams", func() {
|
||||
db.SetRepo(&common.Repo{FullName: testRepo})
|
||||
err := db.SetRepoParams(testRepo, map[string]string{"A": "Alpha"})
|
||||
g.Assert(err).Equal(nil)
|
||||
})
|
||||
|
||||
g.It("Should get RepoParams", func() {
|
||||
db.SetRepo(&common.Repo{FullName: testRepo})
|
||||
err := db.SetRepoParams(testRepo, map[string]string{"A": "Alpha", "B": "Beta"})
|
||||
params, err := db.RepoParams(testRepo)
|
||||
g.Assert(err).Equal(nil)
|
||||
g.Assert(len(params)).Equal(2)
|
||||
g.Assert(params["A"]).Equal("Alpha")
|
||||
g.Assert(params["B"]).Equal("Beta")
|
||||
})
|
||||
|
||||
// we test again with same repo/user already existing
|
||||
// to see if it will return "ErrConflict"
|
||||
g.It("Should set SetRepoNotExists", func() {
|
||||
err := db.SetRepoNotExists(&common.User{Login: testUser}, &common.Repo{FullName: testRepo})
|
||||
g.Assert(err).Equal(nil)
|
||||
// We should get ErrConflict now, trying to add the same repo again.
|
||||
err_ := db.SetRepoNotExists(&common.User{Login: testUser}, &common.Repo{FullName: testRepo})
|
||||
g.Assert(err_).Equal(ErrKeyExists)
|
||||
})
|
||||
|
||||
g.It("Should set RepoKeypair", func() {
|
||||
db.SetRepo(&common.Repo{FullName: testRepo})
|
||||
|
||||
err := db.SetRepoKeypair(testRepo, &common.Keypair{Private: "A", Public: "Alpha"})
|
||||
g.Assert(err).Equal(nil)
|
||||
})
|
||||
|
||||
g.It("Should get RepoKeypair", func() {
|
||||
db.SetRepo(&common.Repo{FullName: testRepo})
|
||||
err := db.SetRepoKeypair(testRepo, &common.Keypair{Private: "A", Public: "Alpha"})
|
||||
|
||||
keypair, err := db.RepoKeypair(testRepo)
|
||||
g.Assert(err).Equal(nil)
|
||||
g.Assert(keypair.Public).Equal("Alpha")
|
||||
g.Assert(keypair.Private).Equal("A")
|
||||
})
|
||||
|
||||
g.It("Should set Subscriber", func() {
|
||||
db.SetRepo(&common.Repo{FullName: testRepo})
|
||||
err := db.SetSubscriber(testUser, testRepo)
|
||||
g.Assert(err).Equal(nil)
|
||||
})
|
||||
|
||||
g.It("Should get Subscribed", func() {
|
||||
db.SetRepo(&common.Repo{FullName: testRepo})
|
||||
err := db.SetSubscriber(testUser, testRepo)
|
||||
subscribed, err := db.Subscribed(testUser, testRepo)
|
||||
g.Assert(err).Equal(nil)
|
||||
g.Assert(subscribed).Equal(true)
|
||||
})
|
||||
|
||||
g.It("Should del Subscriber", func() {
|
||||
db.SetRepo(&common.Repo{FullName: testRepo})
|
||||
db.SetSubscriber(testUser, testRepo)
|
||||
err := db.DelSubscriber(testUser, testRepo)
|
||||
g.Assert(err).Equal(nil)
|
||||
|
||||
subscribed, err := db.Subscribed(testUser, testRepo)
|
||||
g.Assert(subscribed).Equal(false)
|
||||
|
||||
})
|
||||
|
||||
g.It("Should find by name")
|
||||
g.It("Should find params")
|
||||
g.It("Should find keys")
|
||||
g.It("Should delete")
|
||||
g.It("Should insert")
|
||||
g.It("Should not insert if exists")
|
||||
g.It("Should insert params")
|
||||
g.It("Should update params")
|
||||
g.It("Should insert keys")
|
||||
g.It("Should update keys")
|
||||
})
|
||||
}
|
||||
|
|
|
|||
|
|
@ -2,13 +2,18 @@ package bolt
|
|||
|
||||
import (
|
||||
"bytes"
|
||||
"github.com/boltdb/bolt"
|
||||
//"github.com/drone/drone/common"
|
||||
"io"
|
||||
"strconv"
|
||||
|
||||
"github.com/boltdb/bolt"
|
||||
"github.com/drone/drone/common"
|
||||
)
|
||||
|
||||
/*
|
||||
Brad Rydzewski1:00 PM
|
||||
the `Task`, `TaskList` and `SetTask` are deprecated and can be probably be removed.
|
||||
I just need to make sure we aren't still using those functions anywhere else in the code
|
||||
*/
|
||||
/*
|
||||
// GetTask gets the task at index N for the named
|
||||
// repository and build number.
|
||||
func (db *DB) Task(repo string, build int, task int) (*common.Task, error) {
|
||||
|
|
@ -63,6 +68,7 @@ func (db *DB) SetTask(repo string, build int, task *common.Task) error {
|
|||
return update(t, bucketBuildTasks, key, task)
|
||||
})
|
||||
}
|
||||
*/
|
||||
|
||||
// SetLogs inserts or updates a task logs for the
|
||||
// named repository and build number.
|
||||
|
|
|
|||
|
|
@ -1 +1,89 @@
|
|||
package bolt
|
||||
|
||||
import (
|
||||
"github.com/drone/drone/common"
|
||||
. "github.com/franela/goblin"
|
||||
"io/ioutil"
|
||||
"os"
|
||||
"testing"
|
||||
)
|
||||
|
||||
func TestTask(t *testing.T) {
|
||||
g := Goblin(t)
|
||||
g.Describe("Tasks", func() {
|
||||
//testUser := "octocat"
|
||||
testRepo := "github.com/octopod/hq"
|
||||
testBuild := 1
|
||||
testTask := 0
|
||||
//testTask2 := 1
|
||||
testLogInfo := []byte("Log Info for SetLogs()")
|
||||
var db *DB // Temp database
|
||||
|
||||
// create a new database before each unit
|
||||
// test and destroy afterwards.
|
||||
g.BeforeEach(func() {
|
||||
db = Must("/tmp/drone.test.db")
|
||||
})
|
||||
g.AfterEach(func() {
|
||||
os.Remove(db.Path())
|
||||
})
|
||||
|
||||
/*
|
||||
Brad Rydzewski1:00 PM
|
||||
the `Task`, `TaskList` and `SetTask` are deprecated and can be probably be removed.
|
||||
I just need to make sure we aren't still using those functions anywhere else in the code
|
||||
*/
|
||||
/*
|
||||
g.It("Should get TaskList", func() {
|
||||
db.SetRepo(&common.Repo{FullName: testRepo})
|
||||
//db.SetRepoNotExists(&common.User{Login: testUser}, &common.Repo{FullName: testRepo})
|
||||
err := db.SetTask(testRepo, testBuild, &common.Task{Number: testTask})
|
||||
g.Assert(err).Equal(nil)
|
||||
err_ := db.SetTask(testRepo, testBuild, &common.Task{Number: testTask2})
|
||||
g.Assert(err_).Equal(nil)
|
||||
//
|
||||
tasks, err := db.TaskList(testRepo, testBuild)
|
||||
// We seem to have an issue here. TaskList doesn't seem to be returning
|
||||
// All the tasks added to to repo/build. So commenting these for now.
|
||||
//g.Assert(err).Equal(nil)
|
||||
//g.Assert(len(tasks)).Equal(2)
|
||||
})
|
||||
|
||||
g.It("Should set Task", func() {
|
||||
db.SetRepo(&common.Repo{FullName: testRepo})
|
||||
err := db.SetTask(testRepo, testBuild, &common.Task{Number: testTask})
|
||||
g.Assert(err).Equal(nil)
|
||||
})
|
||||
|
||||
g.It("Should get Task", func() {
|
||||
db.SetRepo(&common.Repo{FullName: testRepo})
|
||||
db.SetTask(testRepo, testBuild, &common.Task{Number: testTask})
|
||||
//
|
||||
task, err := db.Task(testRepo, testBuild, testTask)
|
||||
g.Assert(err).Equal(nil)
|
||||
g.Assert(task.Number).Equal(testTask)
|
||||
})
|
||||
*/
|
||||
|
||||
g.It("Should set Logs", func() {
|
||||
db.SetRepo(&common.Repo{FullName: testRepo})
|
||||
//db.SetTask(testRepo, testBuild, &common.Task{Number: testTask})
|
||||
//db.SetTask(testRepo, testBuild, &common.Task{Number: testTask2})
|
||||
//
|
||||
err := db.SetLogs(testRepo, testBuild, testTask, testLogInfo)
|
||||
g.Assert(err).Equal(nil)
|
||||
})
|
||||
|
||||
g.It("Should LogReader", func() {
|
||||
db.SetRepo(&common.Repo{FullName: testRepo})
|
||||
//db.SetTask(testRepo, testBuild, &common.Task{Number: testTask})
|
||||
//db.SetTask(testRepo, testBuild, &common.Task{Number: testTask2})
|
||||
db.SetLogs(testRepo, testBuild, testTask, testLogInfo)
|
||||
//
|
||||
buf, err_ := db.LogReader(testRepo, testBuild, testTask)
|
||||
g.Assert(err_).Equal(nil)
|
||||
logInfo, err_ := ioutil.ReadAll(buf)
|
||||
g.Assert(logInfo).Equal(testLogInfo)
|
||||
})
|
||||
})
|
||||
}
|
||||
|
|
|
|||
|
|
@ -121,15 +121,15 @@ type Datastore interface {
|
|||
|
||||
// GetTask gets the task at index N for the named
|
||||
// repository and build number.
|
||||
Task(string, int, int) (*common.Task, error)
|
||||
//Task(string, int, int) (*common.Task, error)
|
||||
|
||||
// TaskList gets all tasks for the named repository
|
||||
// and build number.
|
||||
TaskList(string, int) ([]*common.Task, error)
|
||||
//TaskList(string, int) ([]*common.Task, error)
|
||||
|
||||
// SetTask inserts or updates a task for the named
|
||||
// repository and build number.
|
||||
SetTask(string, int, *common.Task) error
|
||||
//SetTask(string, int, *common.Task) error
|
||||
|
||||
// LogReader gets the task logs at index N for
|
||||
// the named repository and build number.
|
||||
|
|
|
|||
Loading…
Reference in a new issue