diff --git a/src/js/remote.js b/src/js/remote.js index 085a27985..b363e6068 100644 --- a/src/js/remote.js +++ b/src/js/remote.js @@ -821,20 +821,37 @@ Remote.prototype.request_account_offers = function (accountID, account_index, cu .ledger_choose(current); }; -Remote.prototype.request_account_tx = function (accountID, ledger_min, ledger_max) { + +/* + account: account, + ledger_index_min: ledger_index, // optional, defaults to -1 if ledger_index_max is specified. + ledger_index_max: ledger_index, // optional, defaults to -1 if ledger_index_min is specified. + binary: boolean, // optional, defaults to false + count: boolean, // optional, defaults to false + descending: boolean, // optional, defaults to false + offset: integer, // optional, defaults to 0 + limit: integer // optional +*/ + +Remote.prototype.request_account_tx = function (obj) { // XXX Does this require the server to be trusted? //utils.assert(this.trusted); var request = new Request(this, 'account_tx'); - request.message.account = accountID; + request.message.account = obj.account; - if (ledger_min === ledger_max) { - request.message.ledger = ledger_min; + if (false && ledger_min === ledger_max) { + //request.message.ledger = ledger_min; } else { - request.message.ledger_min = ledger_min; - request.message.ledger_max = ledger_max; + if (obj.ledger_index_min) {request.message.ledger_index_min = obj.ledger_index_min;} + if (obj.ledger_index_max) {request.message.ledger_index_max = obj.ledger_index_max;} + if (obj.binary) {request.message.binary = obj.binary;} + if (obj.count) {request.message.count = obj.count;} + if (obj.descending) {request.message.descending = obj.descending;} + if (obj.offset) {request.message.offset = obj.offset;} + if (obj.limit) {request.message.limit = obj.limit;} } return request; diff --git a/test/account_tx-test.js b/test/account_tx-test.js new file mode 100644 index 000000000..cdc8b29f5 --- /dev/null +++ b/test/account_tx-test.js @@ -0,0 +1,190 @@ +var async = require("async"); +var buster = require("buster"); + +var Amount = require("../src/js/amount").Amount; +var Remote = require("../src/js/remote").Remote; +var Transaction = require("../src/js/transaction").Transaction; +var Server = require("./server").Server; + +var testutils = require("./testutils"); + +require('../src/js/config').load(require('./config')); + +buster.testRunner.timeout = 250000; //This is a very long test! + + +// Hard-coded limits we'll be testing: +var BINARY_LIMIT = 500; +var NONBINARY_LIMIT = 200; + +var ACCOUNT = "rHb9CJAWyB4rj91VRWn96DkukG4bwdtyTh"; +var FIRST_BATCH = 199; // Within both limits +var OFFSET = 180; +var LIMIT = 170; +var SECOND_BATCH = 10; // Between NONBINARY_LIMIT and BINARY_LIMIT +var THIRD_BATCH = 295; // Exceeds both limits + +buster.testCase("Account_tx tests", { + 'setUp' : testutils.build_setup(), + 'tearDown' : testutils.build_teardown(), + + "make a lot of transactions and query using account_tx" : + function (done) { + var self = this; + var final_create; + + var transactionCounter = 0; + + var createOfferFunction = function (callback) { + self.remote.transaction() + .offer_create("root", "500", "100/USD/root") + .on('proposed', function (m) { + transactionCounter++; + console.log('Submitted transaction', transactionCounter); + callback(m.result !== 'tesSUCCESS'); + }) + .on('final', function (m) { + buster.assert.equals('tesSUCCESS', m.metadata.TransactionResult); + buster.assert(final_create); + }) + .submit(); + }; + + function lotsOfTransactions(number, whenDone) { + var bunchOfOffers = []; + for (var i=0; it2.inLedger || (t1.inLedger==t2.inLedger && t1.hash > t2.hash ), + "Transactions were not ordered correctly: "+t1.inLedger+"#"+t1.hash+" should not have come before "+t2.inLedger+"#"+t2.hash); + } + callback(false); + }) + .on('error', standardErrorHandler(callback)) + .request(); + }, + + + ], function (error) { + buster.refute(error); + finalCallback(); + } + ); + } + } +}); + + + +// TODO: +// Test the "count" feature. \ No newline at end of file