summaryrefslogtreecommitdiffstats
path: root/models/issues
diff options
context:
space:
mode:
author6543 <6543@obermui.de>2023-09-29 14:04:14 +0100
committerGitHub <noreply@github.com>2023-09-29 13:04:14 +0000
commit2a2b0cb1d2f7de9dcdb8e84250bfb9ef501fd748 (patch)
tree1c5d051eb9c7a3472ad00ab79d9674d5c4f5f57b /models/issues
parentcf0df023be06c8acec4fc8fb05eab1d1c2e52fd1 (diff)
downloadgitea-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.go4
-rw-r--r--models/issues/tracked_time_test.go10
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)
}