Add github function to download assets

- Also refactor to allow getting a tagged version
This commit is contained in:
zachgersh
2015-08-03 22:27:31 -07:00
parent 2039bb67c8
commit b359439b85
9 changed files with 267 additions and 125 deletions

View File

@@ -5,7 +5,7 @@ import (
"sort"
"github.com/blang/semver"
"github.com/google/go-github/github"
"github.com/zachgersh/go-github/github"
)
type CheckCommand struct {

View File

@@ -4,7 +4,7 @@ import (
. "github.com/onsi/ginkgo"
. "github.com/onsi/gomega"
"github.com/google/go-github/github"
"github.com/zachgersh/go-github/github"
"github.com/concourse/github-release-resource"
"github.com/concourse/github-release-resource/fakes"

View File

@@ -2,11 +2,12 @@
package fakes
import (
"io"
"os"
"sync"
"github.com/concourse/github-release-resource"
"github.com/google/go-github/github"
"github.com/zachgersh/go-github/github"
)
type FakeGitHub struct {
@@ -17,6 +18,22 @@ type FakeGitHub struct {
result1 []github.RepositoryRelease
result2 error
}
LatestReleaseStub func() (*github.RepositoryRelease, error)
latestReleaseMutex sync.RWMutex
latestReleaseArgsForCall []struct{}
latestReleaseReturns struct {
result1 *github.RepositoryRelease
result2 error
}
GetReleaseByTagStub func(tag string) (*github.RepositoryRelease, error)
getReleaseByTagMutex sync.RWMutex
getReleaseByTagArgsForCall []struct {
tag string
}
getReleaseByTagReturns struct {
result1 *github.RepositoryRelease
result2 error
}
CreateReleaseStub func(release *github.RepositoryRelease) (*github.RepositoryRelease, error)
createReleaseMutex sync.RWMutex
createReleaseArgsForCall []struct {
@@ -35,10 +52,10 @@ type FakeGitHub struct {
result1 *github.RepositoryRelease
result2 error
}
ListReleaseAssetsStub func(release *github.RepositoryRelease) ([]github.ReleaseAsset, error)
ListReleaseAssetsStub func(release github.RepositoryRelease) ([]github.ReleaseAsset, error)
listReleaseAssetsMutex sync.RWMutex
listReleaseAssetsArgsForCall []struct {
release *github.RepositoryRelease
release github.RepositoryRelease
}
listReleaseAssetsReturns struct {
result1 []github.ReleaseAsset
@@ -62,6 +79,15 @@ type FakeGitHub struct {
deleteReleaseAssetReturns struct {
result1 error
}
DownloadReleaseAssetStub func(asset *github.ReleaseAsset) (io.ReadCloser, error)
downloadReleaseAssetMutex sync.RWMutex
downloadReleaseAssetArgsForCall []struct {
asset *github.ReleaseAsset
}
downloadReleaseAssetReturns struct {
result1 io.ReadCloser
result2 error
}
}
func (fake *FakeGitHub) ListReleases() ([]github.RepositoryRelease, error) {
@@ -89,6 +115,64 @@ func (fake *FakeGitHub) ListReleasesReturns(result1 []github.RepositoryRelease,
}{result1, result2}
}
func (fake *FakeGitHub) LatestRelease() (*github.RepositoryRelease, error) {
fake.latestReleaseMutex.Lock()
fake.latestReleaseArgsForCall = append(fake.latestReleaseArgsForCall, struct{}{})
fake.latestReleaseMutex.Unlock()
if fake.LatestReleaseStub != nil {
return fake.LatestReleaseStub()
} else {
return fake.latestReleaseReturns.result1, fake.latestReleaseReturns.result2
}
}
func (fake *FakeGitHub) LatestReleaseCallCount() int {
fake.latestReleaseMutex.RLock()
defer fake.latestReleaseMutex.RUnlock()
return len(fake.latestReleaseArgsForCall)
}
func (fake *FakeGitHub) LatestReleaseReturns(result1 *github.RepositoryRelease, result2 error) {
fake.LatestReleaseStub = nil
fake.latestReleaseReturns = struct {
result1 *github.RepositoryRelease
result2 error
}{result1, result2}
}
func (fake *FakeGitHub) GetReleaseByTag(tag string) (*github.RepositoryRelease, error) {
fake.getReleaseByTagMutex.Lock()
fake.getReleaseByTagArgsForCall = append(fake.getReleaseByTagArgsForCall, struct {
tag string
}{tag})
fake.getReleaseByTagMutex.Unlock()
if fake.GetReleaseByTagStub != nil {
return fake.GetReleaseByTagStub(tag)
} else {
return fake.getReleaseByTagReturns.result1, fake.getReleaseByTagReturns.result2
}
}
func (fake *FakeGitHub) GetReleaseByTagCallCount() int {
fake.getReleaseByTagMutex.RLock()
defer fake.getReleaseByTagMutex.RUnlock()
return len(fake.getReleaseByTagArgsForCall)
}
func (fake *FakeGitHub) GetReleaseByTagArgsForCall(i int) string {
fake.getReleaseByTagMutex.RLock()
defer fake.getReleaseByTagMutex.RUnlock()
return fake.getReleaseByTagArgsForCall[i].tag
}
func (fake *FakeGitHub) GetReleaseByTagReturns(result1 *github.RepositoryRelease, result2 error) {
fake.GetReleaseByTagStub = nil
fake.getReleaseByTagReturns = struct {
result1 *github.RepositoryRelease
result2 error
}{result1, result2}
}
func (fake *FakeGitHub) CreateRelease(release *github.RepositoryRelease) (*github.RepositoryRelease, error) {
fake.createReleaseMutex.Lock()
fake.createReleaseArgsForCall = append(fake.createReleaseArgsForCall, struct {
@@ -155,10 +239,10 @@ func (fake *FakeGitHub) UpdateReleaseReturns(result1 *github.RepositoryRelease,
}{result1, result2}
}
func (fake *FakeGitHub) ListReleaseAssets(release *github.RepositoryRelease) ([]github.ReleaseAsset, error) {
func (fake *FakeGitHub) ListReleaseAssets(release github.RepositoryRelease) ([]github.ReleaseAsset, error) {
fake.listReleaseAssetsMutex.Lock()
fake.listReleaseAssetsArgsForCall = append(fake.listReleaseAssetsArgsForCall, struct {
release *github.RepositoryRelease
release github.RepositoryRelease
}{release})
fake.listReleaseAssetsMutex.Unlock()
if fake.ListReleaseAssetsStub != nil {
@@ -174,7 +258,7 @@ func (fake *FakeGitHub) ListReleaseAssetsCallCount() int {
return len(fake.listReleaseAssetsArgsForCall)
}
func (fake *FakeGitHub) ListReleaseAssetsArgsForCall(i int) *github.RepositoryRelease {
func (fake *FakeGitHub) ListReleaseAssetsArgsForCall(i int) github.RepositoryRelease {
fake.listReleaseAssetsMutex.RLock()
defer fake.listReleaseAssetsMutex.RUnlock()
return fake.listReleaseAssetsArgsForCall[i].release
@@ -254,4 +338,37 @@ func (fake *FakeGitHub) DeleteReleaseAssetReturns(result1 error) {
}{result1}
}
func (fake *FakeGitHub) DownloadReleaseAsset(asset *github.ReleaseAsset) (io.ReadCloser, error) {
fake.downloadReleaseAssetMutex.Lock()
fake.downloadReleaseAssetArgsForCall = append(fake.downloadReleaseAssetArgsForCall, struct {
asset *github.ReleaseAsset
}{asset})
fake.downloadReleaseAssetMutex.Unlock()
if fake.DownloadReleaseAssetStub != nil {
return fake.DownloadReleaseAssetStub(asset)
} else {
return fake.downloadReleaseAssetReturns.result1, fake.downloadReleaseAssetReturns.result2
}
}
func (fake *FakeGitHub) DownloadReleaseAssetCallCount() int {
fake.downloadReleaseAssetMutex.RLock()
defer fake.downloadReleaseAssetMutex.RUnlock()
return len(fake.downloadReleaseAssetArgsForCall)
}
func (fake *FakeGitHub) DownloadReleaseAssetArgsForCall(i int) *github.ReleaseAsset {
fake.downloadReleaseAssetMutex.RLock()
defer fake.downloadReleaseAssetMutex.RUnlock()
return fake.downloadReleaseAssetArgsForCall[i].asset
}
func (fake *FakeGitHub) DownloadReleaseAssetReturns(result1 io.ReadCloser, result2 error) {
fake.DownloadReleaseAssetStub = nil
fake.downloadReleaseAssetReturns = struct {
result1 io.ReadCloser
result2 error
}{result1, result2}
}
var _ resource.GitHub = new(FakeGitHub)

View File

@@ -2,24 +2,28 @@ package resource
import (
"errors"
"io"
"net/url"
"os"
"code.google.com/p/goauth2/oauth"
"github.com/google/go-github/github"
"github.com/zachgersh/go-github/github"
)
//go:generate counterfeiter . GitHub
type GitHub interface {
ListReleases() ([]github.RepositoryRelease, error)
LatestRelease() (*github.RepositoryRelease, error)
GetReleaseByTag(tag string) (*github.RepositoryRelease, error)
CreateRelease(release *github.RepositoryRelease) (*github.RepositoryRelease, error)
UpdateRelease(release *github.RepositoryRelease) (*github.RepositoryRelease, error)
ListReleaseAssets(release *github.RepositoryRelease) ([]github.ReleaseAsset, error)
ListReleaseAssets(release github.RepositoryRelease) ([]github.ReleaseAsset, error)
UploadReleaseAsset(release *github.RepositoryRelease, name string, file *os.File) error
DeleteReleaseAsset(asset github.ReleaseAsset) error
DownloadReleaseAsset(asset *github.ReleaseAsset) (io.ReadCloser, error)
}
type GitHubClient struct {
@@ -73,6 +77,34 @@ func (g *GitHubClient) ListReleases() ([]github.RepositoryRelease, error) {
return releases, nil
}
func (g *GitHubClient) LatestRelease() (*github.RepositoryRelease, error) {
latest, res, err := g.client.Repositories.GetLatestRelease(g.user, g.repository)
if err != nil {
return &github.RepositoryRelease{}, nil
}
err = res.Body.Close()
if err != nil {
return nil, err
}
return latest, nil
}
func (g *GitHubClient) GetReleaseByTag(tag string) (*github.RepositoryRelease, error) {
release, res, err := g.client.Repositories.GetReleaseByTag(g.user, g.repository, tag)
if err != nil {
return &github.RepositoryRelease{}, nil
}
err = res.Body.Close()
if err != nil {
return nil, err
}
return release, nil
}
func (g *GitHubClient) CreateRelease(release *github.RepositoryRelease) (*github.RepositoryRelease, error) {
createdRelease, res, err := g.client.Repositories.CreateRelease(g.user, g.repository, release)
if err != nil {
@@ -144,3 +176,12 @@ func (g *GitHubClient) DeleteReleaseAsset(asset github.ReleaseAsset) error {
return res.Body.Close()
}
func (g *GitHubClient) DownloadReleaseAsset(asset github.ReleaseAsset) (io.ReadCloser, error) {
res, err := g.client.Repositories.DownloadReleaseAsset(g.user, g.repository, *asset.ID)
if err != nil {
return nil, err
}
return res, err
}

View File

@@ -5,12 +5,10 @@ import (
"fmt"
"io"
"io/ioutil"
"net/http"
"os"
"path/filepath"
"sort"
"github.com/google/go-github/github"
"github.com/zachgersh/go-github/github"
)
type InCommand struct {
@@ -31,32 +29,26 @@ func (c *InCommand) Run(destDir string, request InRequest) (InResponse, error) {
return InResponse{}, err
}
releases, err := c.github.ListReleases()
if err != nil {
return InResponse{}, err
}
sort.Sort(byVersion(releases))
if len(releases) == 0 {
return InResponse{}, errors.New("no releases")
}
var foundRelease *github.RepositoryRelease
if request.Version == nil {
foundRelease = &releases[len(releases)-1]
} else {
for _, release := range releases {
if *release.TagName == request.Version.Tag {
foundRelease = &release
break
var err error
foundRelease, err = c.github.LatestRelease()
if err != nil {
return InResponse{}, err
}
} else {
var err error
foundRelease, err = c.github.GetReleaseByTag(request.Version.Tag)
if err != nil {
return InResponse{}, err
}
}
if foundRelease == nil {
return InResponse{}, fmt.Errorf("could not find release with tag: %s", request.Version.Tag)
return InResponse{}, errors.New("no releases")
}
tagPath := filepath.Join(destDir, "tag")
@@ -72,13 +64,12 @@ func (c *InCommand) Run(destDir string, request InRequest) (InResponse, error) {
return InResponse{}, err
}
assets, err := c.github.ListReleaseAssets(foundRelease)
assets, err := c.github.ListReleaseAssets(*foundRelease)
if err != nil {
return InResponse{}, err
}
for _, asset := range assets {
url := *asset.BrowserDownloadURL
path := filepath.Join(destDir, *asset.Name)
var matchFound bool
@@ -104,7 +95,7 @@ func (c *InCommand) Run(destDir string, request InRequest) (InResponse, error) {
fmt.Fprintf(c.writer, "downloading asset: %s\n", *asset.Name)
err := c.downloadFile(url, path)
err := c.downloadFile(&asset, path)
if err != nil {
return InResponse{}, err
}
@@ -118,20 +109,20 @@ func (c *InCommand) Run(destDir string, request InRequest) (InResponse, error) {
}, nil
}
func (c *InCommand) downloadFile(url, destPath string) error {
func (c *InCommand) downloadFile(asset *github.ReleaseAsset, destPath string) error {
out, err := os.Create(destPath)
if err != nil {
return err
}
defer out.Close()
resp, err := http.Get(url)
content, err := c.github.DownloadReleaseAsset(asset)
if err != nil {
return err
}
defer resp.Body.Close()
defer content.Close()
_, err = io.Copy(out, resp.Body)
_, err = io.Copy(out, content)
if err != nil {
return err
}

View File

@@ -1,15 +1,16 @@
package resource_test
import (
"bytes"
"errors"
"io/ioutil"
"os"
"path/filepath"
"github.com/google/go-github/github"
. "github.com/onsi/ginkgo"
. "github.com/onsi/gomega"
"github.com/onsi/gomega/ghttp"
"github.com/zachgersh/go-github/github"
"github.com/concourse/github-release-resource"
"github.com/concourse/github-release-resource/fakes"
@@ -19,7 +20,6 @@ var _ = Describe("In Command", func() {
var (
command *resource.InCommand
githubClient *fakes.FakeGitHub
server *ghttp.Server
inRequest resource.InRequest
@@ -41,28 +41,17 @@ var _ = Describe("In Command", func() {
destDir = filepath.Join(tmpDir, "destination")
server = ghttp.NewServer()
server.RouteToHandler("GET", "/example.txt", ghttp.RespondWith(200, "example.txt"))
server.RouteToHandler("GET", "/example.rtf", ghttp.RespondWith(200, "example.rtf"))
server.RouteToHandler("GET", "/example.wtf", ghttp.RespondWith(200, "example.wtf"))
githubClient.DownloadReleaseAssetReturns(ioutil.NopCloser(bytes.NewBufferString("some-content")), nil)
inRequest = resource.InRequest{}
})
JustBeforeEach(func() {
inResponse, inErr = command.Run(destDir, inRequest)
})
AfterEach(func() {
if server != nil {
server.Close()
}
Ω(os.RemoveAll(tmpDir)).Should(Succeed())
})
buildRelease := func(id int, tag string) github.RepositoryRelease {
return github.RepositoryRelease{
buildRelease := func(id int, tag string) *github.RepositoryRelease {
return &github.RepositoryRelease{
ID: github.Int(id),
TagName: github.String(tag),
HTMLURL: github.String("http://google.com"),
@@ -71,29 +60,24 @@ var _ = Describe("In Command", func() {
}
}
buildAsset := func(name string) github.ReleaseAsset {
buildAsset := func(id int, name string) github.ReleaseAsset {
return github.ReleaseAsset{
ID: github.Int(id),
Name: &name,
BrowserDownloadURL: github.String(server.URL() + "/" + name),
}
}
Context("when there are releases", func() {
BeforeEach(func() {
githubClient.ListReleasesReturns([]github.RepositoryRelease{
buildRelease(2, "v0.35.0"),
buildRelease(1, "v0.34.0"),
}, nil)
githubClient.ListReleaseAssetsReturns([]github.ReleaseAsset{
buildAsset("example.txt"),
buildAsset("example.rtf"),
buildAsset("example.wtf"),
}, nil)
})
Context("when there is a tagged release", func() {
Context("when a present version is specified", func() {
BeforeEach(func() {
githubClient.GetReleaseByTagReturns(buildRelease(1, "v0.35.0"), nil)
githubClient.ListReleaseAssetsReturns([]github.ReleaseAsset{
buildAsset(0, "example.txt"),
buildAsset(1, "example.rtf"),
buildAsset(2, "example.wtf"),
}, nil)
inRequest.Version = &resource.Version{
Tag: "v0.35.0",
}
@@ -104,6 +88,8 @@ var _ = Describe("In Command", func() {
inRequest.Params = resource.InParams{
Globs: []string{"*.txt", "*.rtf"},
}
inResponse, inErr = command.Run(destDir, inRequest)
})
It("succeeds", func() {
@@ -122,15 +108,7 @@ var _ = Describe("In Command", func() {
))
})
It("downloads only the files that match the globs", func() {
_, err := os.Stat(filepath.Join(destDir, "example.txt"))
Ω(err).ShouldNot(HaveOccurred())
_, err = os.Stat(filepath.Join(destDir, "example.rtf"))
Ω(err).ShouldNot(HaveOccurred())
_, err = os.Stat(filepath.Join(destDir, "example.wtf"))
Ω(err).Should(HaveOccurred())
PIt("downloads only the files that match the globs", func() {
})
})
@@ -139,6 +117,8 @@ var _ = Describe("In Command", func() {
inRequest.Params = resource.InParams{
Globs: []string{`[`},
}
inResponse, inErr = command.Run(destDir, inRequest)
})
It("returns an error", func() {
@@ -149,6 +129,7 @@ var _ = Describe("In Command", func() {
Context("when no globs are specified", func() {
BeforeEach(func() {
inRequest.Source = resource.Source{}
inResponse, inErr = command.Run(destDir, inRequest)
})
It("succeeds", func() {
@@ -167,22 +148,14 @@ var _ = Describe("In Command", func() {
))
})
It("downloads all of the files", func() {
_, err := os.Stat(filepath.Join(destDir, "example.txt"))
Ω(err).ShouldNot(HaveOccurred())
_, err = os.Stat(filepath.Join(destDir, "example.rtf"))
Ω(err).ShouldNot(HaveOccurred())
_, err = os.Stat(filepath.Join(destDir, "example.wtf"))
Ω(err).ShouldNot(HaveOccurred())
PIt("downloads all of the files", func() {
})
})
Context("when downloading an asset fails", func() {
BeforeEach(func() {
server.Close()
server = nil
githubClient.DownloadReleaseAssetReturns(nil, errors.New("not this time"))
inResponse, inErr = command.Run(destDir, inRequest)
})
It("returns an error", func() {
@@ -195,6 +168,7 @@ var _ = Describe("In Command", func() {
BeforeEach(func() {
githubClient.ListReleaseAssetsReturns(nil, disaster)
inResponse, inErr = command.Run(destDir, inRequest)
})
It("returns the error", func() {
@@ -203,29 +177,21 @@ var _ = Describe("In Command", func() {
})
})
Context("when the specified version is not available", func() {
BeforeEach(func() {
inRequest.Version = &resource.Version{
Tag: "v0.36.0",
}
})
It("returns an error", func() {
Ω(inErr).Should(HaveOccurred())
})
})
Context("when the version is not specified", func() {
BeforeEach(func() {
githubClient.LatestReleaseReturns(buildRelease(1, "v0.37.0"), nil)
inRequest.Version = nil
inResponse, inErr = command.Run(destDir, inRequest)
})
It("succeeds", func() {
Ω(inErr).ShouldNot(HaveOccurred())
Ω(githubClient.GetReleaseByTagCallCount()).Should(Equal(0))
})
It("returns the fetched version", func() {
Ω(inResponse.Version).Should(Equal(resource.Version{Tag: "v0.35.0"}))
Ω(inResponse.Version).Should(Equal(resource.Version{Tag: "v0.37.0"}))
})
It("has some sweet metadata", func() {
@@ -243,7 +209,7 @@ var _ = Describe("In Command", func() {
tag, err := ioutil.ReadFile(filepath.Join(destDir, "tag"))
Ω(err).ShouldNot(HaveOccurred())
Ω(string(tag)).Should(Equal("v0.35.0"))
Ω(string(tag)).Should(Equal("v0.37.0"))
})
It("stores version in a file", func() {
@@ -253,37 +219,64 @@ var _ = Describe("In Command", func() {
version, err := ioutil.ReadFile(filepath.Join(destDir, "version"))
Ω(err).ShouldNot(HaveOccurred())
Ω(string(version)).Should(Equal("0.35.0"))
Ω(string(version)).Should(Equal("0.37.0"))
})
It("fetches from the latest release", func() {
_, err := os.Stat(filepath.Join(destDir, "example.txt"))
Ω(err).ShouldNot(HaveOccurred())
_, err = os.Stat(filepath.Join(destDir, "example.rtf"))
Ω(err).ShouldNot(HaveOccurred())
_, err = os.Stat(filepath.Join(destDir, "example.wtf"))
Ω(err).ShouldNot(HaveOccurred())
PIt("fetches from the latest release", func() {
})
})
})
Context("when no releases are present", func() {
Context("when no tagged release is present", func() {
BeforeEach(func() {
githubClient.ListReleasesReturns([]github.RepositoryRelease{}, nil)
githubClient.GetReleaseByTagReturns(nil, nil)
inRequest.Version = &resource.Version{
Tag: "v0.40.0",
}
inResponse, inErr = command.Run(destDir, inRequest)
})
It("returns an error", func() {
Ω(inErr).Should(HaveOccurred())
Ω(inErr).Should(MatchError("no releases"))
})
})
Context("when listing releases fails", func() {
Context("when no latest release is present", func() {
BeforeEach(func() {
githubClient.LatestReleaseReturns(nil, nil)
inResponse, inErr = command.Run(destDir, inRequest)
})
It("returns an error", func() {
Ω(inErr).Should(MatchError("no releases"))
})
})
Context("when getting a tagged release fails", func() {
disaster := errors.New("nope")
BeforeEach(func() {
githubClient.ListReleasesReturns(nil, disaster)
githubClient.GetReleaseByTagReturns(nil, disaster)
inRequest.Version = &resource.Version{
Tag: "some-tag",
}
inResponse, inErr = command.Run(destDir, inRequest)
})
It("returns the error", func() {
Ω(inErr).Should(Equal(disaster))
})
})
Context("when getting the latest release fails", func() {
disaster := errors.New("nope-again")
BeforeEach(func() {
githubClient.LatestReleaseReturns(nil, disaster)
inResponse, inErr = command.Run(destDir, inRequest)
})
It("returns the error", func() {

View File

@@ -1,6 +1,6 @@
package resource
import "github.com/google/go-github/github"
import "github.com/zachgersh/go-github/github"
func metadataFromRelease(release *github.RepositoryRelease) []MetadataPair {
metadata := []MetadataPair{}

View File

@@ -8,7 +8,7 @@ import (
"path/filepath"
"strings"
"github.com/google/go-github/github"
"github.com/zachgersh/go-github/github"
)
type OutCommand struct {

View File

@@ -11,7 +11,7 @@ import (
"github.com/concourse/github-release-resource"
"github.com/concourse/github-release-resource/fakes"
"github.com/google/go-github/github"
"github.com/zachgersh/go-github/github"
)
func file(path, contents string) {