[bisq-network/bisq] [WIP] Add API test harness for Linux (#4366)

[Stan]

@ghubstan commented on this pull request.



> +// bitcoin-cli -regtest getbalances
+// bitcoin-cli -regtest getrpcinfo
+
+ at Slf4j
+public class BitcoinDaemon extends AbstractLinuxProcess implements LinuxProcess {
+
+    public BitcoinDaemon(ApiTestConfig config) {
+        super("bitcoind", config);
+    }
+
+    @Override
+    public void start() throws InterruptedException, IOException {
+        String bitcoindCmd = "export LD_LIBRARY_PATH=" + config.berkeleyDbLibPath + ";"
+                + " " + config.bitcoinPath + "/bitcoind"
+                + " -datadir=" + config.bitcoinDatadir
+                + " -daemon";

`export LD_LIBRARY_PATH` is necessary for my bitcoin-core installation.  This issue is something we'll need to sort out for an unknown variety of setups.   

I think `config.bitcoinPath` should point to the directory containing bitcoin-core binaries, not the `bitcoind` and `bitcoin-cli` binaries themselves -- requiring two `apitest` config options instead of one.   On the other hand, my `bitcoin-core` binaries are isolated in my `bitcoin-core` src directory (I never ran `make install`), not a default system `bin` directory where many unrelated executables are installed.  Let's keep discussing...

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/bisq-network/bisq/pull/4366#discussion_r452999082
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20200710/fd2cd358/attachment.html>