Skip to content

Commit 69c9872

Browse files
voidashstoropoli
andauthored
chore: delete legacy functional-tests directory (STR-2085) (#1581)
* chore: delete legacy functional-tests directory (STR-2085) Remove the entire functional-tests/ directory. All valuable tests have been migrated to functional-tests-new/ or their invariants tracked in JIRA/Notion. Sync tests: added test_sync_genesis to functional-tests-new. Fullnode lag restart test deferred — fullnode sync not yet implemented in new strata binary (tracked in STR-2091). Also: - Fix bug in strata_seq_fullnode.py: pass genesis_l1.blk.height (int) not GenesisL1View object to create_node() - Delete .github/workflows/functional.yml (legacy test workflow) - Update test-coverage.yml, main-eest.yml, dependabot.yml, CODEOWNERS - Update AGENTS.md, README.md, CONTRIBUTING.md references * chore: rename functional-tests-new/ to functional-tests/ (STR-2085) Now that the legacy functional-tests/ directory is deleted, drop the -new suffix. Updates all references across CI workflows, justfile, CODEOWNERS, Dockerfile, docs, and .dockerignore. * chore: exclude functional-tests/_dd from taplo formatting Test run data directory contains generated TOML files that should not be linted. * chore: clean up stray references after functional-tests rename (STR-2085) - CODEOWNERS: drop fn_*.py and constants.py patterns that match nothing in the new layout. - dependabot.yml: restore the uv ecosystem entry for functional-tests/. It was dropped with the legacy delete, but the path is unchanged after the rename so the new framework still needs Python dependency updates. - functional.yml: filter the coverage-instrumented cargo build to the binaries the new tests actually use, and update the which assertion from strata-client to strata. - FUNCTIONAL_TEST_MIGRATION_STATUS.md: delete the migration tracking doc now that the legacy tree is gone. * fix: move btcio tests missed during functional-tests rename (STR-2085) The btcio tests (added by PR #1451) were added to main after the rename commit was written. The rebase didn't capture them, leaving orphaned tracked files under functional-tests-new/tests/btcio/. --------- Co-authored-by: Jose Storopoli <jose@storopoli.com>
1 parent 6dcc0fb commit 69c9872

176 files changed

Lines changed: 1727 additions & 10240 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

.dockerignore

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,6 @@
44
.github/
55
.ruff_cache/
66
.vscode/
7-
functional-tests/
87
docker/.data
98

109
# Ignore build artifacts and cache
@@ -14,7 +13,7 @@ target/*
1413

1514
# Ignore test-related files and directories
1615
test/
17-
functional-tests-new/_dd/
16+
functional-tests/_dd/
1817
**/_dd/
1918

2019
# Ignore cache, temporary, and virtual environment files

.github/CODEOWNERS

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -75,18 +75,9 @@ bin/strata/ @alpenlabs/bin @alpenlabs/chain-core
7575

7676
# Functional tests
7777
functional-tests/ @alpenlabs/python # catch all
78-
functional-tests/fn_*.py @alpenlabs/python @alpenlabs/ci
79-
functional-tests/constants.py @alpenlabs/python @alpenlabs/ci
8078
functional-tests/entry.py @alpenlabs/python @alpenlabs/ci
8179
functional-tests/uv.lock @alpenlabs/python
8280

83-
# Functional tests new
84-
functional-tests-new/ @alpenlabs/python # catch all
85-
functional-tests-new/fn_*.py @alpenlabs/python @alpenlabs/ci
86-
functional-tests-new/constants.py @alpenlabs/python @alpenlabs/ci
87-
functional-tests-new/entry.py @alpenlabs/python @alpenlabs/ci
88-
functional-tests-new/uv.lock @alpenlabs/python
89-
9081

9182
# Docker/Containers
9283
**/Docker* @alpenlabs/docker

.github/workflows/functional-new.yml

Lines changed: 0 additions & 178 deletions
This file was deleted.

0 commit comments

Comments
 (0)