Skip to content

Commit a66cb8c

Browse files
grandcamelclaude
andcommitted
chore(tests): remove live integration tests migrated to splunk-as
Live integration tests have been moved to the splunk-as package. This removes the duplicate test infrastructure and updates documentation to reference splunk-as for live testing. - Delete 14 live_integration directories (~5700 lines) - Update pytest.ini to remove live-test markers and pythonpath - Remove test-live target from Makefile - Remove --ignore flags from CI workflows - Update TESTING.md, DEVELOPMENT.md, ARCHITECTURE.md Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
1 parent b6202d5 commit a66cb8c

35 files changed

Lines changed: 26 additions & 5752 deletions

File tree

.github/workflows/release.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ jobs:
4444
- name: Run unit tests
4545
run: |
4646
pip install pytest
47-
pytest skills/*/tests/ -v --ignore=skills/*/tests/live_integration/
47+
pytest skills/*/tests/ -v
4848
4949
publish-release:
5050
needs: [release-please, test]
@@ -67,7 +67,7 @@ jobs:
6767
- name: Run tests
6868
run: |
6969
pip install pytest pytest-asyncio
70-
pytest skills/*/tests/ -v --ignore=skills/*/tests/live_integration/
70+
pytest skills/*/tests/ -v
7171
7272
- name: Create release archive
7373
run: |

.github/workflows/validate.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ jobs:
2727
pip install pytest
2828
2929
- name: Run unit tests
30-
run: pytest skills/*/tests/ -v --ignore=skills/*/tests/live_integration/
30+
run: pytest skills/*/tests/ -v
3131

3232
validate-docs:
3333
runs-on: ubuntu-latest

Makefile

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@ help:
88
@echo "Targets:"
99
@echo " install Install dependencies"
1010
@echo " test Run unit tests"
11-
@echo " test-live Run live integration tests (requires Splunk)"
1211
@echo " lint Run linting (black, isort)"
1312
@echo " lint-fix Fix linting issues automatically"
1413
@echo " validate-docs Validate CLI documentation matches splunk-as"
@@ -21,10 +20,7 @@ install:
2120
pip install pytest black isort pre-commit
2221

2322
test:
24-
pytest skills/*/tests/ -v --ignore=skills/*/tests/live_integration/
25-
26-
test-live:
27-
pytest skills/*/tests/live_integration/ -v -m "live"
23+
pytest skills/*/tests/ -v
2824

2925
lint:
3026
black --check --diff skills/ scripts/

docs/ARCHITECTURE.md

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -119,12 +119,11 @@ skills/{skill-name}/
119119
├── tests/ # Unit tests
120120
│ ├── conftest.py # Skill-specific fixtures
121121
│ └── test_*.py
122-
├── tests/live_integration/ # Integration tests
123-
│ ├── conftest.py # Live test fixtures
124-
│ └── test_*.py
125122
└── references/ # API docs, examples
126123
```
127124

125+
**Note:** Live integration tests are in the [splunk-as](https://github.com/grandcamel/splunk-as) package at `tests/live/`.
126+
128127
### Plugin Manifest
129128

130129
The `.claude-plugin/plugin.json` defines the plugin structure. Skills are autodiscovered via `skills/*/SKILL.md`:

docs/DEVELOPMENT.md

Lines changed: 3 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -63,9 +63,6 @@ skills/new-skill/
6363
├── tests/ # Unit tests
6464
│ ├── conftest.py # Skill-specific fixtures only (see note below)
6565
│ └── test_*.py
66-
├── tests/live_integration/ # Integration tests
67-
│ ├── conftest.py # Live test fixtures
68-
│ └── test_*.py
6966
└── references/ # API docs, examples
7067
```
7168

@@ -183,12 +180,10 @@ def main():
183180
- Mock external dependencies
184181
- Use fixtures for common setup
185182

186-
### Integration Tests
183+
### Live Integration Tests
187184

188-
- Mark with `@pytest.mark.live`
189-
- Use `@pytest.mark.destructive` for tests that modify data
190-
- Clean up after tests
191-
- Use `fresh_test_data` fixture for isolation
185+
Live integration tests are now in the [splunk-as](https://github.com/grandcamel/splunk-as) package.
186+
See `splunk-as/tests/live/` for integration test examples.
192187

193188
### Running Tests
194189

0 commit comments

Comments
 (0)