fix avelino#1446

mrKappen · Aug 12, 2020 · dd8234b · dd8234b
1 parent 30efeaa
commit dd8234b
Show file tree

Hide file tree

Showing 3 changed files with 54 additions and 44 deletions.
diff --git a/repo_test.go b/repo_test.go
@@ -1,7 +1,6 @@
 package main
 
 import (
-	"bytes"
 	"io/ioutil"
 	"os"
 	"regexp"
@@ -11,7 +10,6 @@ import (
 	"text/template"
 
 	"github.com/PuerkitoBio/goquery"
-	"github.com/russross/blackfriday"
 	gfm "github.com/shurcooL/github_flavored_markdown"
 )
 
@@ -97,26 +95,6 @@ func testList(t *testing.T, list *goquery.Selection) {
 	})
 }
 
-func readme() []byte {
-	input, err := ioutil.ReadFile("./README.md")
-	if err != nil {
-		panic(err)
-	}
-	html := append([]byte("<body>"), blackfriday.MarkdownCommon(input)...)
-	html = append(html, []byte("</body>")...)
-	return html
-}
-
-func startQuery() *goquery.Document {
-	buf := bytes.NewBuffer(readme())
-	query, err := goquery.NewDocumentFromReader(buf)
-	if err != nil {
-		panic(err)
-	}
-
-	return query
-}
-
 func checkAlphabeticOrder(t *testing.T, s *goquery.Selection) {
 	items := s.Find("li > a:first-child").Map(func(_ int, li *goquery.Selection) string {
 		return strings.ToLower(li.Text())

diff --git a/scripts.go b/scripts.go
@@ -0,0 +1,29 @@
+package main
+
+import (
+	"bytes"
+	"io/ioutil"
+
+	"github.com/PuerkitoBio/goquery"
+	"github.com/russross/blackfriday"
+)
+
+func readme() []byte {
+	input, err := ioutil.ReadFile("./README.md")
+	if err != nil {
+		panic(err)
+	}
+	html := append([]byte("<body>"), blackfriday.MarkdownCommon(input)...)
+	html = append(html, []byte("</body>")...)
+	return html
+}
+
+func startQuery() *goquery.Document {
+	buf := bytes.NewBuffer(readme())
+	query, err := goquery.NewDocumentFromReader(buf)
+	if err != nil {
+		panic(err)
+	}
+
+	return query
+}
diff --git a/stale_repositories_test.go → test_stale_repositories.go b/stale_repositories_test.go → test_stale_repositories.go
@@ -4,11 +4,11 @@ import (
 	"bytes"
 	"encoding/json"
 	"fmt"
+	"log"
 	"net/http"
 	"os"
 	"regexp"
 	"strings"
-	"testing"
 	"time"
 
 	"github.com/PuerkitoBio/goquery"
@@ -18,88 +18,91 @@ import (
 var reGithubRepo = regexp.MustCompile("https://github.com/[a-zA-Z0-9-.]+/[a-zA-Z0-9-.]+$")
 var githubGETCOMMITS = "https://api.github.com/repos%s/commits"
 var githubPOSTISSUES = "https://api.github.com/repos/avelino/awesome-go/issues"
-var NUMBEROFYEARS time.Duration = 1
-var ISSUEFMT = "Investigate %s. Has not had any new commits in a while"
-var DELAYINSECONDS time.Duration = 1
+var numberOfYears time.Duration = 1
+var issueFmt = "Investigate %s. Has not had any new commits in a while"
+var delay time.Duration = 1
 
-type TokenSource struct {
+type tokenSource struct {
 	AccessToken string
 }
-type Issue struct {
+type issue struct {
 	Title string `json:"title"`
 }
 
-func (t *TokenSource) Token() (*oauth2.Token, error) {
+func (t *tokenSource) Token() (*oauth2.Token, error) {
 	token := &oauth2.Token{
 		AccessToken: t.AccessToken,
 	}
 	return token, nil
 }
-
-func createIssues(issueRequests []*http.Request, oauthClient *http.Client, t *testing.T) {
+func createIssues(issueRequests []*http.Request, oauthClient *http.Client) {
 	for _, req := range issueRequests {
 		oauthClient.Do(req)
-		time.Sleep(DELAYINSECONDS * time.Second)
+		time.Sleep(delay * time.Second)
 	}
 }
-func testCommitAge(href string, oauthClient *http.Client, issueRequests *[]*http.Request, t *testing.T) {
+func testCommitAge(href string, oauthClient *http.Client, issueRequests *[]*http.Request) {
 	var isValidRepo bool
 	var isAged bool
 	var ownerRepo string
 	isValidRepo = reGithubRepo.MatchString(href)
 	var apiCall string
 	var respObj []map[string]interface{}
 	now := time.Now()
-	since := now.Add(-1 * 365 * 24 * NUMBEROFYEARS * time.Hour)
+	since := now.Add(-1 * 365 * 24 * numberOfYears * time.Hour)
 	sinceQuery := since.Format(time.RFC3339)
 	if isValidRepo {
 		ownerRepo = strings.ReplaceAll(href, "https://github.com", "")
 		apiCall = fmt.Sprintf(githubGETCOMMITS, ownerRepo)
 		req, err := http.NewRequest("GET", apiCall, nil)
 		if err != nil {
-			t.Errorf("Failed at repository %s", ownerRepo)
+			log.Printf("Failed at repository %s\n", ownerRepo)
 			return
 		}
 		q := req.URL.Query()
 		q.Add("since", sinceQuery)
 		req.URL.RawQuery = q.Encode()
 		resp, err := oauthClient.Do(req)
 		if err != nil {
-			t.Errorf("Failed at repository %s", ownerRepo)
+			log.Printf("Failed at repository %s\n", ownerRepo)
 			return
 		}
 		defer resp.Body.Close()
 		json.NewDecoder(resp.Body).Decode(&respObj)
 		isAged = len(respObj) == 0
 		if isAged {
-			body := &Issue{
-				Title: fmt.Sprintf(ISSUEFMT, ownerRepo),
+			body := &issue{
+				Title: fmt.Sprintf(issueFmt, ownerRepo),
 			}
 			buf := new(bytes.Buffer)
 			json.NewEncoder(buf).Encode(body)
 			req, err := http.NewRequest("POST", githubPOSTISSUES, buf)
 			if err != nil {
-				t.Errorf("Failed at repository %s", ownerRepo)
+				log.Printf("Failed at repository %s\n", ownerRepo)
 				return
 			}
 			*issueRequests = append(*issueRequests, req)
 		}
 	}
 }
-func TestStaleRepository(t *testing.T) {
+func testStaleRepository() {
 	query := startQuery()
 	var issueRequests []*http.Request
 	oauth := os.Getenv("GITHUB_OAUTH_TOKEN")
-	tokenSource := &TokenSource{
+	tokenSource := &tokenSource{
 		AccessToken: oauth,
 	}
 	oauthClient := oauth2.NewClient(oauth2.NoContext, tokenSource)
 	query.Find("body li > a:first-child").Each(func(_ int, s *goquery.Selection) {
 		href, ok := s.Attr("href")
 		if !ok {
-			t.Error("expected to have href")
+			log.Println("expected to have href")
 		}
-		testCommitAge(href, oauthClient, &issueRequests, t)
+		testCommitAge(href, oauthClient, &issueRequests)
 	})
-	createIssues(issueRequests, oauthClient, t)
+	createIssues(issueRequests, oauthClient)
+}
+
+func main() {
+	testStaleRepository()
 }