PR #3572 review changes included

core/src/main/java/bisq/core/payment/PaymentAccount.java

@@ -44,7 +44,6 @@
 
 import static com.google.common.base.Preconditions.checkNotNull;
 
-@EqualsAndHashCode
 @ToString
 @Getter
 @Slf4j

core/src/test/java/bisq/core/payment/PaymentAccountsTest.java

@@ -79,11 +79,8 @@ private static PaymentAccount createAccountWithAge(AccountAgeWitnessService serv
 
     @Test
     public void testRemoveAccount() {
-        PaymentAccountPayload payload = mock(PaymentAccountPayload.class);
-
-        PaymentAccount account = mock(PaymentAccount.class);
+        PaymentAccount account = new RevolutAccount();
         account.setAccountName("account name");
-        when(account.getPaymentAccountPayload()).thenReturn(payload);
 
         ObservableSet<PaymentAccount> paymentAccountsAsObservable;
         paymentAccountsAsObservable = FXCollections.observableSet(new UserPayload().getPaymentAccounts());
@@ -92,7 +89,7 @@ public void testRemoveAccount() {
         paymentAccountsAsObservable.add(account);
 
         //remove element with some state change
-        account.setAccountName("new account name");
+        account.setAccountName("new account name for new hash code");
 
         assertTrue(paymentAccountsAsObservable.remove(account));
     }