fix all the imports, get it all working

hjwp · hjwp · commit 5183cf62f209 · 2020-01-08T20:41:36.000Z
diff --git a/docker-compose.yml b/docker-compose.yml
@@ -11,7 +11,7 @@ services:
       - DB_HOST=postgres
       - DB_PASSWORD=abc123
     volumes:
-      - ./:/code
+      - ./src:/src
     ports:
       - "5005:80"
 
diff --git a/mypy.ini b/mypy.ini
@@ -1,9 +1,6 @@
 [mypy]
 ignore_missing_imports = False
+mypy_path = ./src
 
-[mypy-pytest.*]
+[mypy-pytest.*,sqlalchemy.*]
 ignore_missing_imports = True
-
-[mypy-sqlalchemy.*]
-ignore_missing_imports = True
-
diff --git a/src/allocation/adapters/orm.py b/src/allocation/adapters/orm.py
@@ -4,7 +4,7 @@
 )
 from sqlalchemy.orm import mapper, relationship
 
-from domain import model
+from allocation.domain import model
 
 
 metadata = MetaData()
diff --git a/src/allocation/adapters/repository.py b/src/allocation/adapters/repository.py
@@ -1,5 +1,5 @@
 import abc
-from domain import model
+from allocation.domain import model
 
 
 class AbstractRepository(abc.ABC):
diff --git a/src/allocation/entrypoints/flask_app.py b/src/allocation/entrypoints/flask_app.py
@@ -3,10 +3,10 @@
 from sqlalchemy import create_engine
 from sqlalchemy.orm import sessionmaker
 
-import config
-from domain import model
-from adapters import orm, repository
-from service_layer import services
+from allocation import config
+from allocation.domain import model
+from allocation.adapters import orm, repository
+from allocation.service_layer import services
 
 orm.start_mappers()
 get_session = sessionmaker(bind=create_engine(config.get_postgres_uri()))
diff --git a/src/allocation/service_layer/services.py b/src/allocation/service_layer/services.py
@@ -2,9 +2,9 @@
 from typing import Optional
 from datetime import date
 
-from domain import model
-from domain.model import OrderLine
-from adapters.repository import AbstractRepository
+from allocation.domain import model
+from allocation.domain.model import OrderLine
+from allocation.adapters.repository import AbstractRepository
 
 class InvalidSku(Exception):
     pass
diff --git a/tests/conftest.py b/tests/conftest.py
@@ -9,8 +9,8 @@
 from sqlalchemy import create_engine
 from sqlalchemy.orm import sessionmaker, clear_mappers
 
-from adapters.orm import metadata, start_mappers
-import config
+from allocation.adapters.orm import metadata, start_mappers
+from allocation import config
 
 
 @pytest.fixture
@@ -66,7 +66,6 @@ def postgres_session(postgres_db):
 
 @pytest.fixture
 def restart_api():
-    (Path(__file__).parent / '../entrypoints/flask_app.py').touch()
+    (Path(__file__).parent / '../src/allocation/entrypoints/flask_app.py').touch()
     time.sleep(0.5)
     wait_for_webapp_to_come_up()
-
diff --git a/tests/e2e/test_api.py b/tests/e2e/test_api.py
@@ -2,7 +2,7 @@
 import pytest
 import requests
 
-import config
+from allocation import config
 
 def random_suffix():
     return uuid.uuid4().hex[:6]
diff --git a/tests/integration/test_orm.py b/tests/integration/test_orm.py
@@ -1,4 +1,4 @@
-from domain import model
+from allocation.domain import model
 from datetime import date
 
 def test_orderline_mapper_can_load_lines(session):
diff --git a/tests/integration/test_repository.py b/tests/integration/test_repository.py
@@ -1,6 +1,6 @@
 # pylint: disable=protected-access
-from domain import model
-from adapters import repository
+from allocation.domain import model
+from allocation.adapters import repository
 
 def test_repository_can_save_a_batch(session):
     batch = model.Batch("batch1", "RUSTY-SOAPDISH", 100, eta=None)
diff --git a/tests/mypy.ini b/tests/mypy.ini
@@ -0,0 +1,6 @@
+[mypy]
+ignore_missing_imports = False
+mypy_path = ../src
+
+[mypy-pytest.*,sqlalchemy.*]
+ignore_missing_imports = True
diff --git a/tests/unit/test_allocate.py b/tests/unit/test_allocate.py
@@ -1,6 +1,6 @@
 from datetime import date, timedelta
 import pytest
-from domain.model import allocate, OrderLine, Batch, OutOfStock
+from allocation.domain.model import allocate, OrderLine, Batch, OutOfStock
 
 today = date.today()
 tomorrow = today + timedelta(days=1)
diff --git a/tests/unit/test_batches.py b/tests/unit/test_batches.py
@@ -1,5 +1,5 @@
 from datetime import date
-from domain.model import Batch, OrderLine
+from allocation.domain.model import Batch, OrderLine
 
 
 def test_allocating_to_a_batch_reduces_the_available_quantity():
diff --git a/tests/unit/test_services.py b/tests/unit/test_services.py
@@ -1,7 +1,7 @@
 import pytest
-from domain import model
-from adapters import repository
-from service_layer import services
+from allocation.domain import model
+from allocation.adapters import repository
+from allocation.service_layer import services
 
 
 class FakeRepository(repository.AbstractRepository):

Original file line number	Diff line number	Diff line change
`@@ -4,7 +4,7 @@`
`4`	`4`	`)`
`5`	`5`	`from sqlalchemy.orm import mapper, relationship`
`6`	`6`
`7`		`-from domain import model`
	`7`	`+from allocation.domain import model`
`8`	`8`
`9`	`9`
`10`	`10`	`metadata = MetaData()`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-from domain import model`
	`1`	`+from allocation.domain import model`
`2`	`2`	`from datetime import date`
`3`	`3`
`4`	`4`	`def test_orderline_mapper_can_load_lines(session):`