[CHORE] Add extra testing

- Do more property testing for null values.
- Add test for `AbsoluteLong`.
This commit is contained in:
Gusted 2024-11-05 21:20:55 +01:00
parent fddde93759
commit 2f8e7a2c67
No known key found for this signature in database
GPG Key ID: FD821B732837125F
2 changed files with 16 additions and 6 deletions

View File

@ -27,7 +27,7 @@ func (du *DateUtils) AbsoluteShort(time any) template.HTML {
// AbsoluteLong renders in "January 01, 2006" format
func (du *DateUtils) AbsoluteLong(time any) template.HTML {
return timeutil.DateTime("short", time)
return timeutil.DateTime("long", time)
}
// FullTime renders in "Jan 01, 2006 20:33:44" format

View File

@ -4,6 +4,7 @@
package templates
import (
"html/template"
"testing"
"time"
@ -12,10 +13,12 @@ import (
"code.gitea.io/gitea/modules/timeutil"
"github.com/stretchr/testify/assert"
"github.com/stretchr/testify/require"
)
func TestDateTime(t *testing.T) {
testTz, _ := time.LoadLocation("America/New_York")
testTz, err := time.LoadLocation("America/New_York")
require.NoError(t, err)
defer test.MockVariableValue(&setting.DefaultUILocation, testTz)()
defer test.MockVariableValue(&setting.IsInTesting, false)()
@ -26,10 +29,11 @@ func TestDateTime(t *testing.T) {
refTime, _ := time.Parse(time.RFC3339, refTimeStr)
refTimeStamp := timeutil.TimeStamp(refTime.Unix())
assert.EqualValues(t, "-", du.AbsoluteShort(nil))
assert.EqualValues(t, "-", du.AbsoluteShort(0))
assert.EqualValues(t, "-", du.AbsoluteShort(time.Time{}))
assert.EqualValues(t, "-", du.AbsoluteShort(timeutil.TimeStamp(0)))
for _, val := range []any{nil, 0, time.Time{}, timeutil.TimeStamp(0)} {
for _, fun := range []func(val any) template.HTML{du.AbsoluteLong, du.AbsoluteShort, du.FullTime} {
assert.EqualValues(t, "-", fun(val))
}
}
actual := dateTimeLegacy("short", "invalid")
assert.EqualValues(t, `-`, actual)
@ -40,12 +44,18 @@ func TestDateTime(t *testing.T) {
actual = du.AbsoluteShort(refTime)
assert.EqualValues(t, `<absolute-date weekday="" year="numeric" month="short" day="numeric" date="2018-01-01T00:00:00Z">2018-01-01</absolute-date>`, actual)
actual = du.AbsoluteLong(refTime)
assert.EqualValues(t, `<absolute-date weekday="" year="numeric" month="long" day="numeric" date="2018-01-01T00:00:00Z">2018-01-01</absolute-date>`, actual)
actual = dateTimeLegacy("short", refDateStr)
assert.EqualValues(t, `<absolute-date weekday="" year="numeric" month="short" day="numeric" date="2018-01-01T00:00:00-05:00">2018-01-01</absolute-date>`, actual)
actual = du.AbsoluteShort(refTimeStamp)
assert.EqualValues(t, `<absolute-date weekday="" year="numeric" month="short" day="numeric" date="2017-12-31T19:00:00-05:00">2017-12-31</absolute-date>`, actual)
actual = du.AbsoluteLong(refTimeStamp)
assert.EqualValues(t, `<absolute-date weekday="" year="numeric" month="long" day="numeric" date="2017-12-31T19:00:00-05:00">2017-12-31</absolute-date>`, actual)
actual = du.FullTime(refTimeStamp)
assert.EqualValues(t, `<relative-time weekday="" year="numeric" format="datetime" month="short" day="numeric" hour="numeric" minute="numeric" second="numeric" data-tooltip-content data-tooltip-interactive="true" datetime="2017-12-31T19:00:00-05:00">2017-12-31 19:00:00 -05:00</relative-time>`, actual)
}