From 92b54295d84dd60bcbc7386568f5a537c816938c Mon Sep 17 00:00:00 2001
From: aeberhardt <alexandre.eberhardt@savoirfairelinux.com>
Date: Mon, 27 Jan 2025 15:10:04 -0500
Subject: [PATCH] tests: fix nameserver test for user registration
use test server ns-test.jami.net instead of ns.jami.net
Name registration had 2 issues :
wrong adress: it should be 40 characters long and be unique
signature and publickey can be "" but not null
Change-Id: I7c160de816b8527316111af8f1489b564f546986
---
.../jaminamserver/PublicNameServerTest.java | 20 ++++++++++---------
1 file changed, 11 insertions(+), 9 deletions(-)
diff --git a/jams-server/src/test/java/net/jami/jams/server/core/jaminamserver/PublicNameServerTest.java b/jams-server/src/test/java/net/jami/jams/server/core/jaminamserver/PublicNameServerTest.java
index 5f257a16..32f6f501 100644
--- a/jams-server/src/test/java/net/jami/jams/server/core/jaminamserver/PublicNameServerTest.java
+++ b/jams-server/src/test/java/net/jami/jams/server/core/jaminamserver/PublicNameServerTest.java
@@ -23,7 +23,6 @@ import net.jami.jams.nameserver.PublicNameServer;
import org.junit.jupiter.api.Assertions;
import org.junit.jupiter.api.BeforeAll;
-import org.junit.jupiter.api.Disabled;
import org.junit.jupiter.api.Test;
import java.util.UUID;
@@ -34,17 +33,20 @@ class PublicNameServerTest {
@BeforeAll
public static void init() {
- nameServer = new PublicNameServer("https://ns.jami.net");
+ nameServer = new PublicNameServer("https://ns-test.jami.net");
}
- @Disabled
@Test
public void testNameRegistration() {
NameRegistrationRequest nameRegistrationRequest = new NameRegistrationRequest();
- nameRegistrationRequest.setAddr("0e76529938772ac7626552ab71");
- nameRegistrationRequest.setOwner("0x74ef2cD532a0f3cE31089b4fd296DDdFB5d6566A");
- nameRegistrationRequest.setSignature(null);
- nameRegistrationRequest.setPublickey(null);
+ nameRegistrationRequest.setAddr(
+ "abcdef0123"
+ + UUID.randomUUID()
+ .toString()
+ .replace("-", "")); // 40 characters long: 8 fixed + 32 random
+ nameRegistrationRequest.setOwner("abcdef0123456789abcdef0123456789abcdef02");
+ nameRegistrationRequest.setSignature("");
+ nameRegistrationRequest.setPublickey("");
String name = UUID.randomUUID().toString().replace("-", "");
Integer res = nameServer.registerName(name, nameRegistrationRequest);
Assertions.assertEquals(200, res, "The response should have been 200!");
@@ -52,13 +54,13 @@ class PublicNameServerTest {
@Test
public void testNameLookUp() {
- NameLookupResponse resp = nameServer.getAddressFromName("sidokhine6");
+ NameLookupResponse resp = nameServer.getAddressFromName("abcdefabcdef");
Assertions.assertNotNull(resp, "The response should exist!");
}
@Test
public void testAddrLookUp() {
- String addr = nameServer.getNameFromAddress("0d1f0002ce728d6aa8b98b5227c75fc773735f9e");
+ String addr = nameServer.getNameFromAddress("abcdef0123456789abcdef0123456789abcdef02");
Assertions.assertNotNull(addr, "The address should exist!");
}
}
--
GitLab