-
Notifications
You must be signed in to change notification settings - Fork 51
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
test(i): Remap alias fields when refreshing test documents #1897
test(i): Remap alias fields when refreshing test documents #1897
Conversation
Codecov ReportPatch coverage has no change and project coverage change:
@@ Coverage Diff @@
## develop #1897 +/- ##
===========================================
- Coverage 70.31% 70.18% -0.14%
===========================================
Files 232 232
Lines 24192 24192
===========================================
- Hits 17010 16977 -33
- Misses 6021 6044 +23
- Partials 1161 1171 +10
Flags with carried forward coverage won't be shown. Click here to find out more. see 8 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks Keenan, and good job finding the fix
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm
…work#1897) ## Relevant issue(s) N/A ## Description This PR fixes a bug that causes documents to not be correctly refreshed when they are created using alias fields. ## Tasks - [x] I made sure the code is well commented, particularly hard-to-understand areas. - [x] I made sure the repository-held documentation is changed accordingly. - [x] I made sure the pull request title adheres to the conventional commit style (the subset used in the project can be found in [tools/configs/chglog/config.yml](tools/configs/chglog/config.yml)). - [x] I made sure to discuss its limitations such as threats to validity, vulnerability to mistake and misuse, robustness to invalidation of assumptions, resource requirements, ... ## How has this been tested? `make test:changes` Specify the platform(s) on which this was tested: - MacOS
Relevant issue(s)
N/A
Description
This PR fixes a bug that causes documents to not be correctly refreshed when they are created using alias fields.
Tasks
How has this been tested?
make test:changes
Specify the platform(s) on which this was tested: