Merge #114: Fix compilation error when using memory database

faa461d Fix compilation error when using memory database (Daniela Brozzoni)

Pull request description:

  This commit also adds the testing for memory database and key value
  database in CI

  <!-- You can erase any parts of this template not applicable to your Pull Request. -->

  ### Description

  <!-- Describe the purpose of this PR, what's being adding and/or fixed -->

  ### Notes to the reviewers

  <!-- In this section you can include notes directed to the reviewers, like explaining why some parts
  of the PR were done in a specific way -->

  ### Checklists

  #### All Submissions:

  * [x] I've signed all my commits
  * [x] I followed the [contribution guidelines](https://github.com/bitcoindevkit/bdk-cli/blob/master/CONTRIBUTING.md)
  * [x] I ran `cargo fmt` and `cargo clippy` before committing

  #### Bugfixes:

  * [ ] This pull request breaks the existing API
  * [x] I've added tests to reproduce the issue which are now passing
  * [ ] I'm linking the issue being fixed by this PR

ACKs for top commit:
  notmandatory:
    ACK faa461d

Tree-SHA512: 6e841b0e7cc73145a27ba18ff5e02d08cf1c716cf4b12c46831642bbc26a4d3a411905e765a16b22137654e3d6451492ecea9a5c76a4a9949e574d60f7971475

Author: notmandatory

.github/workflows/cont_integration.yml

@@ -14,17 +14,19 @@ jobs:
           - 1.56.0 # MSRV
         features:
           - default
+          - electrum,sqlite-db
+          - electrum,key-value-db
           - electrum
-          - esplora-ureq
-          - esplora-reqwest
-          - compiler
-          - compact_filters
-          - rpc
-          - reserves,electrum
-          - electrum,verify
+          - esplora-ureq,sqlite-db
+          - esplora-reqwest,sqlite-db
+          - compiler,sqlite-db
+          - compact_filters,sqlite-db
+          - rpc,sqlite-db
+          - reserves,electrum,sqlite-db
+          - electrum,verify,sqlite-db
           # regtest-* features are experimental and not fully usable
-          - regtest-bitcoin
-          - regtest-electrum
+          - regtest-bitcoin,sqlite-db
+          - regtest-electrum,sqlite-db
 
     steps:
       - name: Checkout
@@ -47,11 +49,11 @@ jobs:
           override: true
           components: rustfmt, clippy
       - name: Build
-        run: cargo build --features ${{ matrix.features }} --locked
+        run: cargo build --no-default-features --features repl,${{ matrix.features }} --locked
       - name: Clippy
         run: cargo clippy -- -D warnings
       - name: Test
-        run: cargo test --features ${{ matrix.features }}
+        run: cargo test --no-default-features --features repl,${{ matrix.features }}
 
   fmt:
     name: Rust fmt

src/utils.rs

@@ -183,6 +183,9 @@ pub(crate) fn open_database(wallet_opts: &WalletOpts) -> Result<AnyDatabase, Err
             .into_string()
             .expect("path string"),
     });
+    #[cfg(not(any(feature = "key-value-db", feature = "sqlite-db")))]
+    let config = AnyDatabaseConfig::Memory(());
+
     let database = AnyDatabase::from_config(&config)?;
     log::debug!("database opened successfully");
     Ok(database)