diff options
author | 6543 <6543@obermui.de> | 2023-09-29 14:04:14 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-09-29 13:04:14 +0000 |
commit | 2a2b0cb1d2f7de9dcdb8e84250bfb9ef501fd748 (patch) | |
tree | 1c5d051eb9c7a3472ad00ab79d9674d5c4f5f57b /models/issues | |
parent | cf0df023be06c8acec4fc8fb05eab1d1c2e52fd1 (diff) | |
download | gitea-2a2b0cb1d2f7de9dcdb8e84250bfb9ef501fd748.tar.gz gitea-2a2b0cb1d2f7de9dcdb8e84250bfb9ef501fd748.zip |
rename TotalTimes() to TotalTimesForEachUser() (#27261)
make function name more descriptive as it would otherwise imply it could
be used for e.g. #26672 too ...
---
*Sponsored by Kithara Software GmbH*
Diffstat (limited to 'models/issues')
-rw-r--r-- | models/issues/tracked_time.go | 4 | ||||
-rw-r--r-- | models/issues/tracked_time_test.go | 10 |
2 files changed, 7 insertions, 7 deletions
diff --git a/models/issues/tracked_time.go b/models/issues/tracked_time.go index 58c6b775f0..c51df627ac 100644 --- a/models/issues/tracked_time.go +++ b/models/issues/tracked_time.go @@ -198,8 +198,8 @@ func addTime(ctx context.Context, user *user_model.User, issue *Issue, amount in return tt, db.Insert(ctx, tt) } -// TotalTimes returns the spent time in seconds for each user by an issue -func TotalTimes(options *FindTrackedTimesOptions) (map[*user_model.User]int64, error) { +// TotalTimesForEachUser returns the spent time in seconds for each user by an issue +func TotalTimesForEachUser(options *FindTrackedTimesOptions) (map[*user_model.User]int64, error) { trackedTimes, err := GetTrackedTimes(db.DefaultContext, options) if err != nil { return nil, err diff --git a/models/issues/tracked_time_test.go b/models/issues/tracked_time_test.go index caced78c9e..24081f1bfa 100644 --- a/models/issues/tracked_time_test.go +++ b/models/issues/tracked_time_test.go @@ -79,10 +79,10 @@ func TestGetTrackedTimes(t *testing.T) { assert.Len(t, times, 0) } -func TestTotalTimes(t *testing.T) { +func TestTotalTimesForEachUser(t *testing.T) { assert.NoError(t, unittest.PrepareTestDatabase()) - total, err := issues_model.TotalTimes(&issues_model.FindTrackedTimesOptions{IssueID: 1}) + total, err := issues_model.TotalTimesForEachUser(&issues_model.FindTrackedTimesOptions{IssueID: 1}) assert.NoError(t, err) assert.Len(t, total, 1) for user, time := range total { @@ -90,7 +90,7 @@ func TestTotalTimes(t *testing.T) { assert.EqualValues(t, 400, time) } - total, err = issues_model.TotalTimes(&issues_model.FindTrackedTimesOptions{IssueID: 2}) + total, err = issues_model.TotalTimesForEachUser(&issues_model.FindTrackedTimesOptions{IssueID: 2}) assert.NoError(t, err) assert.Len(t, total, 2) for user, time := range total { @@ -103,7 +103,7 @@ func TestTotalTimes(t *testing.T) { } } - total, err = issues_model.TotalTimes(&issues_model.FindTrackedTimesOptions{IssueID: 5}) + total, err = issues_model.TotalTimesForEachUser(&issues_model.FindTrackedTimesOptions{IssueID: 5}) assert.NoError(t, err) assert.Len(t, total, 1) for user, time := range total { @@ -111,7 +111,7 @@ func TestTotalTimes(t *testing.T) { assert.EqualValues(t, 1, time) } - total, err = issues_model.TotalTimes(&issues_model.FindTrackedTimesOptions{IssueID: 4}) + total, err = issues_model.TotalTimesForEachUser(&issues_model.FindTrackedTimesOptions{IssueID: 4}) assert.NoError(t, err) assert.Len(t, total, 2) } |