Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Migrate to new test base, 2, 3 #93

Merged
merged 3 commits into from
Oct 7, 2021
Merged

Migrate to new test base, 2, 3 #93

merged 3 commits into from
Oct 7, 2021

Conversation

maread99
Copy link
Collaborator

@maread99 maread99 commented Oct 7, 2021

Migrate to new test base, 2
Migrates EuronextCalendarTestBase and its 4 subclasses.

Revise already migrated test modules
Minor revisions to previously migrated test modules (inc amending fixture scope).

Migrate to new test base, 3
Migrates further 10 test modules.

Migrates to new test base `EuronextCalendarTestBase` and its
subclasses as defined on following test modules:
- `test_xams_calendar.py`
- `test_xbru_calendar.py`
- `test_xlis_calendar.py`
- `test_xpar_calendar.py`
Amends scope of fixtures (from class to default function) on previously
migrated calendar test modules.

Also, minor revisions to previously migrated calendar test modules for
XLON and XHKG to accommodate previous changes to calendar-specific
tests.
Migrates following test modules to new test base:
- `test_xbkk_calendar.py`
- `test_xbog_calendar.py`
- `test_xbom_calendar.py`
- `test_xbse_calendar.py`
- `test_xbud_calendar.py`
- `test_xbue_calendar.py`
- `test_xcbf_calendar.py`
- `test_xcse_calendar.py`
- `test_xdub_calendar.py`
- `test_xetr_calendar.py`
@maread99 maread99 marked this pull request as ready for review October 7, 2021 07:38
@maread99 maread99 merged commit a5f2ca1 into master Oct 7, 2021
@maread99 maread99 deleted the test-migration-3 branch October 7, 2021 07:38
@maread99 maread99 changed the title Test migration 3 Migrate to new test base, 2, 3 Oct 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant