Skip to content

Commit

Permalink
E2e passing tests (axolotl-ai-cloud#576)
Browse files Browse the repository at this point in the history
* run e2e tests after all other checks have passed

* tweak tests so they get run on PRs or push to main

* change dependent action for chcecking

* one test workflow to rule them all

* no need for custom action, just use needs

* whoops, python version should be a string

* e2e tests can run on any available gpu
  • Loading branch information
winglian committed Sep 15, 2023
1 parent 772666d commit fcaded0
Show file tree
Hide file tree
Showing 3 changed files with 44 additions and 50 deletions.
32 changes: 0 additions & 32 deletions .github/workflows/e2e.yml

This file was deleted.

16 changes: 0 additions & 16 deletions .github/workflows/pre-commit.yml

This file was deleted.

46 changes: 44 additions & 2 deletions .github/workflows/tests.yml
Original file line number Diff line number Diff line change
@@ -1,10 +1,26 @@
name: PyTest
name: Tests
on:
# check on push/merge to main, PRs, and manual triggers
push:
branches:
- "main"
pull_request:
workflow_dispatch:

jobs:
test:
pre-commit:
name: pre-commit
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- uses: actions/setup-python@v4
with:
python-version: "3.9"
cache: 'pip' # caching pip dependencies
- uses: pre-commit/action@v3.0.0

pytest:
name: PyTest
runs-on: ubuntu-latest
strategy:
fail-fast: false
Expand All @@ -30,3 +46,29 @@ jobs:
- name: Run tests
run: |
pytest --ignore=tests/e2e/ tests/
e2e-test:
name: E2E Tests
runs-on: [self-hosted, gpu]
timeout-minutes: 20
needs: [pre-commit, pytest]

steps:
- name: Check out repository code
uses: actions/checkout@v3

- name: Setup Python
uses: actions/setup-python@v4
with:
python-version: "3.10"
cache: 'pip' # caching pip dependencies

- name: Install dependencies
run: |
pip3 install -e .
pip3 install flash-attn
pip3 install -r requirements-tests.txt
- name: Run e2e tests
run: |
pytest tests/e2e/

0 comments on commit fcaded0

Please sign in to comment.