Description

Sometimes, when the security exchange timezone is not the same as the data timezone, the class DateChangeTimeKeeper does not produce a delisting event but the SubscriptionDataReader thinks one was produced. Apparently, this happens since in DateChangeTimeKeeper the date returned by _tradableDatesInDataTimeZone is compared with the delisting date after converting it to the exchange timezone. However, in SubscriptionDataReader, that same date is directly compared with the delisting date.

Related Issue

Closes #8413

Motivation and Context

With this change all index option contracts will be exercised after being delisted

Requires Documentation Change

N/A

How Has This Been Tested?

In progress...

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• Refactor (non-breaking change which improves implementation)
• Performance (non-breaking change which improves performance. Please add associated performance test and results)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)
• Non-functional change (xml comments/documentation/etc)

Checklist:

• My code follows the code style of this project.
• I have read the CONTRIBUTING document.
• I have added tests to cover my changes.
• All new and existing tests passed.
• My branch follows the naming convention bug-<issue#>-<description> or feature-<issue#>-<description>