s434730/RandomSec
3
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Fix ConnectionManager and add test for LoginCommand

Browse Source
This commit is contained in:
Antonin Delpeuch 2018-03-05 09:02:00 +00:00
parent 3a58c6824d
commit 580c0bed80
2 changed files with 30 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
extensions/wikidata/src/org/openrefine/wikidata/editing/ConnectionManager.java
View File

@ -54,13 +54,10 @@ public class ConnectionManager {
private PreferenceStore prefStore;
private ApiConnection connection;
private static class ConnectionManagerHolder {
private static final ConnectionManager instance = new ConnectionManager();
}
private static final ConnectionManager instance = new ConnectionManager();
public static ConnectionManager getInstance() {
return ConnectionManagerHolder.instance;
return instance;
}
private ConnectionManager() {
@ -78,7 +75,6 @@ public class ConnectionManager {
obj.put("password", password);
array.put(obj);
prefStore.put(PREFERENCE_STORE_KEY, array);
// TODO save preferences (find out how)
} catch (JSONException e) {
e.printStackTrace();
}
@ -108,7 +104,7 @@ public class ConnectionManager {
public JSONObject getStoredCredentials() {
JSONArray array = (JSONArray) prefStore.get(PREFERENCE_STORE_KEY);
if (array.length() > 0) {
if (array != null && array.length() > 0) {
try {
return array.getJSONObject(0);
} catch (JSONException e) {

27
extensions/wikidata/tests/src/org/openrefine/wikidata/commands/LoginCommandTest.java Normal file
View File

@ -0,0 +1,27 @@
package org.openrefine.wikidata.commands;
import static org.junit.Assert.assertEquals;
import java.io.IOException;
import javax.servlet.ServletException;
import org.json.JSONException;
import org.testng.annotations.BeforeMethod;
import org.testng.annotations.Test;
public class LoginCommandTest extends CommandTest {
@BeforeMethod
public void SetUp()
throws JSONException {
command = new LoginCommand();
}
@Test
public void testNoCredentials() throws ServletException, IOException {
command.doPost(request, response);
assertEquals("{\"logged_in\":false,\"username\":null}", writer.toString());
}
}