-
Notifications
You must be signed in to change notification settings - Fork 5
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
SDK-2870: Use new structure of test data provider #123
Conversation
Codecov Report
@@ Coverage Diff @@
## master #123 +/- ##
============================================
+ Coverage 69.31% 69.33% +0.01%
- Complexity 2022 2024 +2
============================================
Files 147 147
Lines 5306 5306
============================================
+ Hits 3678 3679 +1
+ Misses 1628 1627 -1
|
* | ||
* @return string | ||
*/ | ||
public function getProjectMockOriginalPath(): string |
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.
this method should be protected if it is not a test method or isn't a data provider.
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.
Yes, I missed that. I changed the visibility of the parent methods, it really has no sense to make them public
No description provided.