Skip to content

Commit 9e2946b

Browse files
authored
Remove Python smoketests (#4896)
# Description of Changes Remove the Python smoketests and the CI check that tests for edits. # API and ABI breaking changes None. CI only. # Expected complexity level and risk 1 # Testing - [ ] All CI passes Co-authored-by: Zeke Foppa <bfops@users.noreply.github.com>
1 parent 3b28744 commit 9e2946b

43 files changed

Lines changed: 0 additions & 6765 deletions

Some content is hidden

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

.github/workflows/ci.yml

Lines changed: 0 additions & 30 deletions
Original file line numberDiff line numberDiff line change
@@ -1025,36 +1025,6 @@ jobs:
10251025
- name: Check global.json policy
10261026
run: cargo ci global-json-policy
10271027

1028-
warn-python-smoketests:
1029-
name: Check for Python smoketest edits
1030-
runs-on: ubuntu-latest
1031-
if: github.event_name == 'pull_request'
1032-
permissions:
1033-
contents: read
1034-
steps:
1035-
- name: Checkout sources
1036-
uses: actions/checkout@v4
1037-
with:
1038-
fetch-depth: 0
1039-
1040-
- name: Fail if Python smoketests were modified
1041-
run: |
1042-
MERGE_BASE="$(git merge-base origin/${{ github.base_ref }} HEAD)"
1043-
PYTHON_SMOKETEST_CHANGES="$(git diff --name-only "$MERGE_BASE" HEAD -- 'smoketests/**.py')"
1044-
1045-
if [ -n "$PYTHON_SMOKETEST_CHANGES" ]; then
1046-
echo "::error::This PR modifies legacy Python smoketests. Please add new tests to the Rust smoketests in crates/smoketests/ instead."
1047-
echo ""
1048-
echo "Changed files:"
1049-
echo "$PYTHON_SMOKETEST_CHANGES"
1050-
echo ""
1051-
echo "The Python smoketests are being replaced by Rust smoketests."
1052-
echo "See crates/smoketests/DEVELOP.md for instructions on adding Rust smoketests."
1053-
exit 1
1054-
fi
1055-
1056-
echo "No Python smoketest changes detected."
1057-
10581028
smoketests_mod_rs_complete:
10591029
name: Check smoketests/mod.rs is complete
10601030
runs-on: ubuntu-latest

smoketests/README.md

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

0 commit comments

Comments
 (0)