@JC5 I need your assistance on this one. The logic matches QTD and YTD.

Looks OK to me, I'll check it out later this week. Thanks!

Oh lol no the code change is required, it's the TEST that needs to be updated.

@JC5
I felt that. But I was very confused as MTD is more of a duration. And the tests are like this for MTD QTD YTD. YTD and QTD are passing. I thought they all follow the same logic.

  'MTD'                           => ['frequency' => 'MTD', 'from' => Carbon::now(), 'expected' => Carbon::now()->startOfMonth()->startOfDay()],
            'QTD'                           => ['frequency' => 'QTD', 'from' => Carbon::now(), 'expected' => Carbon::now()->firstOfQuarter()->startOfDay()],
            'YTD'                           => ['frequency' => 'YTD', 'from' => Carbon::now(), 'expected' => Carbon::now()->startOfYear()->startOfDay()],

May you please explain the logic of MTD YTD QTD to me and I will fix it?

@JC5 Check this! I updated the tests for MTD.

Nice work!