Skip to content

Commit f84b57e

Browse files
authored
fit: refactor (#74)
- simplify directories structure - cleanup unused hooks - clean configs
1 parent 4a653b4 commit f84b57e

27 files changed

Lines changed: 20 additions & 43 deletions

File tree

.github/workflows/build_and_test_on_every_pr.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,4 +53,4 @@ jobs:
5353
done < ci/showcase_targets_run.txt
5454
- name: Feature Integration Tests
5555
run: |
56-
bazel test --config bl-x86_64-linux //feature_integration_tests/python_test_cases:fit
56+
bazel test --config bl-x86_64-linux //feature_integration_tests/test_cases:fit

MODULE.bazel

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ pip = use_extension("@rules_python//python/extensions:pip.bzl", "pip", dev_depen
3737
pip.parse(
3838
hub_name = "pip_score_venv_test",
3939
python_version = PYTHON_VERSION,
40-
requirements_lock = "//feature_integration_tests/python_test_cases:requirements.txt.lock",
40+
requirements_lock = "//feature_integration_tests/test_cases:requirements.txt.lock",
4141
)
4242

4343
use_repo(pip, "pip_score_venv_test")

feature_integration_tests/README.md

Lines changed: 10 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -4,15 +4,16 @@ This directory contains Feature Integration Tests for the S-CORE project. It inc
44

55
## Structure
66

7-
- `python_test_cases/` — Python-based integration test cases
7+
- `test_cases/` — Python-based integration test cases
88
- `conftest.py` — Pytest configuration and fixtures
99
- `fit_scenario.py` — Base scenario class
1010
- `requirements.txt` — Python dependencies
1111
- `BUILD` — Bazel build and test definitions
1212
- `tests/` — Test cases (e.g., orchestration with persistency)
13-
- `rust_test_scenarios/` — Rust-based integration test scenarios
14-
- `src/` — Rust source code for test scenarios
15-
- `BUILD` — Bazel build definitions
13+
- `test_scenarios/` — Location of test scenarios
14+
- `rust/` — Rust-based integration test scenarios
15+
- `src/` — Rust source code for test scenarios
16+
- `BUILD` — Bazel build definitions
1617

1718
## Running Tests
1819

@@ -21,25 +22,25 @@ This directory contains Feature Integration Tests for the S-CORE project. It inc
2122
Python tests are managed with Bazel and Pytest. To run the main test target:
2223

2324
```sh
24-
bazel test //feature_integration_tests/python_test_cases:fit
25+
bazel test //feature_integration_tests/test_cases:fit
2526
```
2627

2728
### Rust Test Scenarios
2829

29-
Rust test scenarios are defined in `rust_test_scenarios/src/scenarios`. Build and run them using Bazel:
30+
Rust test scenarios are defined in `test_scenarios/rust/src/scenarios`. Build and run them using Bazel:
3031

3132
```sh
32-
bazel build //feature_integration_tests/rust_test_scenarios
33+
bazel build //feature_integration_tests/test_scenarios/rust:rust_test_scenarios
3334
```
3435

3536
```sh
36-
bazel run //feature_integration_tests/rust_test_scenarios -- --list-scenarios
37+
bazel run //feature_integration_tests/test_scenarios/rust:rust_test_scenarios -- --list-scenarios
3738
```
3839

3940
## Updating Python Requirements
4041

4142
To update Python dependencies:
4243

4344
```sh
44-
bazel run //feature_integration_tests/python_test_cases:requirements.update
45+
bazel run //feature_integration_tests/test_cases:requirements.update
4546
```

feature_integration_tests/python_test_cases/pytest.ini

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

feature_integration_tests/python_test_cases/BUILD renamed to feature_integration_tests/test_cases/BUILD

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,11 +31,11 @@ score_py_pytest(
3131
srcs = glob(["tests/**/*.py"]) + ["conftest.py", "fit_scenario.py", "test_properties.py"],
3232
args = [
3333
"--traces=all",
34-
"--rust-target-path=$(rootpath //feature_integration_tests/rust_test_scenarios)",
34+
"--rust-target-path=$(rootpath //feature_integration_tests/test_scenarios/rust:rust_test_scenarios)",
3535
],
3636
data = [
3737
":python_tc_venv",
38-
"//feature_integration_tests/rust_test_scenarios",
38+
"//feature_integration_tests/test_scenarios/rust:rust_test_scenarios",
3939
],
4040
env = {
4141
"RUST_BACKTRACE": "1",

feature_integration_tests/python_test_cases/conftest.py renamed to feature_integration_tests/test_cases/conftest.py

Lines changed: 0 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -64,20 +64,3 @@ def pytest_sessionstart(session):
6464

6565
except Exception as e:
6666
pytest.exit(str(e), returncode=1)
67-
68-
69-
def pytest_collection_modifyitems(items: list[pytest.Function]):
70-
markers_to_process = (
71-
"PartiallyVerifies",
72-
"FullyVerifies",
73-
"Description",
74-
"TestType",
75-
"DerivationTechnique",
76-
)
77-
for item in items:
78-
# Add custom markers info to XML report
79-
for marker in item.iter_markers():
80-
if marker.name not in markers_to_process:
81-
continue
82-
83-
item.user_properties.append((marker.name, marker.args[0]))

feature_integration_tests/python_test_cases/fit_scenario.py renamed to feature_integration_tests/test_cases/fit_scenario.py

File renamed without changes.
Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
[pytest]
2+
addopts = -v
3+
testpaths = tests

feature_integration_tests/python_test_cases/requirements.txt renamed to feature_integration_tests/test_cases/requirements.txt

File renamed without changes.

feature_integration_tests/python_test_cases/requirements.txt.lock renamed to feature_integration_tests/test_cases/requirements.txt.lock

File renamed without changes.

0 commit comments

Comments
 (0)