-
-
Notifications
You must be signed in to change notification settings - Fork 812
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
Move relationship return properties to the processor class to fix leakage related test fail #12521
Conversation
(Standard links)
|
48e9237
to
a7dec79
Compare
@eileenmcnaughton syntax error
|
@seamuslee001 ug - just rebased - 2 secs let me try again |
a7dec79
to
dd69b0f
Compare
…kage related test fail
dd69b0f
to
704e3e9
Compare
@@ -611,9 +611,6 @@ public function testExportIMData() { | |||
} | |||
} | |||
|
|||
// early return for now until we solve a leakage issue. | |||
return; |
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.
removing this enables a lot of coverage
Tested on my local, works fine. Did export with various use-cases, didn't encountered any issue. Tests passed. Happy with the patch, merging now |
thanks! @monishdeb |
Overview
Code rejigging to support increased testing
Before
Part of im_provider test didn't work except in isolation due to leakage
After
Full test enabled & passes
Technical Details
Now we have the ExportProcessor class moving properties that should be OO onto it solves a catalogue of woes
Comments
The newly enabled test covers this code change very thoroughly