Skip to content

Commit

Permalink
enh: test on pr (#174)
Browse files Browse the repository at this point in the history
  • Loading branch information
cullenwatson authored Jul 19, 2024
1 parent 60d4d91 commit 8678b0b
Show file tree
Hide file tree
Showing 7 changed files with 39 additions and 15 deletions.
22 changes: 22 additions & 0 deletions .github/workflows/python-test.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
name: Python Tests

on:
pull_request:
branches:
- main

jobs:
test:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2
- name: Set up Python
uses: actions/setup-python@v2
with:
python-version: '3.8'
- name: Install dependencies
run: |
pip install poetry
poetry install
- name: Run tests
run: poetry run pytest src/tests/
2 changes: 2 additions & 0 deletions poetry.toml
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
[virtualenvs]
in-project = true
4 changes: 2 additions & 2 deletions src/tests/test_all.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,10 +5,10 @@
def test_all():
result = scrape_jobs(
site_name=["linkedin", "indeed", "zip_recruiter", "glassdoor"],
search_term="software engineer",
search_term="engineer",
results_wanted=5,
)

assert (
isinstance(result, pd.DataFrame) and not result.empty
isinstance(result, pd.DataFrame) and len(result) == 20
), "Result should be a non-empty DataFrame"
8 changes: 5 additions & 3 deletions src/tests/test_glassdoor.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,12 @@
import pandas as pd


def test_indeed():
def test_glassdoor():
result = scrape_jobs(
site_name="glassdoor", search_term="software engineer", country_indeed="USA"
site_name="glassdoor",
search_term="engineer",
results_wanted=5,
)
assert (
isinstance(result, pd.DataFrame) and not result.empty
isinstance(result, pd.DataFrame) and len(result) == 5
), "Result should be a non-empty DataFrame"
6 changes: 4 additions & 2 deletions src/tests/test_indeed.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,10 @@

def test_indeed():
result = scrape_jobs(
site_name="indeed", search_term="software engineer", country_indeed="usa"
site_name="indeed",
search_term="engineer",
results_wanted=5,
)
assert (
isinstance(result, pd.DataFrame) and not result.empty
isinstance(result, pd.DataFrame) and len(result) == 5
), "Result should be a non-empty DataFrame"
7 changes: 2 additions & 5 deletions src/tests/test_linkedin.py
Original file line number Diff line number Diff line change
Expand Up @@ -3,10 +3,7 @@


def test_linkedin():
result = scrape_jobs(
site_name="linkedin",
search_term="software engineer",
)
result = scrape_jobs(site_name="linkedin", search_term="engineer", results_wanted=5)
assert (
isinstance(result, pd.DataFrame) and not result.empty
isinstance(result, pd.DataFrame) and len(result) == 5
), "Result should be a non-empty DataFrame"
5 changes: 2 additions & 3 deletions src/tests/test_ziprecruiter.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,9 @@

def test_ziprecruiter():
result = scrape_jobs(
site_name="zip_recruiter",
search_term="software engineer",
site_name="zip_recruiter", search_term="software engineer", results_wanted=5
)

assert (
isinstance(result, pd.DataFrame) and not result.empty
isinstance(result, pd.DataFrame) and len(result) == 5
), "Result should be a non-empty DataFrame"

0 comments on commit 8678b0b

Please sign in to comment.