Skip to content

Commit 9f23c08

Browse files
pitdickerpitdicker
committed
Tests: replace TimeDelta::days with try_days 
1 parent e8f9b5e commit 9f23c08

File tree

10 files changed

+175
-122
lines changed

10 files changed

+175
-122
lines changed

‎src/datetime/tests.rs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1500,8 +1500,8 @@ fn test_datetime_add_assign_local() {
15001500

15011501
// ensure we cross a DST transition
15021502
for i in 1..=365 {
1503-
datetime_add += TimeDelta::days(1);
1504-
assert_eq!(datetime_add, datetime + TimeDelta::days(i))
1503+
datetime_add += TimeDelta::try_days(1).unwrap();
1504+
assert_eq!(datetime_add, datetime + TimeDelta::try_days(i).unwrap())
15051505
}
15061506
}
15071507

@@ -1709,8 +1709,8 @@ fn test_datetime_sub_assign_local() {
17091709

17101710
// ensure we cross a DST transition
17111711
for i in 1..=365 {
1712-
datetime_sub -= TimeDelta::days(1);
1713-
assert_eq!(datetime_sub, datetime - TimeDelta::days(i))
1712+
datetime_sub -= TimeDelta::try_days(1).unwrap();
1713+
assert_eq!(datetime_sub, datetime - TimeDelta::try_days(i).unwrap())
17141714
}
17151715
}
17161716

‎src/naive/date/mod.rs

Lines changed: 65 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -1057,16 +1057,16 @@ impl NaiveDate {
10571057
///
10581058
/// let d = NaiveDate::from_ymd_opt(2015, 9, 5).unwrap();
10591059
/// assert_eq!(
1060-
/// d.checked_add_signed(TimeDelta::days(40)),
1060+
/// d.checked_add_signed(TimeDelta::try_days(40).unwrap()),
10611061
/// Some(NaiveDate::from_ymd_opt(2015, 10, 15).unwrap())
10621062
/// );
10631063
/// assert_eq!(
1064-
/// d.checked_add_signed(TimeDelta::days(-40)),
1064+
/// d.checked_add_signed(TimeDelta::try_days(-40).unwrap()),
10651065
/// Some(NaiveDate::from_ymd_opt(2015, 7, 27).unwrap())
10661066
/// );
1067-
/// assert_eq!(d.checked_add_signed(TimeDelta::days(1_000_000_000)), None);
1068-
/// assert_eq!(d.checked_add_signed(TimeDelta::days(-1_000_000_000)), None);
1069-
/// assert_eq!(NaiveDate::MAX.checked_add_signed(TimeDelta::days(1)), None);
1067+
/// assert_eq!(d.checked_add_signed(TimeDelta::try_days(1_000_000_000).unwrap()), None);
1068+
/// assert_eq!(d.checked_add_signed(TimeDelta::try_days(-1_000_000_000).unwrap()), None);
1069+
/// assert_eq!(NaiveDate::MAX.checked_add_signed(TimeDelta::try_days(1).unwrap()), None);
10701070
/// ```
10711071
#[must_use]
10721072
pub const fn checked_add_signed(self, rhs: TimeDelta) -> Option<NaiveDate> {
@@ -1090,16 +1090,16 @@ impl NaiveDate {
10901090
///
10911091
/// let d = NaiveDate::from_ymd_opt(2015, 9, 5).unwrap();
10921092
/// assert_eq!(
1093-
/// d.checked_sub_signed(TimeDelta::days(40)),
1093+
/// d.checked_sub_signed(TimeDelta::try_days(40).unwrap()),
10941094
/// Some(NaiveDate::from_ymd_opt(2015, 7, 27).unwrap())
10951095
/// );
10961096
/// assert_eq!(
1097-
/// d.checked_sub_signed(TimeDelta::days(-40)),
1097+
/// d.checked_sub_signed(TimeDelta::try_days(-40).unwrap()),
10981098
/// Some(NaiveDate::from_ymd_opt(2015, 10, 15).unwrap())
10991099
/// );
1100-
/// assert_eq!(d.checked_sub_signed(TimeDelta::days(1_000_000_000)), None);
1101-
/// assert_eq!(d.checked_sub_signed(TimeDelta::days(-1_000_000_000)), None);
1102-
/// assert_eq!(NaiveDate::MIN.checked_sub_signed(TimeDelta::days(1)), None);
1100+
/// assert_eq!(d.checked_sub_signed(TimeDelta::try_days(1_000_000_000).unwrap()), None);
1101+
/// assert_eq!(d.checked_sub_signed(TimeDelta::try_days(-1_000_000_000).unwrap()), None);
1102+
/// assert_eq!(NaiveDate::MIN.checked_sub_signed(TimeDelta::try_days(1).unwrap()), None);
11031103
/// ```
11041104
#[must_use]
11051105
pub const fn checked_sub_signed(self, rhs: TimeDelta) -> Option<NaiveDate> {
@@ -1125,12 +1125,27 @@ impl NaiveDate {
11251125
/// let since = NaiveDate::signed_duration_since;
11261126
///
11271127
/// assert_eq!(since(from_ymd(2014, 1, 1), from_ymd(2014, 1, 1)), TimeDelta::zero());
1128-
/// assert_eq!(since(from_ymd(2014, 1, 1), from_ymd(2013, 12, 31)), TimeDelta::days(1));
1129-
/// assert_eq!(since(from_ymd(2014, 1, 1), from_ymd(2014, 1, 2)), TimeDelta::days(-1));
1130-
/// assert_eq!(since(from_ymd(2014, 1, 1), from_ymd(2013, 9, 23)), TimeDelta::days(100));
1131-
/// assert_eq!(since(from_ymd(2014, 1, 1), from_ymd(2013, 1, 1)), TimeDelta::days(365));
1132-
/// assert_eq!(since(from_ymd(2014, 1, 1), from_ymd(2010, 1, 1)), TimeDelta::days(365 * 4 + 1));
1133-
/// assert_eq!(since(from_ymd(2014, 1, 1), from_ymd(1614, 1, 1)), TimeDelta::days(365 * 400 + 97));
1128+
/// assert_eq!(
1129+
/// since(from_ymd(2014, 1, 1), from_ymd(2013, 12, 31)),
1130+
/// TimeDelta::try_days(1).unwrap()
1131+
/// );
1132+
/// assert_eq!(since(from_ymd(2014, 1, 1), from_ymd(2014, 1, 2)), TimeDelta::try_days(-1).unwrap());
1133+
/// assert_eq!(
1134+
/// since(from_ymd(2014, 1, 1), from_ymd(2013, 9, 23)),
1135+
/// TimeDelta::try_days(100).unwrap()
1136+
/// );
1137+
/// assert_eq!(
1138+
/// since(from_ymd(2014, 1, 1), from_ymd(2013, 1, 1)),
1139+
/// TimeDelta::try_days(365).unwrap()
1140+
/// );
1141+
/// assert_eq!(
1142+
/// since(from_ymd(2014, 1, 1), from_ymd(2010, 1, 1)),
1143+
/// TimeDelta::try_days(365 * 4 + 1).unwrap()
1144+
/// );
1145+
/// assert_eq!(
1146+
/// since(from_ymd(2014, 1, 1), from_ymd(1614, 1, 1)),
1147+
/// TimeDelta::try_days(365 * 400 + 97).unwrap()
1148+
/// );
11341149
/// ```
11351150
#[must_use]
11361151
pub const fn signed_duration_since(self, rhs: NaiveDate) -> TimeDelta {
@@ -1894,11 +1909,17 @@ impl Datelike for NaiveDate {
18941909
/// assert_eq!(from_ymd(2014, 1, 1) + TimeDelta::zero(), from_ymd(2014, 1, 1));
18951910
/// assert_eq!(from_ymd(2014, 1, 1) + TimeDelta::seconds(86399), from_ymd(2014, 1, 1));
18961911
/// assert_eq!(from_ymd(2014, 1, 1) + TimeDelta::seconds(-86399), from_ymd(2014, 1, 1));
1897-
/// assert_eq!(from_ymd(2014, 1, 1) + TimeDelta::days(1), from_ymd(2014, 1, 2));
1898-
/// assert_eq!(from_ymd(2014, 1, 1) + TimeDelta::days(-1), from_ymd(2013, 12, 31));
1899-
/// assert_eq!(from_ymd(2014, 1, 1) + TimeDelta::days(364), from_ymd(2014, 12, 31));
1900-
/// assert_eq!(from_ymd(2014, 1, 1) + TimeDelta::days(365 * 4 + 1), from_ymd(2018, 1, 1));
1901-
/// assert_eq!(from_ymd(2014, 1, 1) + TimeDelta::days(365 * 400 + 97), from_ymd(2414, 1, 1));
1912+
/// assert_eq!(from_ymd(2014, 1, 1) + TimeDelta::try_days(1).unwrap(), from_ymd(2014, 1, 2));
1913+
/// assert_eq!(from_ymd(2014, 1, 1) + TimeDelta::try_days(-1).unwrap(), from_ymd(2013, 12, 31));
1914+
/// assert_eq!(from_ymd(2014, 1, 1) + TimeDelta::try_days(364).unwrap(), from_ymd(2014, 12, 31));
1915+
/// assert_eq!(
1916+
/// from_ymd(2014, 1, 1) + TimeDelta::try_days(365 * 4 + 1).unwrap(),
1917+
/// from_ymd(2018, 1, 1)
1918+
/// );
1919+
/// assert_eq!(
1920+
/// from_ymd(2014, 1, 1) + TimeDelta::try_days(365 * 400 + 97).unwrap(),
1921+
/// from_ymd(2414, 1, 1)
1922+
/// );
19021923
/// ```
19031924
///
19041925
/// [`NaiveDate::checked_add_signed`]: crate::NaiveDate::checked_add_signed
@@ -2037,11 +2058,17 @@ impl Sub<Days> for NaiveDate {
20372058
/// assert_eq!(from_ymd(2014, 1, 1) - TimeDelta::zero(), from_ymd(2014, 1, 1));
20382059
/// assert_eq!(from_ymd(2014, 1, 1) - TimeDelta::seconds(86399), from_ymd(2014, 1, 1));
20392060
/// assert_eq!(from_ymd(2014, 1, 1) - TimeDelta::seconds(-86399), from_ymd(2014, 1, 1));
2040-
/// assert_eq!(from_ymd(2014, 1, 1) - TimeDelta::days(1), from_ymd(2013, 12, 31));
2041-
/// assert_eq!(from_ymd(2014, 1, 1) - TimeDelta::days(-1), from_ymd(2014, 1, 2));
2042-
/// assert_eq!(from_ymd(2014, 1, 1) - TimeDelta::days(364), from_ymd(2013, 1, 2));
2043-
/// assert_eq!(from_ymd(2014, 1, 1) - TimeDelta::days(365 * 4 + 1), from_ymd(2010, 1, 1));
2044-
/// assert_eq!(from_ymd(2014, 1, 1) - TimeDelta::days(365 * 400 + 97), from_ymd(1614, 1, 1));
2061+
/// assert_eq!(from_ymd(2014, 1, 1) - TimeDelta::try_days(1).unwrap(), from_ymd(2013, 12, 31));
2062+
/// assert_eq!(from_ymd(2014, 1, 1) - TimeDelta::try_days(-1).unwrap(), from_ymd(2014, 1, 2));
2063+
/// assert_eq!(from_ymd(2014, 1, 1) - TimeDelta::try_days(364).unwrap(), from_ymd(2013, 1, 2));
2064+
/// assert_eq!(
2065+
/// from_ymd(2014, 1, 1) - TimeDelta::try_days(365 * 4 + 1).unwrap(),
2066+
/// from_ymd(2010, 1, 1)
2067+
/// );
2068+
/// assert_eq!(
2069+
/// from_ymd(2014, 1, 1) - TimeDelta::try_days(365 * 400 + 97).unwrap(),
2070+
/// from_ymd(1614, 1, 1)
2071+
/// );
20452072
/// ```
20462073
///
20472074
/// [`NaiveDate::checked_sub_signed`]: crate::NaiveDate::checked_sub_signed
@@ -2088,12 +2115,18 @@ impl SubAssign<TimeDelta> for NaiveDate {
20882115
/// let from_ymd = |y, m, d| NaiveDate::from_ymd_opt(y, m, d).unwrap();
20892116
///
20902117
/// assert_eq!(from_ymd(2014, 1, 1) - from_ymd(2014, 1, 1), TimeDelta::zero());
2091-
/// assert_eq!(from_ymd(2014, 1, 1) - from_ymd(2013, 12, 31), TimeDelta::days(1));
2092-
/// assert_eq!(from_ymd(2014, 1, 1) - from_ymd(2014, 1, 2), TimeDelta::days(-1));
2093-
/// assert_eq!(from_ymd(2014, 1, 1) - from_ymd(2013, 9, 23), TimeDelta::days(100));
2094-
/// assert_eq!(from_ymd(2014, 1, 1) - from_ymd(2013, 1, 1), TimeDelta::days(365));
2095-
/// assert_eq!(from_ymd(2014, 1, 1) - from_ymd(2010, 1, 1), TimeDelta::days(365 * 4 + 1));
2096-
/// assert_eq!(from_ymd(2014, 1, 1) - from_ymd(1614, 1, 1), TimeDelta::days(365 * 400 + 97));
2118+
/// assert_eq!(from_ymd(2014, 1, 1) - from_ymd(2013, 12, 31), TimeDelta::try_days(1).unwrap());
2119+
/// assert_eq!(from_ymd(2014, 1, 1) - from_ymd(2014, 1, 2), TimeDelta::try_days(-1).unwrap());
2120+
/// assert_eq!(from_ymd(2014, 1, 1) - from_ymd(2013, 9, 23), TimeDelta::try_days(100).unwrap());
2121+
/// assert_eq!(from_ymd(2014, 1, 1) - from_ymd(2013, 1, 1), TimeDelta::try_days(365).unwrap());
2122+
/// assert_eq!(
2123+
/// from_ymd(2014, 1, 1) - from_ymd(2010, 1, 1),
2124+
/// TimeDelta::try_days(365 * 4 + 1).unwrap()
2125+
/// );
2126+
/// assert_eq!(
2127+
/// from_ymd(2014, 1, 1) - from_ymd(1614, 1, 1),
2128+
/// TimeDelta::try_days(365 * 400 + 97).unwrap()
2129+
/// );
20972130
/// ```
20982131
impl Sub<NaiveDate> for NaiveDate {
20992132
type Output = TimeDelta;

‎src/naive/date/tests.rs

Lines changed: 29 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -457,20 +457,28 @@ fn test_date_add() {
457457
check((2014, 1, 1), TimeDelta::seconds(86399), Some((2014, 1, 1)));
458458
// always round towards zero
459459
check((2014, 1, 1), TimeDelta::seconds(-86399), Some((2014, 1, 1)));
460-
check((2014, 1, 1), TimeDelta::days(1), Some((2014, 1, 2)));
461-
check((2014, 1, 1), TimeDelta::days(-1), Some((2013, 12, 31)));
462-
check((2014, 1, 1), TimeDelta::days(364), Some((2014, 12, 31)));
463-
check((2014, 1, 1), TimeDelta::days(365 * 4 + 1), Some((2018, 1, 1)));
464-
check((2014, 1, 1), TimeDelta::days(365 * 400 + 97), Some((2414, 1, 1)));
460+
check((2014, 1, 1), TimeDelta::try_days(1).unwrap(), Some((2014, 1, 2)));
461+
check((2014, 1, 1), TimeDelta::try_days(-1).unwrap(), Some((2013, 12, 31)));
462+
check((2014, 1, 1), TimeDelta::try_days(364).unwrap(), Some((2014, 12, 31)));
463+
check((2014, 1, 1), TimeDelta::try_days(365 * 4 + 1).unwrap(), Some((2018, 1, 1)));
464+
check((2014, 1, 1), TimeDelta::try_days(365 * 400 + 97).unwrap(), Some((2414, 1, 1)));
465465

466-
check((-7, 1, 1), TimeDelta::days(365 * 12 + 3), Some((5, 1, 1)));
466+
check((-7, 1, 1), TimeDelta::try_days(365 * 12 + 3).unwrap(), Some((5, 1, 1)));
467467

468468
// overflow check
469-
check((0, 1, 1), TimeDelta::days(MAX_DAYS_FROM_YEAR_0 as i64), Some((MAX_YEAR, 12, 31)));
470-
check((0, 1, 1), TimeDelta::days(MAX_DAYS_FROM_YEAR_0 as i64 + 1), None);
469+
check(
470+
(0, 1, 1),
471+
TimeDelta::try_days(MAX_DAYS_FROM_YEAR_0 as i64).unwrap(),
472+
Some((MAX_YEAR, 12, 31)),
473+
);
474+
check((0, 1, 1), TimeDelta::try_days(MAX_DAYS_FROM_YEAR_0 as i64 + 1).unwrap(), None);
471475
check((0, 1, 1), TimeDelta::max_value(), None);
472-
check((0, 1, 1), TimeDelta::days(MIN_DAYS_FROM_YEAR_0 as i64), Some((MIN_YEAR, 1, 1)));
473-
check((0, 1, 1), TimeDelta::days(MIN_DAYS_FROM_YEAR_0 as i64 - 1), None);
476+
check(
477+
(0, 1, 1),
478+
TimeDelta::try_days(MIN_DAYS_FROM_YEAR_0 as i64).unwrap(),
479+
Some((MIN_YEAR, 1, 1)),
480+
);
481+
check((0, 1, 1), TimeDelta::try_days(MIN_DAYS_FROM_YEAR_0 as i64 - 1).unwrap(), None);
474482
check((0, 1, 1), TimeDelta::min_value(), None);
475483
}
476484

@@ -484,14 +492,14 @@ fn test_date_sub() {
484492
}
485493

486494
check((2014, 1, 1), (2014, 1, 1), TimeDelta::zero());
487-
check((2014, 1, 2), (2014, 1, 1), TimeDelta::days(1));
488-
check((2014, 12, 31), (2014, 1, 1), TimeDelta::days(364));
489-
check((2015, 1, 3), (2014, 1, 1), TimeDelta::days(365 + 2));
490-
check((2018, 1, 1), (2014, 1, 1), TimeDelta::days(365 * 4 + 1));
491-
check((2414, 1, 1), (2014, 1, 1), TimeDelta::days(365 * 400 + 97));
495+
check((2014, 1, 2), (2014, 1, 1), TimeDelta::try_days(1).unwrap());
496+
check((2014, 12, 31), (2014, 1, 1), TimeDelta::try_days(364).unwrap());
497+
check((2015, 1, 3), (2014, 1, 1), TimeDelta::try_days(365 + 2).unwrap());
498+
check((2018, 1, 1), (2014, 1, 1), TimeDelta::try_days(365 * 4 + 1).unwrap());
499+
check((2414, 1, 1), (2014, 1, 1), TimeDelta::try_days(365 * 400 + 97).unwrap());
492500

493-
check((MAX_YEAR, 12, 31), (0, 1, 1), TimeDelta::days(MAX_DAYS_FROM_YEAR_0 as i64));
494-
check((MIN_YEAR, 1, 1), (0, 1, 1), TimeDelta::days(MIN_DAYS_FROM_YEAR_0 as i64));
501+
check((MAX_YEAR, 12, 31), (0, 1, 1), TimeDelta::try_days(MAX_DAYS_FROM_YEAR_0 as i64).unwrap());
502+
check((MIN_YEAR, 1, 1), (0, 1, 1), TimeDelta::try_days(MIN_DAYS_FROM_YEAR_0 as i64).unwrap());
495503
}
496504

497505
#[test]
@@ -539,19 +547,19 @@ fn test_date_sub_days() {
539547
fn test_date_addassignment() {
540548
let ymd = |y, m, d| NaiveDate::from_ymd_opt(y, m, d).unwrap();
541549
let mut date = ymd(2016, 10, 1);
542-
date += TimeDelta::days(10);
550+
date += TimeDelta::try_days(10).unwrap();
543551
assert_eq!(date, ymd(2016, 10, 11));
544-
date += TimeDelta::days(30);
552+
date += TimeDelta::try_days(30).unwrap();
545553
assert_eq!(date, ymd(2016, 11, 10));
546554
}
547555

548556
#[test]
549557
fn test_date_subassignment() {
550558
let ymd = |y, m, d| NaiveDate::from_ymd_opt(y, m, d).unwrap();
551559
let mut date = ymd(2016, 10, 11);
552-
date -= TimeDelta::days(10);
560+
date -= TimeDelta::try_days(10).unwrap();
553561
assert_eq!(date, ymd(2016, 10, 1));
554-
date -= TimeDelta::days(2);
562+
date -= TimeDelta::try_days(2).unwrap();
555563
assert_eq!(date, ymd(2016, 9, 29));
556564
}
557565

‎src/naive/datetime/mod.rs

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -491,7 +491,7 @@ impl NaiveDateTime {
491491
/// ```
492492
/// # use chrono::{TimeDelta, NaiveDate};
493493
/// # let hms = |h, m, s| NaiveDate::from_ymd_opt(2016, 7, 8).unwrap().and_hms_opt(h, m, s).unwrap();
494-
/// assert_eq!(hms(3, 5, 7).checked_add_signed(TimeDelta::days(1_000_000_000)), None);
494+
/// assert_eq!(hms(3, 5, 7).checked_add_signed(TimeDelta::try_days(1_000_000_000).unwrap()), None);
495495
/// ```
496496
///
497497
/// Leap seconds are handled,
@@ -514,7 +514,7 @@ impl NaiveDateTime {
514514
/// Some(hmsm(3, 6, 9, 300)));
515515
/// assert_eq!(leap.checked_add_signed(TimeDelta::seconds(-10)),
516516
/// Some(hmsm(3, 5, 50, 300)));
517-
/// assert_eq!(leap.checked_add_signed(TimeDelta::days(1)),
517+
/// assert_eq!(leap.checked_add_signed(TimeDelta::try_days(1).unwrap()),
518518
/// Some(from_ymd(2016, 7, 9).and_hms_milli_opt(3, 5, 59, 300).unwrap()));
519519
/// ```
520520
#[must_use]
@@ -666,7 +666,7 @@ impl NaiveDateTime {
666666
/// ```
667667
/// # use chrono::{TimeDelta, NaiveDate};
668668
/// # let hms = |h, m, s| NaiveDate::from_ymd_opt(2016, 7, 8).unwrap().and_hms_opt(h, m, s).unwrap();
669-
/// assert_eq!(hms(3, 5, 7).checked_sub_signed(TimeDelta::days(1_000_000_000)), None);
669+
/// assert_eq!(hms(3, 5, 7).checked_sub_signed(TimeDelta::try_days(1_000_000_000).unwrap()), None);
670670
/// ```
671671
///
672672
/// Leap seconds are handled,
@@ -685,7 +685,7 @@ impl NaiveDateTime {
685685
/// Some(hmsm(3, 5, 59, 800)));
686686
/// assert_eq!(leap.checked_sub_signed(TimeDelta::seconds(60)),
687687
/// Some(hmsm(3, 5, 0, 300)));
688-
/// assert_eq!(leap.checked_sub_signed(TimeDelta::days(1)),
688+
/// assert_eq!(leap.checked_sub_signed(TimeDelta::try_days(1).unwrap()),
689689
/// Some(from_ymd(2016, 7, 7).and_hms_milli_opt(3, 6, 0, 300).unwrap()));
690690
/// ```
691691
#[must_use]
@@ -1586,7 +1586,7 @@ impl Timelike for NaiveDateTime {
15861586
/// from_ymd(2016, 7, 9).and_hms_opt(3, 5, 7).unwrap()
15871587
/// );
15881588
/// assert_eq!(
1589-
/// hms(3, 5, 7) + TimeDelta::days(365),
1589+
/// hms(3, 5, 7) + TimeDelta::try_days(365).unwrap(),
15901590
/// from_ymd(2017, 7, 8).and_hms_opt(3, 5, 7).unwrap()
15911591
/// );
15921592
///
@@ -1608,7 +1608,7 @@ impl Timelike for NaiveDateTime {
16081608
/// assert_eq!(leap + TimeDelta::milliseconds(800), hmsm(3, 6, 0, 100));
16091609
/// assert_eq!(leap + TimeDelta::seconds(10), hmsm(3, 6, 9, 300));
16101610
/// assert_eq!(leap + TimeDelta::seconds(-10), hmsm(3, 5, 50, 300));
1611-
/// assert_eq!(leap + TimeDelta::days(1),
1611+
/// assert_eq!(leap + TimeDelta::try_days(1).unwrap(),
16121612
/// from_ymd(2016, 7, 9).and_hms_milli_opt(3, 5, 59, 300).unwrap());
16131613
/// ```
16141614
///
@@ -1776,7 +1776,7 @@ impl Add<Months> for NaiveDateTime {
17761776
/// from_ymd(2016, 7, 7).and_hms_opt(3, 5, 7).unwrap()
17771777
/// );
17781778
/// assert_eq!(
1779-
/// hms(3, 5, 7) - TimeDelta::days(365),
1779+
/// hms(3, 5, 7) - TimeDelta::try_days(365).unwrap(),
17801780
/// from_ymd(2015, 7, 9).and_hms_opt(3, 5, 7).unwrap()
17811781
/// );
17821782
///
@@ -1796,7 +1796,7 @@ impl Add<Months> for NaiveDateTime {
17961796
/// assert_eq!(leap - TimeDelta::milliseconds(200), hmsm(3, 5, 59, 1_100));
17971797
/// assert_eq!(leap - TimeDelta::milliseconds(500), hmsm(3, 5, 59, 800));
17981798
/// assert_eq!(leap - TimeDelta::seconds(60), hmsm(3, 5, 0, 300));
1799-
/// assert_eq!(leap - TimeDelta::days(1),
1799+
/// assert_eq!(leap - TimeDelta::try_days(1).unwrap(),
18001800
/// from_ymd(2016, 7, 7).and_hms_milli_opt(3, 6, 0, 300).unwrap());
18011801
/// ```
18021802
///

‎src/naive/datetime/tests.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,7 @@ fn test_datetime_addassignment() {
7575
let mut date = ymdhms(2016, 10, 1, 10, 10, 10);
7676
date += TimeDelta::minutes(10_000_000);
7777
assert_eq!(date, ymdhms(2035, 10, 6, 20, 50, 10));
78-
date += TimeDelta::days(10);
78+
date += TimeDelta::try_days(10).unwrap();
7979
assert_eq!(date, ymdhms(2035, 10, 16, 20, 50, 10));
8080
}
8181

@@ -86,7 +86,7 @@ fn test_datetime_subassignment() {
8686
let mut date = ymdhms(2016, 10, 1, 10, 10, 10);
8787
date -= TimeDelta::minutes(10_000_000);
8888
assert_eq!(date, ymdhms(1997, 9, 26, 23, 30, 10));
89-
date -= TimeDelta::days(10);
89+
date -= TimeDelta::try_days(10).unwrap();
9090
assert_eq!(date, ymdhms(1997, 9, 16, 23, 30, 10));
9191
}
9292

‎src/naive/time/mod.rs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1166,7 +1166,7 @@ impl Timelike for NaiveTime {
11661166
/// # let from_hmsm = |h, m, s, milli| { NaiveTime::from_hms_milli_opt(h, m, s, milli).unwrap() };
11671167
/// assert_eq!(from_hmsm(3, 5, 7, 0) + TimeDelta::seconds(22*60*60), from_hmsm(1, 5, 7, 0));
11681168
/// assert_eq!(from_hmsm(3, 5, 7, 0) + TimeDelta::seconds(-8*60*60), from_hmsm(19, 5, 7, 0));
1169-
/// assert_eq!(from_hmsm(3, 5, 7, 0) + TimeDelta::days(800), from_hmsm(3, 5, 7, 0));
1169+
/// assert_eq!(from_hmsm(3, 5, 7, 0) + TimeDelta::try_days(800).unwrap(), from_hmsm(3, 5, 7, 0));
11701170
/// ```
11711171
///
11721172
/// Leap seconds are handled, but the addition assumes that it is the only leap second happened.
@@ -1181,7 +1181,7 @@ impl Timelike for NaiveTime {
11811181
/// assert_eq!(leap + TimeDelta::milliseconds(800), from_hmsm(3, 6, 0, 100));
11821182
/// assert_eq!(leap + TimeDelta::seconds(10), from_hmsm(3, 6, 9, 300));
11831183
/// assert_eq!(leap + TimeDelta::seconds(-10), from_hmsm(3, 5, 50, 300));
1184-
/// assert_eq!(leap + TimeDelta::days(1), from_hmsm(3, 5, 59, 300));
1184+
/// assert_eq!(leap + TimeDelta::try_days(1).unwrap(), from_hmsm(3, 5, 59, 300));
11851185
/// ```
11861186
///
11871187
/// [leap second handling]: crate::NaiveTime#leap-second-handling
@@ -1281,7 +1281,7 @@ impl Add<FixedOffset> for NaiveTime {
12811281
/// # use chrono::{TimeDelta, NaiveTime};
12821282
/// # let from_hmsm = |h, m, s, milli| { NaiveTime::from_hms_milli_opt(h, m, s, milli).unwrap() };
12831283
/// assert_eq!(from_hmsm(3, 5, 7, 0) - TimeDelta::seconds(8*60*60), from_hmsm(19, 5, 7, 0));
1284-
/// assert_eq!(from_hmsm(3, 5, 7, 0) - TimeDelta::days(800), from_hmsm(3, 5, 7, 0));
1284+
/// assert_eq!(from_hmsm(3, 5, 7, 0) - TimeDelta::try_days(800).unwrap(), from_hmsm(3, 5, 7, 0));
12851285
/// ```
12861286
///
12871287
/// Leap seconds are handled, but the subtraction assumes that it is the only leap second happened.
@@ -1294,7 +1294,7 @@ impl Add<FixedOffset> for NaiveTime {
12941294
/// assert_eq!(leap - TimeDelta::milliseconds(200), from_hmsm(3, 5, 59, 1_100));
12951295
/// assert_eq!(leap - TimeDelta::milliseconds(500), from_hmsm(3, 5, 59, 800));
12961296
/// assert_eq!(leap - TimeDelta::seconds(60), from_hmsm(3, 5, 0, 300));
1297-
/// assert_eq!(leap - TimeDelta::days(1), from_hmsm(3, 6, 0, 300));
1297+
/// assert_eq!(leap - TimeDelta::try_days(1).unwrap(), from_hmsm(3, 6, 0, 300));
12981298
/// ```
12991299
///
13001300
/// [leap second handling]: crate::NaiveTime#leap-second-handling

0 commit comments

Comments
 (0)