Skip to content

Commit 312f11e

Browse files
committed
Fix setup for running pytest
Added missing requirements.txt file for dependencies. Fixed incorrect module imports in test files.
1 parent 8c324dc commit 312f11e

File tree

4 files changed

+9
-5
lines changed

4 files changed

+9
-5
lines changed

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -132,7 +132,7 @@ use `tox` (see the configuration in `tox.ini`):
132132
$ tox
133133
```
134134

135-
To run integration tests:
135+
To run integration tests, make sure the Docker daemon is running and then run:
136136

137137
```
138138
$ pytest integration_tests

integration_tests/fixtures.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,15 +20,15 @@
2020
from uuid import uuid4
2121

2222
import click
23-
import prestodb.logging
23+
import logging
2424
import pytest
2525
import requests
2626
from prestodb.client import PrestoQuery, PrestoRequest
2727
from prestodb.constants import DEFAULT_PORT
2828
from prestodb.exceptions import TimeoutError
2929

3030

31-
logger = prestodb.logging.get_logger(__name__)
31+
logger = logging.getLogger(__name__)
3232

3333

3434
def get_latest_release():

integration_tests/test_dbapi.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,10 +11,10 @@
1111
# limitations under the License.
1212
from __future__ import absolute_import, division, print_function
1313

14-
import fixtures
14+
import integration_tests.fixtures as fixtures
1515
import prestodb
1616
import pytest
17-
from fixtures import run_presto
17+
from integration_tests.fixtures import run_presto
1818
from prestodb.transaction import IsolationLevel
1919

2020
TEST_SESSION_PROPERTIES = {

tests/requirements.txt

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
httpretty
2+
requests
3+
requests_kerberos
4+
click

0 commit comments

Comments
 (0)