-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
fix #985 make sFactory private in factories #2006
Conversation
maxme
commented
Nov 20, 2014
- fix Make sFactory members private and prevent call to the setter in non test mode #985 make sFactory private in factories
- UI tests are unusable, so I removed all of them
- fix WPHtml test
import java.net.URI; | ||
|
||
public class XMLRPCFactory { | ||
public static XMLRPCFactoryAbstract sFactory; | ||
private static XMLRPCFactoryAbstract sFactory; | ||
|
||
public static XMLRPCClientInterface instantiate(URI uri, String httpUser, String httpPassword) { | ||
if (sFactory == null) { | ||
sFactory = new XMLRPCFactoryDefault(); |
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.
Should we move this (and others like it) to a static initializer?
Good discussion here.
…ctory-private Conflicts: WordPress/src/androidTest/java/org/wordpress/android/ActivityRobotiumTestCase.java
I tested this by running the "connectedCheck" task and saw this: https://cloudup.com/cosWi_l8fv1 |
@nbradbury needs another review, tests are failing for the WordPressComRest subproject. To run tests for the WordPress project only: |
That worked |
…y-private fix #985 make sFactory private in factories