cmu-delphi/delphi-epidata

python client integration test is redundant with server tests

Open

#68 opened on Apr 20, 2020

View on GitHub
 (0 comments) (0 reactions) (0 assignees)Python (67 forks)auto 404
Engineeringgood first issuehelp wantedrefactor

Repository metrics

Stars
 (104 stars)
PR merge metrics
 (PR metrics pending)

Description

the python client integration test consists of hitting the covidcast endpoints. with #67 , the covidcast integration tests will use the python client to hit the server. these are equivalent, and redundant.

it might make more sense for the client integration test to validate non-source-specific behavior (e.g. handling of lists and ranges, etc) instead of particular endpoints.

Contributor guide

Research direction
Identify redundant integration tests in the Python client and propose alternative tests that validate non source specific behavior (e.g., handling of lists and ranges).
Tech stack
python
Domain
backendtesting
Issue type
Test
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
GitPythonTesting
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
80