Merge pull request #29 from Ariho-Seth/master

Add constructor tests for FileCreateRequest

Author: Ndacyayisenga-droid

hedera-base/src/main/java/com/openelements/hedera/base/NftClient.java

@@ -3,8 +3,6 @@
 import com.hedera.hashgraph.sdk.AccountId;
 import com.hedera.hashgraph.sdk.PrivateKey;
 import com.hedera.hashgraph.sdk.TokenId;
-import java.nio.charset.StandardCharsets;
-import java.util.Arrays;
 import java.util.List;
 import java.util.Objects;
 import java.util.Set;

hedera-base/src/main/java/com/openelements/hedera/base/NftRepository.java

@@ -3,7 +3,6 @@
 import com.hedera.hashgraph.sdk.AccountId;
 import com.hedera.hashgraph.sdk.TokenId;
 import com.openelements.hedera.base.mirrornode.Page;
-import java.util.List;
 import java.util.Objects;
 import java.util.Optional;
 import org.jspecify.annotations.NonNull;

hedera-base/src/main/java/com/openelements/hedera/base/implementation/NftRepositoryImpl.java

@@ -7,7 +7,6 @@
 import com.openelements.hedera.base.NftRepository;
 import com.openelements.hedera.base.mirrornode.MirrorNodeClient;
 import com.openelements.hedera.base.mirrornode.Page;
-import java.util.List;
 import java.util.Objects;
 import java.util.Optional;
 import org.jspecify.annotations.NonNull;

hedera-base/src/main/java/com/openelements/hedera/base/mirrornode/MirrorNodeClient.java

@@ -4,7 +4,6 @@
 import com.hedera.hashgraph.sdk.TokenId;
 import com.openelements.hedera.base.HederaException;
 import com.openelements.hedera.base.Nft;
-import java.util.List;
 import java.util.Objects;
 import java.util.Optional;
 import org.jspecify.annotations.NonNull;

hedera-base/src/test/java/com/openelements/hedera/base/test/ProtocolLayerDataCreationTests.java

@@ -45,6 +45,8 @@
 import com.openelements.hedera.base.protocol.TokenAssociateRequest;
 import com.openelements.hedera.base.protocol.FileUpdateRequest;
 import com.openelements.hedera.base.protocol.FileInfoRequest;
+import com.openelements.hedera.base.protocol.FileDeleteRequest;
+import com.openelements.hedera.base.protocol.FileCreateRequest;
 
 import java.lang.reflect.Constructor;
 import java.nio.charset.StandardCharsets;
@@ -669,4 +671,46 @@ void testFileContentsRequestCreation() {
         Assertions.assertThrows(NullPointerException.class, () -> FileContentsRequest.of((FileId) null));
     }
 
+    @Test
+    void testFileCreateRequestCreation(){
+        //given
+        final Hbar maxTransactionFee= Hbar.fromTinybars(1000);
+        final Duration transactionValidDuration= Duration.ofSeconds(10);
+        final byte[] contents= new byte[]{};
+        final Instant expirationTime= Instant.MAX;
+        final byte[] largeContents = IntStream.range(0, 2050).mapToObj(i -> "a").reduce("", (a,b) -> a+b).getBytes();
+        final String fileMemo= "fileMemo";
+
+        //then
+        Assertions.assertDoesNotThrow(() -> FileCreateRequest.of(contents));
+        Assertions.assertDoesNotThrow(() -> FileCreateRequest.of(contents, null));
+        Assertions.assertDoesNotThrow(() -> new FileCreateRequest(maxTransactionFee, transactionValidDuration, contents, expirationTime, fileMemo));
+        Assertions.assertDoesNotThrow(() -> new FileCreateRequest(maxTransactionFee, transactionValidDuration, contents, null, fileMemo));
+        Assertions.assertDoesNotThrow(() -> new FileCreateRequest(maxTransactionFee, transactionValidDuration, contents, expirationTime, null));
+        Assertions.assertDoesNotThrow(() -> new FileCreateRequest(maxTransactionFee, transactionValidDuration, contents, null, null));
+        Assertions.assertThrows(IllegalArgumentException.class, () -> FileCreateRequest.of(largeContents));
+        Assertions.assertThrows(NullPointerException.class, () -> FileCreateRequest.of(null));
+        Assertions.assertThrows(NullPointerException.class, () -> FileCreateRequest.of(null, expirationTime));
+        Assertions.assertThrows(NullPointerException.class, () -> FileCreateRequest.of(null, null));
+        Assertions.assertThrows(NullPointerException.class, () -> new FileCreateRequest(null, null, null, null, null));
+    }
+    @Test
+    void testFileDeleteRequestCreation(){
+        //given
+        final Hbar maxTransactionFee= Hbar.fromTinybars(1000);
+        final Duration transactionValidDuration= Duration.ofSeconds(10);
+        final String fileIdString= "0.0.12345";
+        final FileId fileId= FileId.fromString(fileIdString);
+
+        //then
+        Assertions.assertDoesNotThrow(() -> FileDeleteRequest.of(fileId));
+        Assertions.assertDoesNotThrow(() -> FileDeleteRequest.of(fileIdString));
+        Assertions.assertDoesNotThrow(() -> new FileDeleteRequest(maxTransactionFee, transactionValidDuration, fileId));
+        Assertions.assertThrows(NullPointerException.class, () -> FileDeleteRequest.of((FileId) null));
+        Assertions.assertThrows(NullPointerException.class, () -> FileDeleteRequest.of((String) null));
+        Assertions.assertThrows(NullPointerException.class, () -> new FileDeleteRequest(maxTransactionFee,transactionValidDuration, null));
+        Assertions.assertThrows(NullPointerException.class, () -> new FileDeleteRequest(maxTransactionFee,null, null));
+        Assertions.assertThrows(NullPointerException.class, () -> new FileDeleteRequest(null, transactionValidDuration, null));
+        Assertions.assertThrows(NullPointerException.class, () -> new FileDeleteRequest(null, null, null));
+    }
 }

hedera-microprofile/src/test/java/com/openelements/hedera/microprofile/test/FileClientTests.java

@@ -1,11 +1,8 @@
 package com.openelements.hedera.microprofile.test;
 
 import com.openelements.hedera.base.FileClient;
-import io.quarkus.test.junit.QuarkusTest;
 import javax.inject.Inject;
 import org.junit.jupiter.api.Assertions;
-import org.junit.jupiter.api.Disabled;
-import org.junit.jupiter.api.Test;
 
 //@QuarkusTest
 public class FileClientTests {