Remove some sprout-specific tests
This commit is contained in:
@@ -10,25 +10,6 @@
|
|||||||
|
|
||||||
//TODO: Update these tests for Sapling
|
//TODO: Update these tests for Sapling
|
||||||
/*
|
/*
|
||||||
TEST(checktransaction_tests, check_vpub_not_both_nonzero) {
|
|
||||||
CMutableTransaction tx;
|
|
||||||
tx.nVersion = 2;
|
|
||||||
|
|
||||||
{
|
|
||||||
// Ensure that values within the joinsplit are well-formed.
|
|
||||||
CMutableTransaction newTx(tx);
|
|
||||||
CValidationState state;
|
|
||||||
|
|
||||||
newTx.vjoinsplit.push_back(JSDescription());
|
|
||||||
|
|
||||||
JSDescription *jsdesc = &newTx.vjoinsplit[0];
|
|
||||||
jsdesc->vpub_old = 1;
|
|
||||||
jsdesc->vpub_new = 1;
|
|
||||||
|
|
||||||
EXPECT_FALSE(CheckTransactionWithoutProofVerification(newTx, state));
|
|
||||||
EXPECT_EQ(state.GetRejectReason(), "bad-txns-vpubs-both-nonzero");
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
class MockCValidationState : public CValidationState {
|
class MockCValidationState : public CValidationState {
|
||||||
public:
|
public:
|
||||||
@@ -47,56 +28,6 @@ public:
|
|||||||
MOCK_CONST_METHOD0(GetRejectReason, std::string());
|
MOCK_CONST_METHOD0(GetRejectReason, std::string());
|
||||||
};
|
};
|
||||||
|
|
||||||
void CreateJoinSplitSignature(CMutableTransaction& mtx, uint32_t consensusBranchId);
|
|
||||||
|
|
||||||
CMutableTransaction GetValidTransaction() {
|
|
||||||
uint32_t consensusBranchId = SPROUT_BRANCH_ID;
|
|
||||||
|
|
||||||
CMutableTransaction mtx;
|
|
||||||
mtx.vin.resize(2);
|
|
||||||
mtx.vin[0].prevout.hash = uint256S("0000000000000000000000000000000000000000000000000000000000000001");
|
|
||||||
mtx.vin[0].prevout.n = 0;
|
|
||||||
mtx.vin[1].prevout.hash = uint256S("0000000000000000000000000000000000000000000000000000000000000002");
|
|
||||||
mtx.vin[1].prevout.n = 0;
|
|
||||||
mtx.vout.resize(2);
|
|
||||||
// mtx.vout[0].scriptPubKey =
|
|
||||||
mtx.vout[0].nValue = 0;
|
|
||||||
mtx.vout[1].nValue = 0;
|
|
||||||
mtx.vjoinsplit.resize(2);
|
|
||||||
mtx.vjoinsplit[0].nullifiers.at(0) = uint256S("0000000000000000000000000000000000000000000000000000000000000000");
|
|
||||||
mtx.vjoinsplit[0].nullifiers.at(1) = uint256S("0000000000000000000000000000000000000000000000000000000000000001");
|
|
||||||
mtx.vjoinsplit[1].nullifiers.at(0) = uint256S("0000000000000000000000000000000000000000000000000000000000000002");
|
|
||||||
mtx.vjoinsplit[1].nullifiers.at(1) = uint256S("0000000000000000000000000000000000000000000000000000000000000003");
|
|
||||||
|
|
||||||
CreateJoinSplitSignature(mtx, consensusBranchId);
|
|
||||||
return mtx;
|
|
||||||
}
|
|
||||||
|
|
||||||
void CreateJoinSplitSignature(CMutableTransaction& mtx, uint32_t consensusBranchId) {
|
|
||||||
// Generate an ephemeral keypair.
|
|
||||||
uint256 joinSplitPubKey;
|
|
||||||
unsigned char joinSplitPrivKey[crypto_sign_SECRETKEYBYTES];
|
|
||||||
crypto_sign_keypair(joinSplitPubKey.begin(), joinSplitPrivKey);
|
|
||||||
mtx.joinSplitPubKey = joinSplitPubKey;
|
|
||||||
|
|
||||||
// Compute the correct hSig.
|
|
||||||
// TODO: #966.
|
|
||||||
static const uint256 one(uint256S("0000000000000000000000000000000000000000000000000000000000000001"));
|
|
||||||
// Empty output script.
|
|
||||||
CScript scriptCode;
|
|
||||||
CTransaction signTx(mtx);
|
|
||||||
uint256 dataToBeSigned = SignatureHash(scriptCode, signTx, NOT_AN_INPUT, SIGHASH_ALL, 0, consensusBranchId);
|
|
||||||
if (dataToBeSigned == one) {
|
|
||||||
throw std::runtime_error("SignatureHash failed");
|
|
||||||
}
|
|
||||||
|
|
||||||
// Add the signature
|
|
||||||
assert(crypto_sign_detached(&mtx.joinSplitSig[0], NULL,
|
|
||||||
dataToBeSigned.begin(), 32,
|
|
||||||
joinSplitPrivKey
|
|
||||||
) == 0);
|
|
||||||
}
|
|
||||||
|
|
||||||
TEST(checktransaction_tests, valid_transaction) {
|
TEST(checktransaction_tests, valid_transaction) {
|
||||||
CMutableTransaction mtx = GetValidTransaction();
|
CMutableTransaction mtx = GetValidTransaction();
|
||||||
CTransaction tx(mtx);
|
CTransaction tx(mtx);
|
||||||
@@ -515,42 +446,6 @@ TEST(checktransaction_tests, bad_txns_invalid_joinsplit_signature) {
|
|||||||
ContextualCheckTransaction(0,tx, state, 0, 100, []() { return false; });
|
ContextualCheckTransaction(0,tx, state, 0, 100, []() { return false; });
|
||||||
}
|
}
|
||||||
|
|
||||||
TEST(checktransaction_tests, non_canonical_ed25519_signature) {
|
|
||||||
SelectParams(CBaseChainParams::REGTEST);
|
|
||||||
|
|
||||||
CMutableTransaction mtx = GetValidTransaction();
|
|
||||||
|
|
||||||
// Check that the signature is valid before we add L
|
|
||||||
{
|
|
||||||
CTransaction tx(mtx);
|
|
||||||
MockCValidationState state;
|
|
||||||
EXPECT_TRUE(ContextualCheckTransaction(0,tx, state, 0, 100));
|
|
||||||
}
|
|
||||||
|
|
||||||
// Copied from libsodium/crypto_sign/ed25519/ref10/open.c
|
|
||||||
static const unsigned char L[32] =
|
|
||||||
{ 0xed, 0xd3, 0xf5, 0x5c, 0x1a, 0x63, 0x12, 0x58,
|
|
||||||
0xd6, 0x9c, 0xf7, 0xa2, 0xde, 0xf9, 0xde, 0x14,
|
|
||||||
0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
|
|
||||||
0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x10 };
|
|
||||||
|
|
||||||
// Add L to S, which starts at mtx.joinSplitSig[32].
|
|
||||||
unsigned int s = 0;
|
|
||||||
for (size_t i = 0; i < 32; i++) {
|
|
||||||
s = mtx.joinSplitSig[32 + i] + L[i] + (s >> 8);
|
|
||||||
mtx.joinSplitSig[32 + i] = s & 0xff;
|
|
||||||
}
|
|
||||||
|
|
||||||
CTransaction tx(mtx);
|
|
||||||
|
|
||||||
MockCValidationState state;
|
|
||||||
// during initial block download, DoS ban score should be zero, else 100
|
|
||||||
EXPECT_CALL(state, DoS(0, false, REJECT_INVALID, "bad-txns-invalid-joinsplit-signature", false)).Times(1);
|
|
||||||
ContextualCheckTransaction(0,tx, state, 0, 100, []() { return true; });
|
|
||||||
EXPECT_CALL(state, DoS(100, false, REJECT_INVALID, "bad-txns-invalid-joinsplit-signature", false)).Times(1);
|
|
||||||
ContextualCheckTransaction(0,tx, state, 0, 100, []() { return false; });
|
|
||||||
}
|
|
||||||
|
|
||||||
TEST(checktransaction_tests, OverwinterConstructors) {
|
TEST(checktransaction_tests, OverwinterConstructors) {
|
||||||
CMutableTransaction mtx;
|
CMutableTransaction mtx;
|
||||||
mtx.fOverwintered = true;
|
mtx.fOverwintered = true;
|
||||||
|
|||||||
Reference in New Issue
Block a user