Skip to content
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

chore: fix Javadoc warnings in 'test-clients' #14337

Merged
merged 1 commit into from
Jul 22, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,6 @@
import static com.hedera.hapi.util.HapiUtils.parseAccount;
import static com.hedera.node.app.hapi.utils.CommonPbjConverters.fromPbj;
import static com.hedera.services.bdd.junit.hedera.ExternalPath.ADDRESS_BOOK;
import static com.swirlds.platform.config.legacy.LegacyConfigPropertiesLoader.loadConfigFile;
jjohannes marked this conversation as resolved.
Show resolved Hide resolved
import static com.swirlds.platform.system.InitTrigger.GENESIS;
import static com.swirlds.platform.system.status.PlatformStatus.ACTIVE;
import static com.swirlds.platform.system.status.PlatformStatus.FREEZE_COMPLETE;
Expand All @@ -45,6 +44,7 @@
import com.hederahashgraph.api.proto.java.TransactionResponse;
import com.swirlds.common.constructable.ConstructableRegistry;
import com.swirlds.common.platform.NodeId;
import com.swirlds.platform.config.legacy.LegacyConfigPropertiesLoader;
import com.swirlds.platform.listeners.PlatformStatusChangeNotification;
import com.swirlds.platform.state.PlatformState;
import com.swirlds.platform.system.address.Address;
Expand Down Expand Up @@ -235,7 +235,7 @@ private static byte[] usedBytesFrom(@NonNull final BufferedData responseBuffer)

private static AddressBook loadAddressBook(@NonNull final Path path) {
requireNonNull(path);
final var configFile = loadConfigFile(path.toAbsolutePath());
final var configFile = LegacyConfigPropertiesLoader.loadConfigFile(path.toAbsolutePath());
final var randomAddressBook = RandomAddressBookBuilder.create(new Random())
.withSize(1)
.withRealKeysEnabled(true)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -16,12 +16,12 @@

package com.hedera.services.bdd.junit.hedera.utils;

import static com.swirlds.platform.config.legacy.LegacyConfigPropertiesLoader.loadConfigFile;
import static java.util.Objects.requireNonNull;
import static java.util.Spliterators.spliteratorUnknownSize;
import static java.util.stream.StreamSupport.stream;

import com.hedera.services.bdd.spec.props.JutilPropertySource;
import com.swirlds.platform.config.legacy.LegacyConfigPropertiesLoader;
import com.swirlds.platform.system.address.AddressBook;
import com.swirlds.platform.test.fixtures.addressbook.RandomAddressBookBuilder;
import edu.umd.cs.findbugs.annotations.NonNull;
Expand Down Expand Up @@ -309,7 +309,7 @@ public static void ensureDir(@NonNull final String path) {
*/
public static AddressBook loadAddressBook(@NonNull final Path path) {
requireNonNull(path);
final var configFile = loadConfigFile(path.toAbsolutePath());
final var configFile = LegacyConfigPropertiesLoader.loadConfigFile(path.toAbsolutePath());
final var randomAddressBook = RandomAddressBookBuilder.create(new Random())
.withSize(1)
.withRealKeysEnabled(true)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -130,7 +130,7 @@ public DissociateTokensOperation dissociateTokens(@NonNull final SpecToken... to
* Returns an operation to associate the contract with the given tokens. Will ultimately fail if the contract
* does not have an admin key.
*
* @param tokens the tokens to associate
* @param token the tokens to associate
* @return the operation
*/
public TransferTokenOperation transferToken(
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -45,9 +45,9 @@
* <li><b>(Optional)</b> A <i>settings.txt</i> at the given path.</li>
* <li><b>(Optional)</b> A <i>config.txt</i> obtained by removing a requested
* node from the target network's current <i>config.txt</i>. If this is specified,
* the target network must be a {@link SubProcessNetwork}</li>, since this is
* the target network must be a {@link SubProcessNetwork}, since this is
* the only case in which the test can meaningfully inspect and change the
* address book.
* address book.</li>
* </ol>
*/
public class BuildUpgradeZipOp extends UtilOp {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@

/**
* Tests expected behavior when the {@code cryptoCreateWithAlias.enabled} feature flag is off for
* <a href="https://hips.hedera.com/hip/hip-583">HIP-583, "Expand alias support in CryptoCreate & CryptoTransfer Transactions"</a>.
* <a href="https://hips.hedera.com/hip/hip-583">HIP-583, "Expand alias support in CryptoCreate &amp; CryptoTransfer Transactions"</a>.
*/
@HapiTestLifecycle
public class CreateWithAliasDisabledTest {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -95,7 +95,7 @@ default HapiSpecOperation reconnectNode(@NonNull final NodeSelector selector, fi
}

/**
* Returns an operation that builds a fake upgrade ZIP, uploads it to file {@code 0.0.150),
* Returns an operation that builds a fake upgrade ZIP, uploads it to file {@code 0.0.150},
* issues a {@code PREPARE_UPGRADE}, and awaits writing of the <i>execute_immediate.mf</i>.
* @return the operation
*/
Expand Down
Loading