Fix log backup filtering to use End time instead of Start in Select-DbaBackupInformation

[bertvanlandeghem]

FIX: Select-DbaBackupInformation returns the last log backup to restore twice, when the RestorePoint falls between the log backup Start and End time, or exactly at the log backup End time.

Type of Change

• Bug fix (non-breaking change, fixes Select-DbaBackupInformation returns the last log backup to restore twice, when the RestorePoint falls between the log backup Start and End time, or exactly at the log backup end. #9626 )
• New feature (non-breaking change, adds functionality, fixes # )
• Breaking change (affects multiple commands or functionality, fixes # )
• Ran manual Pester test and has passed (Invoke-ManualPester)
• Adding code coverage to existing functionality
• Pester test is included
• If new file reference added for test, has is been added to github.com/dataplat/appveyor-lab ?
• Unit test is included
• Documentation
• Build system

[potatoqualitee]

Oh, man! I accidentally merged this one thinking it was another one. Select-DbaBackupInformation is an essential command -- @niphlod do you see any issues? I recall the start/end debate was a debate. If there is an issue, I can revert. Thank you, btw, @bertvanlandeghem

[niphlod]

@potatoqualitee : 1. we need to restore unittest functionality 2. yeah, I'd say this fix would break existing logics just to cover a niche usecase that may be better identified with a proper set of backup files, then we insert a regression test, then we can think about a better way to fix it rather than completely change the filtering logic like this