ci: set up Codecov (#45)

* ci: set up Codecov

* commit: add more tests

* repo_commit: improve cache hit rate

* commit: add more tests

* commit: add more tests

* commit: finish all tests

Author: unknwon

.travis.yml

@@ -6,7 +6,16 @@ go:
   - 1.11.x
   - 1.12.x
   - 1.13.x
+go_import_path: github.com/gogs/git-module
+
+env:
+  - GO111MODULE=on
+
+install:
+  - go get -t -v ./...
 
-install: go get -v -t ./...
 script:
-  - go test -v -cover -race -verbose
+  - go test -v -race -verbose -coverprofile=coverage.txt -covermode=atomic
+
+after_success:
+  - bash <(curl -s https://codecov.io/bash)

README.md

@@ -1,6 +1,6 @@
 # Git Module 
 
-[![Build Status](https://img.shields.io/travis/gogs/git-module/master.svg?style=for-the-badge&logo=travis)](https://travis-ci.org/gogs/git-module) [![Build status](https://img.shields.io/appveyor/ci/unknwon/gogs-git-module?logo=appveyor&style=for-the-badge)](https://ci.appveyor.com/project/unknwon/gogs-git-module/branch/master) [![Sourcegraph](https://img.shields.io/badge/view%20on-Sourcegraph-brightgreen.svg?style=for-the-badge&logo=sourcegraph)](https://sourcegraph.com/github.com/gogs/git-module)
+[![Build Status](https://img.shields.io/travis/gogs/git-module/master.svg?style=for-the-badge&logo=travis)](https://travis-ci.org/gogs/git-module) [![Build status](https://img.shields.io/appveyor/ci/unknwon/gogs-git-module?logo=appveyor&style=for-the-badge)](https://ci.appveyor.com/project/unknwon/gogs-git-module/branch/master) [![codecov](https://img.shields.io/codecov/c/github/gogs/git-module/master?logo=codecov&style=for-the-badge)](https://codecov.io/gh/gogs/git-module) [![Sourcegraph](https://img.shields.io/badge/view%20on-Sourcegraph-brightgreen.svg?style=for-the-badge&logo=sourcegraph)](https://sourcegraph.com/github.com/gogs/git-module)
 
 Package git-module is a Go module for Git access through shell commands.
 

codecov.yml

@@ -0,0 +1,2 @@
+comment:
+  layout: 'diff, files'

commit.go

@@ -5,6 +5,9 @@
 package git
 
 import (
+	"bytes"
+	"io"
+	"io/ioutil"
 	"net/http"
 	"strings"
 	"sync"
@@ -103,9 +106,9 @@ func (c *Commit) CommitsCount(opts ...RevListCountOptions) (int64, error) {
 	return c.repo.RevListCount([]string{c.id.String()}, opts...)
 }
 
-// FilesChangedSince returns a list of files changed since given commit ID.
-func (c *Commit) FilesChangedSince(commitID string, opts ...DiffNameOnlyOptions) ([]string, error) {
-	return c.repo.DiffNameOnly(commitID, c.id.String(), opts...)
+// FilesChangedSince returns a list of files changed after given commit ID.
+func (c *Commit) FilesChangedAfter(after string, opts ...DiffNameOnlyOptions) ([]string, error) {
+	return c.repo.DiffNameOnly(after, c.id.String(), opts...)
 }
 
 // CommitsAfter returns a list of commits after given commit ID up to this commit. The returned
@@ -116,28 +119,59 @@ func (c *Commit) CommitsAfter(after string, opts ...RevListOptions) ([]*Commit,
 
 // Ancestors returns a list of ancestors of this commit in reverse chronological order.
 func (c *Commit) Ancestors(opts ...LogOptions) ([]*Commit, error) {
-	return c.repo.Log(c.id.String(), opts...)
+	if c.ParentsCount() == 0 {
+		return []*Commit{}, nil
+	}
+
+	var opt LogOptions
+	if len(opts) > 0 {
+		opt = opts[0]
+	}
+
+	opt.Skip++
+
+	return c.repo.Log(c.id.String(), opt)
+}
+
+type limitWriter struct {
+	W io.Writer
+	N int64
 }
 
-func isImageFile(data []byte) (string, bool) {
-	contentType := http.DetectContentType(data)
-	if strings.Contains(contentType, "image/") {
-		return contentType, true
+func (w *limitWriter) Write(p []byte) (int, error) {
+	if w.N <= 0 {
+		return len(p), nil
 	}
-	return contentType, false
+
+	limit := int64(len(p))
+	if limit > w.N {
+		limit = w.N
+	}
+	n, err := w.W.Write(p[:limit])
+	w.N -= int64(n)
+
+	// Prevent "short write" error
+	return len(p), err
 }
 
-// IsImageFile returns true if the commit is a image blob.
-func (c *Commit) IsImageFile(name string) bool {
+// IsImageFile returns true if the commit is an image blob.
+func (c *Commit) IsImageFile(name string) (bool, error) {
 	blob, err := c.Blob(name)
 	if err != nil {
-		return false
+		return false, err
 	}
 
-	p, err := blob.Bytes()
+	buf := new(bytes.Buffer)
+	buf.Grow(512)
+	stdout := &limitWriter{
+		W: buf,
+		N: int64(buf.Cap()),
+	}
+
+	err = blob.Pipeline(stdout, ioutil.Discard)
 	if err != nil {
-		return false
+		return false, err
 	}
-	_, isImage := isImageFile(p)
-	return isImage
+
+	return strings.Contains(http.DetectContentType(buf.Bytes()), "image/"), nil
 }