STACKITGit/cli
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Notify new releases only once per day

Browse source
This commit is contained in:
Cristian Dominguez 2020-11-26 11:00:27 -03:00 • committed by Mislav Marohnić
parent cee4f853dc
commit 0ec8c2e9ed
2 changed files with 29 additions and 9 deletions
Download patch file Download diff file Expand all files Collapse all files

21
update/update.go
View file

@ -24,22 +24,26 @@ type StateEntry struct {
// CheckForUpdate checks whether this software has had a newer release on GitHub
func CheckForUpdate(client *api.Client, stateFilePath, repo, currentVersion string) (*ReleaseInfo, error) {
latestRelease, err := getLatestReleaseInfo(client, stateFilePath, repo, currentVersion)
releaseInfo, err := getLatestReleaseInfo(client, stateFilePath, repo)
if err != nil {
return nil, err
}
if versionGreaterThan(latestRelease.Version, currentVersion) {
return latestRelease, nil
if releaseInfo == nil {
return nil, nil
}
if versionGreaterThan(releaseInfo.LatestRelease.Version, currentVersion) {
return &releaseInfo.LatestRelease, nil
}
return nil, nil
}
func getLatestReleaseInfo(client *api.Client, stateFilePath, repo, currentVersion string) (*ReleaseInfo, error) {
func getLatestReleaseInfo(client *api.Client, stateFilePath, repo string) (*StateEntry, error) {
stateEntry, err := getStateEntry(stateFilePath)
if err == nil && time.Since(stateEntry.CheckedForUpdateAt).Hours() < 24 {
return &stateEntry.LatestRelease, nil
return nil, nil
}
var latestRelease ReleaseInfo
@ -53,7 +57,12 @@ func getLatestReleaseInfo(client *api.Client, stateFilePath, repo, currentVersio
return nil, err
}
return &latestRelease, nil
stateEntry, err = getStateEntry(stateFilePath)
if err != nil {
return nil, err
}
return stateEntry, nil
}
func getStateEntry(stateFilePath string) (*StateEntry, error) {

17
update/update_test.go
View file

@ -52,6 +52,9 @@ func TestCheckForUpdate(t *testing.T) {
for _, s := range scenarios {
t.Run(s.Name, func(t *testing.T) {
stateFilePath := tempFilePath()
defer os.Remove(stateFilePath)
http := &httpmock.Registry{}
client := api.NewClient(api.ReplaceTripper(http))
http.StubResponse(200, bytes.NewBufferString(fmt.Sprintf(`{
@ -59,7 +62,7 @@ func TestCheckForUpdate(t *testing.T) {
"html_url": "%s"
}`, s.LatestVersion, s.LatestURL)))
rel, err := CheckForUpdate(client, tempFilePath(), "OWNER/REPO", s.CurrentVersion)
rel, err := CheckForUpdate(client, stateFilePath, "OWNER/REPO", s.CurrentVersion)
if err != nil {
t.Fatal(err)
}
@ -93,10 +96,18 @@ func TestCheckForUpdate(t *testing.T) {
}
func tempFilePath() string {
file, err := ioutil.TempFile("", "")
content := []byte(
`checked_for_update_at: 2020-11-22T09:53:32.609610344-03:00
latest_release:
version: v1.1.0
url: https://github.com/cli/cli/releases/tag/v1.1.0`)
file, err := ioutil.TempFile("", "state*.yml")
if err != nil {
log.Fatal(err)
}
os.Remove(file.Name())
if _, err := file.Write(content); err != nil {
log.Fatal(err)
}
return file.Name()
}