yesBad/proton-bridge
1
0
Fork 0
forked from Silverfish/proton-bridge
Code Actions Activity

fix(GODT-3187): Fix numberOfDay computation when changing date.

Browse Source
This commit is contained in:
Romain LE JEUNE
2024-01-02 15:24:01 +01:00
parent 25ebcffde3
commit 43df20c25d
2 changed files with 1 additions and 33 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
internal/configstatus/configuration_progress.go
View File

@ -52,10 +52,7 @@ func numberOfDay(now, prev time.Time) int {
return 1 return 1
} }
if now.Year() > prev.Year() { if now.Year() > prev.Year() {
if now.YearDay() > prev.YearDay() { return (365 * (now.Year() - prev.Year())) + now.YearDay() - prev.YearDay()
return 365 + (now.YearDay() - prev.YearDay()) + (365 * (now.Year() - prev.Year() - 1))
}
return (365 - prev.YearDay()) + now.YearDay() + (365 * (now.Year() - prev.Year() - 1))
} else if now.YearDay() > prev.YearDay() { } else if now.YearDay() > prev.YearDay() {
return now.YearDay() - prev.YearDay() return now.YearDay() - prev.YearDay()
} }

29
internal/configstatus/configuration_progress_test.go
View File

@ -98,32 +98,3 @@ func TestConfigurationProgress_fed_year_change(t *testing.T) {
require.Equal(t, 370, req.Values.NbDay) require.Equal(t, 370, req.Values.NbDay)
require.Equal(t, 2, req.Values.NbDaySinceLast) require.Equal(t, 2, req.Values.NbDaySinceLast)
} }
func TestConfigurationProgress_fed_day_higher(t *testing.T) {
dir := t.TempDir()
file := filepath.Join(dir, "dummy.json")
var data = configstatus.ConfigurationStatusData{
Metadata: configstatus.Metadata{Version: "1.0.0"},
DataV1: configstatus.DataV1{
PendingSince: time.Now().AddDate(-1, 0, -5),
LastProgress: time.Now().AddDate(0, 0, -2),
Autoconf: "Mr TBird",
ClickedLink: 42,
ReportSent: false,
ReportClick: true,
FailureDetails: "Not an error",
},
}
require.NoError(t, dumpConfigStatusInFile(&data, file))
config, err := configstatus.LoadConfigurationStatus(file)
require.NoError(t, err)
var builder = configstatus.ConfigProgressBuilder{}
req := builder.New(config)
require.Equal(t, "bridge.any.configuration", req.MeasurementGroup)
require.Equal(t, "bridge_config_progress", req.Event)
require.Equal(t, 370, req.Values.NbDay)
require.Equal(t, 2, req.Values.NbDaySinceLast)
}