Register the full IDate S3 class for S4 dispatch

[aitap]

As recommended by ?setOldClass, register c("IDate", "Date"), not just "IDate". The unit test doesn't involve lubridate, but also tested with lubridate manually:

> as.IDate(Sys.Date()) - days(1)
[1] "2025-02-27"
> as.IDate("2020-01-30") %m+% months(1)
[1] "2020-02-29"

Fixes: #6839

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.60%. Comparing base (3bb4e16) to head (f984218).
Report is 2 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #6844   +/-   ##
=======================================
  Coverage   98.60%   98.60%           
=======================================
  Files          79       79           
  Lines       14661    14661           
=======================================
  Hits        14457    14457           
  Misses        204      204           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[MichaelChirico]

up on L6 I see multiple calls for a similar case on the data.table/data.frame class. should we remove the one-class call there, or is there benefit from having both calls here too?

[MichaelChirico]

based on the description of the test= argument I guess we should remove L6

[github-actions]

Generated via commit f984218

Download link for the artifact containing the test results: ↓ atime-results.zip

Task	Duration
R setup and installing dependencies	4 minutes and 35 seconds
Installing different package versions	8 minutes and 10 seconds
Running and plotting the test cases	2 minutes and 21 seconds

[MichaelChirico]

looks great! missing NEWS

[aitap]

Hopefully no reverse dependency actually imported the data.frame S4 class from us.