MirrorMirror
/
Signal-Server
mirror of https://github.com/signalapp/Signal-Server.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Remove duplicate number entry

This commit is contained in:
Moxie Marlinspike 2019-01-10 10:24:20 -08:00
parent 5e3357d062
commit 052fd35c72
3 changed files with 61 additions and 61 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/main/java/org/whispersystems/textsecuregcm/storage/Account.java
View File

@ -32,7 +32,7 @@ public class Account implements Principal {
static final int MEMCACHE_VERION = 5; static final int MEMCACHE_VERION = 5;
@JsonProperty @JsonIgnore
private String number; private String number;
@JsonProperty @JsonProperty

92
src/test/java/org/whispersystems/textsecuregcm/tests/controllers/TransparentDataControllerTest.java
View File

@ -68,52 +68,52 @@ public class TransparentDataControllerTest {
when(accountsManager.get(eq("+14151232222"))).thenReturn(Optional.of(accountTwo)); when(accountsManager.get(eq("+14151232222"))).thenReturn(Optional.of(accountTwo));
} }
// @Test @Test
// public void testAccountOne() throws IOException { public void testAccountOne() throws IOException {
// Response response = resources.getJerseyTest() Response response = resources.getJerseyTest()
// .target(String.format("/v1/transparency/account/%s", "1")) .target(String.format("/v1/transparency/account/%s", "1"))
// .request() .request()
// .get(); .get();
//
// assertEquals(200, response.getStatus()); assertEquals(200, response.getStatus());
//
// Account result = response.readEntity(PublicAccount.class); Account result = response.readEntity(PublicAccount.class);
//
// assertTrue(result.getPin().isPresent()); assertTrue(result.getPin().isPresent());
// assertEquals("******", result.getPin().get()); assertEquals("******", result.getPin().get());
// assertNull(result.getNumber()); assertNull(result.getNumber());
// assertEquals("OneProfileName", result.getProfileName()); assertEquals("OneProfileName", result.getProfileName());
//
// assertThat("Account serialization works", assertThat("Account serialization works",
// asJson(result), asJson(result),
// is(equalTo(jsonFixture("fixtures/transparent_account.json")))); is(equalTo(jsonFixture("fixtures/transparent_account.json"))));
//
// verify(accountsManager, times(1)).get(eq("+14151231111")); verify(accountsManager, times(1)).get(eq("+14151231111"));
// verifyNoMoreInteractions(accountsManager); verifyNoMoreInteractions(accountsManager);
// } }
//
// @Test @Test
// public void testAccountTwo() throws IOException { public void testAccountTwo() throws IOException {
// Response response = resources.getJerseyTest() Response response = resources.getJerseyTest()
// .target(String.format("/v1/transparency/account/%s", "2")) .target(String.format("/v1/transparency/account/%s", "2"))
// .request() .request()
// .get(); .get();
//
// assertEquals(200, response.getStatus()); assertEquals(200, response.getStatus());
//
// Account result = response.readEntity(PublicAccount.class); Account result = response.readEntity(PublicAccount.class);
//
// assertTrue(result.getPin().isPresent()); assertTrue(result.getPin().isPresent());
// assertEquals("******", result.getPin().get()); assertEquals("******", result.getPin().get());
// assertNull(result.getNumber()); assertNull(result.getNumber());
// assertEquals("TwoProfileName", result.getProfileName()); assertEquals("TwoProfileName", result.getProfileName());
//
// assertThat("Account serialization works 2", assertThat("Account serialization works 2",
// asJson(result), asJson(result),
// is(equalTo(jsonFixture("fixtures/transparent_account2.json")))); is(equalTo(jsonFixture("fixtures/transparent_account2.json"))));
//
// verify(accountsManager, times(1)).get(eq("+14151232222")); verify(accountsManager, times(1)).get(eq("+14151232222"));
// } }
@Test @Test
public void testAccountMissing() { public void testAccountMissing() {

28
src/test/java/org/whispersystems/textsecuregcm/tests/storage/PublicAccountTest.java
View File

@ -19,20 +19,20 @@ import static junit.framework.TestCase.assertNull;
public class PublicAccountTest { public class PublicAccountTest {
// @Test @Test
// public void testPinSanitation() throws IOException { public void testPinSanitation() throws IOException {
// Set<Device> devices = Collections.singleton(new Device(1, "foo", "bar", "12345", null, "gcm-1234", null, null, true, 1234, new SignedPreKey(1, "public-foo", "signature-foo"), 31337, 31336, "Android4Life", true)); Set<Device> devices = Collections.singleton(new Device(1, "foo", "bar", "12345", null, "gcm-1234", null, null, true, 1234, new SignedPreKey(1, "public-foo", "signature-foo"), 31337, 31336, "Android4Life", true));
// Account account = new Account("+14151231234", devices, new byte[16]); Account account = new Account("+14151231234", devices, new byte[16]);
// account.setPin("123456"); account.setPin("123456");
//
// PublicAccount publicAccount = new PublicAccount(account); PublicAccount publicAccount = new PublicAccount(account);
//
// String serialized = SystemMapper.getMapper().writeValueAsString(publicAccount); String serialized = SystemMapper.getMapper().writeValueAsString(publicAccount);
// JsonNode result = SystemMapper.getMapper().readTree(serialized); JsonNode result = SystemMapper.getMapper().readTree(serialized);
//
// assertEquals("******", result.get("pin").textValue()); assertEquals("******", result.get("pin").textValue());
// assertNull(result.get("number")); assertNull(result.get("number"));
// } }
} }