Move `RoundId` first in the params list
MODIFY
WalletWasabi.Tests/UnitTests/WabiSabi/Client/ArenaClientTests.cs
WalletWasabi.Tests/UnitTests/WabiSabi/Client/ArenaClientTests.cs
@@ -52,7 +52,7 @@ namespace WalletWasabi.Tests.UnitTests.WabiSabi.Client
ZeroCredentialPool zeroVsizeCredentials = new();
var aliceArenaClient = new ArenaClient(round.AmountCredentialIssuerParameters, round.VsizeCredentialIssuerParameters, zeroAmountCredentials, zeroVsizeCredentials, wabiSabiApi, new InsecureRandom());
- var inputRegistrationResponse = await aliceArenaClient.RegisterInputAsync(outpoint, key, round.Id, CancellationToken.None);
+ var inputRegistrationResponse = await aliceArenaClient.RegisterInputAsync(round.Id, outpoint, key, CancellationToken.None);
var aliceId = inputRegistrationResponse.Value;
var amountsToRequest = new[]

MODIFY
WalletWasabi.Tests/UnitTests/WabiSabi/Integration/WabiSabiHttpApiIntegrationTests.cs
WalletWasabi.Tests/UnitTests/WabiSabi/Integration/WabiSabiHttpApiIntegrationTests.cs
@@ -41,7 +41,7 @@ namespace WalletWasabi.Tests.UnitTests.WabiSabi.Integration
using var signingKey = new Key();
var ex = await Assert.ThrowsAsync<HttpRequestException>(async () =>
- await apiClient.RegisterInputAsync(nonExistingOutPoint, signingKey, round.Id, CancellationToken.None));
+ await apiClient.RegisterInputAsync(round.Id, nonExistingOutPoint, signingKey, CancellationToken.None));
var wex = Assert.IsType<WabiSabiProtocolException>(ex.InnerException);
Assert.Equal(WabiSabiProtocolErrorCode.InputSpent, wex.ErrorCode);
@@ -149,7 +149,7 @@ namespace WalletWasabi.Tests.UnitTests.WabiSabi.Integration
var rounds = await apiClient.GetStatusAsync(CancellationToken.None);
var round = rounds.First(x => x.CoinjoinState is ConstructionState);
- var response = await apiClient.RegisterInputAsync(coinToRegister.Outpoint, signingKey, round.Id, CancellationToken.None);
+ var response = await apiClient.RegisterInputAsync(round.Id, coinToRegister.Outpoint, signingKey, CancellationToken.None);
Assert.NotEqual(uint256.Zero, response.Value);
}
@@ -182,7 +182,7 @@ namespace WalletWasabi.Tests.UnitTests.WabiSabi.Integration
var rounds = await apiClient.GetStatusAsync(CancellationToken.None);
var round = rounds.First(x => x.CoinjoinState is ConstructionState);
- var response = await apiClient.RegisterInputAsync(coinToRegister.Outpoint, signingKey, round.Id, CancellationToken.None);
+ var response = await apiClient.RegisterInputAsync(round.Id, coinToRegister.Outpoint, signingKey, CancellationToken.None);
Assert.NotEqual(uint256.Zero, response.Value);
}

MODIFY
WalletWasabi/WabiSabi/Client/AliceClient.cs
WalletWasabi/WabiSabi/Client/AliceClient.cs
@@ -35,7 +35,7 @@ namespace WalletWasabi.WabiSabi.Client
public async Task RegisterInputAsync(CancellationToken cancellationToken)
{
- var response = await ArenaClient.RegisterInputAsync(Coin.Outpoint, BitcoinSecret.PrivateKey, RoundId, cancellationToken).ConfigureAwait(false);
+ var response = await ArenaClient.RegisterInputAsync(RoundId, Coin.Outpoint, BitcoinSecret.PrivateKey, cancellationToken).ConfigureAwait(false);
var remoteAliceId = response.Value;
if (AliceId != remoteAliceId)
{

MODIFY
WalletWasabi/WabiSabi/Client/ArenaClient.cs
WalletWasabi/WabiSabi/Client/ArenaClient.cs
@@ -41,9 +41,9 @@ namespace WalletWasabi.WabiSabi.Client
public IWabiSabiApiRequestHandler RequestHandler { get; }
public async Task<ArenaResponse<uint256>> RegisterInputAsync(
+ uint256 roundId,
OutPoint outPoint,
Key key,
- uint256 roundId,
CancellationToken cancellationToken)
{
var ownershipProof = OwnershipProof.GenerateCoinJoinInputProof(

Move `RoundId` first in the params list