changes to intense block_tests

[oxarbitrage]

changes discussed at #565

• add ltm to update_account_keys test.
• remove use_address support of update_account_keys test as it is not allowed in live chain.
• witness_order_mc_test was commented out as it will fail because the feature is not currently implemented however i decided to keep it commented as we might add this in the future, also the monte carlo approach can help as a guide for other tests.
• the bulk_discount test was removed fully as the feature is not available in the core since a lot, never was live.

[pmconrad]

Any idea why these tests are not in the regular test suite? Just because they take longer to run?

[abitmore]

Read the commit message of cryptonomex/graphene@b134fbe:

Move some unacceptably slow tests to intense_tests
Speeds up chain_tests runs from ~30s to ~5s on my system.

[abitmore]

Should we move them back to regular test suite?

[pmconrad]

On my machine, the modified intense_test takes 6 seconds, chain_test takes 48.
IMO we should move them back.

[oxarbitrage]

ok, moving them back to regular test suite. @pmconrad you think including them in block_tests.cpp is ok or prefer to have a new intense_tests.cpp with them ?

[pmconrad]

block_tests makes the most sense IMO.

[oxarbitrage]

moved.

[pmconrad]

Not sure if you want to keep the PR open for #720... unless you are working on #720 I'd say merge this now and create a new PR for the rest.

[oxarbitrage]

lets merge now if @abitmore agree, #720 in a new pull.

[abitmore]

Please merge this.