Skip to content

Commit bb387c1

Browse files
committed
fix: set demo directory to be default ones
Turns out there's a hidden dependency in CI between the ledger.db and the snapshot tests. While we find a way to make tests aware of the LEDGER_DIR env variable, it seems simpler to just stick to default dirs. Signed-off-by: Arnaud Bailly <arnaud@pankzsoft.com>
1 parent 9b8b3cd commit bb387c1

1 file changed

Lines changed: 3 additions & 5 deletions

File tree

Makefile

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -5,11 +5,9 @@ DEMO_TARGET_EPOCH ?= 174
55
HASKELL_NODE_CONFIG_SOURCE := https://book.world.dev.cardano.org/environments
66
COVERAGE_DIR ?= coverage
77
COVERAGE_CRATES ?=
8-
DB_PREFIX ?= amaru
98
LISTEN_ADDRESS ?= 0.0.0.0:0
10-
11-
LEDGER_DIR = $(DB_PREFIX)-ledger
12-
CHAIN_DIR = $(DB_PREFIX)-chain
9+
LEDGER_DIR ?= ./ledger.db
10+
CHAIN_DIR ?= ./chain.db
1311

1412
.PHONY: help bootstrap run import-snapshots import-headers import-nonces download-haskell-config coverage-html coverage-lconv check-llvm-cov
1513

@@ -96,7 +94,7 @@ coverage-html: check-llvm-cov ## Run test coverage for Amaru
9694
cargo llvm-cov \
9795
--no-cfg-coverage \
9896
--html \
99-
--output-dir $(COVERAGE_DIR) $(foreach package,$(COVERAGE_CRATES), --package $(package))
97+
--output-dir $(COVERAGE_DIR) $(foreach package,$(COVERAGE_CRATES), --package $(package))
10098

10199
coverage-lconv: ## Run test coverage for CI to upload to Codecov
102100
cargo llvm-cov \

0 commit comments

Comments
 (0)