embarklabs
/
embark
mirror of https://github.com/embarklabs/embark.git
2
0
Fork 0
Code Issues Projects Releases Wiki Activity

fix(@embark/tests): fix slow embark test because of the tx-logger 

The transaction-logger was slowing down tests because each Tx, it
would read a file and write to it, but that's slow and even got
slower as the file grew bigger
To fix, I removed the Tx-logger from text, along with the profiler
as they are useless for tests

(cherry picked from commit a656eea7dda83ad970615cffd51cce5d53a1d034)
This commit is contained in:
Jonathan Rainville 2019-12-20 10:46:34 -05:00 committed by Iuri Matias
parent 765e889121
commit 1e9b8d8595
2 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
packages/embark/src/cmd/cmd_controller.js
View File

@ -768,7 +768,12 @@ class EmbarkController {
engine.registerModuleGroup("blockchain");
engine.registerModuleGroup("compiler");
engine.registerModuleGroup("contracts");
engine.registerModulePackage('embark-ganache');
engine.registerModulePackage('embark-ethereum-blockchain-client');
engine.registerModulePackage('embark-web3');
engine.registerModulePackage('embark-accounts-manager');
engine.registerModulePackage('embark-rpc-manager');
engine.registerModulePackage('embark-specialconfigs', { plugins: engine.plugins });
engine.registerModuleGroup("pipeline");
engine.registerModuleGroup("tests", options);
engine.registerModulePackage('embark-deploy-tracker', { plugins: engine.plugins, trackContracts: false });

1
packages/plugins/transaction-logger/src/index.js
View File

@ -47,6 +47,7 @@ export default class TransactionLogger {
});
this.writeLogFile = async.cargo((tasks, callback) => {
// TODO change this to only read once then use memory, because it slows things down a lot to read on each TX
const data = this._readLogs();
tasks.forEach(task => {