Fix more bugs
This commit is contained in:
parent
d9a613b095
commit
62c9b30b89
|
|
@ -186,13 +186,13 @@ test-integration:
|
|||
- export TEST_TARGET=INTEGRATION
|
||||
- docker compose -f flights-domain/docker-compose.yml --env-file $ENV_DEV_FILE down
|
||||
- docker compose -f flights-domain/docker-compose.yml --env-file $ENV_DEV_FILE pull
|
||||
- docker compose -f flights-domain/docker-compose.yml --env-file $ENV_DEV_FILE up --abort-on-container-exit
|
||||
- docker compose -f flights-domain/docker-compose.yml --env-file $ENV_DEV_FILE up --abort-on-container-exit -d
|
||||
|
||||
- export API_IMAGE=$USER_MANAGER_TEST_IMAGE_NAME
|
||||
- export TEST_TARGET=INTEGRATION
|
||||
- docker compose -f auth-domain/docker-compose.yml --env-file $ENV_DEV_FILE down
|
||||
- docker compose -f auth-domain/docker-compose.yml --env-file $ENV_DEV_FILE pull
|
||||
- docker compose -f auth-domain/docker-compose.yml --env-file $ENV_DEV_FILE up --abort-on-container-exit
|
||||
- docker compose -f auth-domain/docker-compose.yml --env-file $ENV_DEV_FILE up --abort-on-container-exit -d
|
||||
needs:
|
||||
- job: test-flights-api
|
||||
- job: test-auth-api
|
||||
|
|
|
|||
|
|
@ -4,7 +4,6 @@
|
|||
|
||||
```
|
||||
pre-commit install
|
||||
pre-commit run --all-files
|
||||
```
|
||||
|
||||
### auth-domain
|
||||
|
|
|
|||
|
|
@ -1,224 +0,0 @@
|
|||
<?xml version="1.0" ?>
|
||||
<coverage version="7.3.2" timestamp="1698083039765" lines-valid="150" lines-covered="81" line-rate="0.54" branches-valid="26" branches-covered="12" branch-rate="0.4615" complexity="0">
|
||||
<!-- Generated by coverage.py: https://coverage.readthedocs.io/en/7.3.2 -->
|
||||
<!-- Based on https://raw.githubusercontent.com/cobertura/web/master/htdocs/xml/coverage-04.dtd -->
|
||||
<sources>
|
||||
<source>/home/slococo/ITBA/MICRO/fids/flights-domain/flights-information/src</source>
|
||||
</sources>
|
||||
<packages>
|
||||
<package name="." line-rate="0" branch-rate="0" complexity="0">
|
||||
<classes>
|
||||
<class name="config.py" filename="config.py" complexity="0" line-rate="0" branch-rate="0">
|
||||
<methods/>
|
||||
<lines>
|
||||
<line number="1" hits="0"/>
|
||||
<line number="4" hits="0"/>
|
||||
<line number="5" hits="0"/>
|
||||
<line number="6" hits="0"/>
|
||||
<line number="7" hits="0"/>
|
||||
<line number="8" hits="0"/>
|
||||
<line number="9" hits="0"/>
|
||||
<line number="12" hits="0"/>
|
||||
<line number="13" hits="0"/>
|
||||
<line number="14" hits="0"/>
|
||||
<line number="17" hits="0"/>
|
||||
<line number="18" hits="0"/>
|
||||
<line number="19" hits="0"/>
|
||||
<line number="20" hits="0"/>
|
||||
<line number="21" hits="0"/>
|
||||
<line number="22" hits="0"/>
|
||||
<line number="25" hits="0"/>
|
||||
<line number="26" hits="0"/>
|
||||
<line number="27" hits="0"/>
|
||||
<line number="28" hits="0"/>
|
||||
<line number="30" hits="0"/>
|
||||
<line number="31" hits="0"/>
|
||||
<line number="32" hits="0"/>
|
||||
<line number="33" hits="0" branch="true" condition-coverage="0% (0/2)" missing-branches="34,35"/>
|
||||
<line number="34" hits="0"/>
|
||||
<line number="35" hits="0"/>
|
||||
</lines>
|
||||
</class>
|
||||
</classes>
|
||||
</package>
|
||||
<package name="api" line-rate="0.6522" branch-rate="1" complexity="0">
|
||||
<classes>
|
||||
<class name="db.py" filename="api/db.py" complexity="0" line-rate="0.7143" branch-rate="1">
|
||||
<methods/>
|
||||
<lines>
|
||||
<line number="1" hits="1"/>
|
||||
<line number="3" hits="1"/>
|
||||
<line number="4" hits="1"/>
|
||||
<line number="5" hits="1"/>
|
||||
<line number="7" hits="1"/>
|
||||
<line number="8" hits="1"/>
|
||||
<line number="10" hits="1"/>
|
||||
<line number="12" hits="1"/>
|
||||
<line number="14" hits="1"/>
|
||||
<line number="17" hits="1"/>
|
||||
<line number="18" hits="0"/>
|
||||
<line number="19" hits="0"/>
|
||||
<line number="20" hits="0"/>
|
||||
<line number="22" hits="0"/>
|
||||
</lines>
|
||||
</class>
|
||||
<class name="main.py" filename="api/main.py" complexity="0" line-rate="0.5556" branch-rate="1">
|
||||
<methods/>
|
||||
<lines>
|
||||
<line number="1" hits="1"/>
|
||||
<line number="2" hits="1"/>
|
||||
<line number="3" hits="1"/>
|
||||
<line number="4" hits="1"/>
|
||||
<line number="6" hits="1"/>
|
||||
<line number="8" hits="0"/>
|
||||
<line number="9" hits="0"/>
|
||||
<line number="10" hits="0"/>
|
||||
<line number="11" hits="0"/>
|
||||
</lines>
|
||||
</class>
|
||||
</classes>
|
||||
</package>
|
||||
<package name="api.cruds" line-rate="0.3704" branch-rate="0" complexity="0">
|
||||
<classes>
|
||||
<class name="flight.py" filename="api/cruds/flight.py" complexity="0" line-rate="0.3704" branch-rate="0">
|
||||
<methods/>
|
||||
<lines>
|
||||
<line number="1" hits="1"/>
|
||||
<line number="2" hits="1"/>
|
||||
<line number="3" hits="1"/>
|
||||
<line number="4" hits="1"/>
|
||||
<line number="7" hits="1"/>
|
||||
<line number="8" hits="0"/>
|
||||
<line number="11" hits="1"/>
|
||||
<line number="12" hits="0"/>
|
||||
<line number="15" hits="1"/>
|
||||
<line number="16" hits="0"/>
|
||||
<line number="25" hits="0"/>
|
||||
<line number="26" hits="0"/>
|
||||
<line number="27" hits="0"/>
|
||||
<line number="28" hits="0"/>
|
||||
<line number="31" hits="1"/>
|
||||
<line number="32" hits="0"/>
|
||||
<line number="33" hits="0" branch="true" condition-coverage="0% (0/2)" missing-branches="34,36"/>
|
||||
<line number="34" hits="0"/>
|
||||
<line number="36" hits="0"/>
|
||||
<line number="37" hits="0"/>
|
||||
<line number="38" hits="0"/>
|
||||
<line number="39" hits="0"/>
|
||||
<line number="40" hits="0"/>
|
||||
<line number="43" hits="1"/>
|
||||
<line number="44" hits="0"/>
|
||||
<line number="47" hits="1"/>
|
||||
<line number="48" hits="0"/>
|
||||
</lines>
|
||||
</class>
|
||||
</classes>
|
||||
</package>
|
||||
<package name="api.models" line-rate="1" branch-rate="1" complexity="0">
|
||||
<classes>
|
||||
<class name="flight.py" filename="api/models/flight.py" complexity="0" line-rate="1" branch-rate="1">
|
||||
<methods/>
|
||||
<lines>
|
||||
<line number="1" hits="1"/>
|
||||
<line number="2" hits="1"/>
|
||||
<line number="3" hits="1"/>
|
||||
<line number="6" hits="1"/>
|
||||
<line number="7" hits="1"/>
|
||||
<line number="9" hits="1"/>
|
||||
<line number="10" hits="1"/>
|
||||
<line number="11" hits="1"/>
|
||||
<line number="12" hits="1"/>
|
||||
<line number="13" hits="1"/>
|
||||
<line number="14" hits="1"/>
|
||||
<line number="15" hits="1"/>
|
||||
<line number="16" hits="1"/>
|
||||
<line number="17" hits="1"/>
|
||||
</lines>
|
||||
</class>
|
||||
</classes>
|
||||
</package>
|
||||
<package name="api.routes" line-rate="0.5588" branch-rate="0.5556" complexity="0">
|
||||
<classes>
|
||||
<class name="flights.py" filename="api/routes/flights.py" complexity="0" line-rate="0.5172" branch-rate="0.5">
|
||||
<methods/>
|
||||
<lines>
|
||||
<line number="1" hits="1"/>
|
||||
<line number="3" hits="1"/>
|
||||
<line number="4" hits="1"/>
|
||||
<line number="5" hits="1"/>
|
||||
<line number="6" hits="1"/>
|
||||
<line number="7" hits="1"/>
|
||||
<line number="9" hits="1"/>
|
||||
<line number="12" hits="1"/>
|
||||
<line number="13" hits="1" branch="true" condition-coverage="100% (2/2)"/>
|
||||
<line number="14" hits="0"/>
|
||||
<line number="15" hits="0" branch="true" condition-coverage="0% (0/2)" missing-branches="16,17"/>
|
||||
<line number="16" hits="0"/>
|
||||
<line number="17" hits="0"/>
|
||||
<line number="20" hits="1"/>
|
||||
<line number="21" hits="1" branch="true" condition-coverage="100% (2/2)"/>
|
||||
<line number="22" hits="0"/>
|
||||
<line number="25" hits="1"/>
|
||||
<line number="26" hits="1" branch="true" condition-coverage="100% (2/2)"/>
|
||||
<line number="27" hits="0"/>
|
||||
<line number="30" hits="1"/>
|
||||
<line number="31" hits="1" branch="true" condition-coverage="100% (2/2)"/>
|
||||
<line number="36" hits="0" branch="true" condition-coverage="0% (0/2)" missing-branches="37,38"/>
|
||||
<line number="37" hits="0"/>
|
||||
<line number="38" hits="0" branch="true" condition-coverage="0% (0/2)" missing-branches="39,41"/>
|
||||
<line number="39" hits="0"/>
|
||||
<line number="41" hits="0"/>
|
||||
<line number="43" hits="0" branch="true" condition-coverage="0% (0/2)" missing-branches="44,46"/>
|
||||
<line number="44" hits="0"/>
|
||||
<line number="46" hits="0"/>
|
||||
</lines>
|
||||
</class>
|
||||
<class name="health.py" filename="api/routes/health.py" complexity="0" line-rate="0.8" branch-rate="1">
|
||||
<methods/>
|
||||
<lines>
|
||||
<line number="1" hits="1"/>
|
||||
<line number="3" hits="1"/>
|
||||
<line number="6" hits="1"/>
|
||||
<line number="7" hits="1" branch="true" condition-coverage="100% (2/2)"/>
|
||||
<line number="8" hits="0"/>
|
||||
</lines>
|
||||
</class>
|
||||
</classes>
|
||||
</package>
|
||||
<package name="api.schemas" line-rate="0.8846" branch-rate="0.5" complexity="0">
|
||||
<classes>
|
||||
<class name="flight.py" filename="api/schemas/flight.py" complexity="0" line-rate="0.8846" branch-rate="0.5">
|
||||
<methods/>
|
||||
<lines>
|
||||
<line number="1" hits="1"/>
|
||||
<line number="3" hits="1"/>
|
||||
<line number="6" hits="1"/>
|
||||
<line number="7" hits="1"/>
|
||||
<line number="8" hits="1"/>
|
||||
<line number="9" hits="1"/>
|
||||
<line number="10" hits="1"/>
|
||||
<line number="11" hits="1"/>
|
||||
<line number="12" hits="1"/>
|
||||
<line number="13" hits="1"/>
|
||||
<line number="14" hits="1"/>
|
||||
<line number="18" hits="1"/>
|
||||
<line number="19" hits="1" branch="true" condition-coverage="100% (2/2)"/>
|
||||
<line number="20" hits="0" branch="true" condition-coverage="0% (0/2)" missing-branches="21,22"/>
|
||||
<line number="21" hits="0"/>
|
||||
<line number="22" hits="0"/>
|
||||
<line number="25" hits="1"/>
|
||||
<line number="26" hits="1"/>
|
||||
<line number="27" hits="1"/>
|
||||
<line number="28" hits="1"/>
|
||||
<line number="29" hits="1"/>
|
||||
<line number="30" hits="1"/>
|
||||
<line number="31" hits="1"/>
|
||||
<line number="32" hits="1"/>
|
||||
<line number="35" hits="1"/>
|
||||
<line number="36" hits="1"/>
|
||||
</lines>
|
||||
</class>
|
||||
</classes>
|
||||
</package>
|
||||
</packages>
|
||||
</coverage>
|
||||
|
|
@ -1,88 +0,0 @@
|
|||
<?xml version="1.0" encoding="utf-8"?><testsuites><testsuite name="pytest" errors="1" failures="0" skipped="0" tests="1" time="0.827" timestamp="2023-10-23T14:43:56.711153" hostname="satella"><testcase classname="" name="dummy_test" time="0.000"><error message="collection failure">/home/slococo/.conda/envs/flights/lib/python3.11/site-packages/sqlalchemy/engine/base.py:145: in __init__
|
||||
self._dbapi_connection = engine.raw_connection()
|
||||
/home/slococo/.conda/envs/flights/lib/python3.11/site-packages/sqlalchemy/engine/base.py:3292: in raw_connection
|
||||
return self.pool.connect()
|
||||
/home/slococo/.conda/envs/flights/lib/python3.11/site-packages/sqlalchemy/pool/base.py:452: in connect
|
||||
return _ConnectionFairy._checkout(self)
|
||||
/home/slococo/.conda/envs/flights/lib/python3.11/site-packages/sqlalchemy/pool/base.py:1269: in _checkout
|
||||
fairy = _ConnectionRecord.checkout(pool)
|
||||
/home/slococo/.conda/envs/flights/lib/python3.11/site-packages/sqlalchemy/pool/base.py:716: in checkout
|
||||
rec = pool._do_get()
|
||||
/home/slococo/.conda/envs/flights/lib/python3.11/site-packages/sqlalchemy/pool/impl.py:169: in _do_get
|
||||
with util.safe_reraise():
|
||||
/home/slococo/.conda/envs/flights/lib/python3.11/site-packages/sqlalchemy/util/langhelpers.py:146: in __exit__
|
||||
raise exc_value.with_traceback(exc_tb)
|
||||
/home/slococo/.conda/envs/flights/lib/python3.11/site-packages/sqlalchemy/pool/impl.py:167: in _do_get
|
||||
return self._create_connection()
|
||||
/home/slococo/.conda/envs/flights/lib/python3.11/site-packages/sqlalchemy/pool/base.py:393: in _create_connection
|
||||
return _ConnectionRecord(self)
|
||||
/home/slococo/.conda/envs/flights/lib/python3.11/site-packages/sqlalchemy/pool/base.py:678: in __init__
|
||||
self.__connect()
|
||||
/home/slococo/.conda/envs/flights/lib/python3.11/site-packages/sqlalchemy/pool/base.py:902: in __connect
|
||||
with util.safe_reraise():
|
||||
/home/slococo/.conda/envs/flights/lib/python3.11/site-packages/sqlalchemy/util/langhelpers.py:146: in __exit__
|
||||
raise exc_value.with_traceback(exc_tb)
|
||||
/home/slococo/.conda/envs/flights/lib/python3.11/site-packages/sqlalchemy/pool/base.py:898: in __connect
|
||||
self.dbapi_connection = connection = pool._invoke_creator(self)
|
||||
/home/slococo/.conda/envs/flights/lib/python3.11/site-packages/sqlalchemy/engine/create.py:637: in connect
|
||||
return dialect.connect(*cargs, **cparams)
|
||||
/home/slococo/.conda/envs/flights/lib/python3.11/site-packages/sqlalchemy/engine/default.py:616: in connect
|
||||
return self.loaded_dbapi.connect(*cargs, **cparams)
|
||||
/home/slococo/.conda/envs/flights/lib/python3.11/site-packages/psycopg2/__init__.py:122: in connect
|
||||
conn = _connect(dsn, connection_factory=connection_factory, **kwasync)
|
||||
E psycopg2.OperationalError: could not translate host name "usermanager-db" to address: Name or service not known
|
||||
|
||||
The above exception was the direct cause of the following exception:
|
||||
src/tests/dummy_test.py:3: in <module>
|
||||
from src.api.main import app
|
||||
src/api/main.py:6: in <module>
|
||||
Base.metadata.create_all(bind=engine)
|
||||
/home/slococo/.conda/envs/flights/lib/python3.11/site-packages/sqlalchemy/sql/schema.py:5828: in create_all
|
||||
bind._run_ddl_visitor(
|
||||
/home/slococo/.conda/envs/flights/lib/python3.11/site-packages/sqlalchemy/engine/base.py:3242: in _run_ddl_visitor
|
||||
with self.begin() as conn:
|
||||
/home/slococo/.conda/envs/flights/lib/python3.11/contextlib.py:137: in __enter__
|
||||
return next(self.gen)
|
||||
/home/slococo/.conda/envs/flights/lib/python3.11/site-packages/sqlalchemy/engine/base.py:3232: in begin
|
||||
with self.connect() as conn:
|
||||
/home/slococo/.conda/envs/flights/lib/python3.11/site-packages/sqlalchemy/engine/base.py:3268: in connect
|
||||
return self._connection_cls(self)
|
||||
/home/slococo/.conda/envs/flights/lib/python3.11/site-packages/sqlalchemy/engine/base.py:147: in __init__
|
||||
Connection._handle_dbapi_exception_noconnection(
|
||||
/home/slococo/.conda/envs/flights/lib/python3.11/site-packages/sqlalchemy/engine/base.py:2430: in _handle_dbapi_exception_noconnection
|
||||
raise sqlalchemy_exception.with_traceback(exc_info[2]) from e
|
||||
/home/slococo/.conda/envs/flights/lib/python3.11/site-packages/sqlalchemy/engine/base.py:145: in __init__
|
||||
self._dbapi_connection = engine.raw_connection()
|
||||
/home/slococo/.conda/envs/flights/lib/python3.11/site-packages/sqlalchemy/engine/base.py:3292: in raw_connection
|
||||
return self.pool.connect()
|
||||
/home/slococo/.conda/envs/flights/lib/python3.11/site-packages/sqlalchemy/pool/base.py:452: in connect
|
||||
return _ConnectionFairy._checkout(self)
|
||||
/home/slococo/.conda/envs/flights/lib/python3.11/site-packages/sqlalchemy/pool/base.py:1269: in _checkout
|
||||
fairy = _ConnectionRecord.checkout(pool)
|
||||
/home/slococo/.conda/envs/flights/lib/python3.11/site-packages/sqlalchemy/pool/base.py:716: in checkout
|
||||
rec = pool._do_get()
|
||||
/home/slococo/.conda/envs/flights/lib/python3.11/site-packages/sqlalchemy/pool/impl.py:169: in _do_get
|
||||
with util.safe_reraise():
|
||||
/home/slococo/.conda/envs/flights/lib/python3.11/site-packages/sqlalchemy/util/langhelpers.py:146: in __exit__
|
||||
raise exc_value.with_traceback(exc_tb)
|
||||
/home/slococo/.conda/envs/flights/lib/python3.11/site-packages/sqlalchemy/pool/impl.py:167: in _do_get
|
||||
return self._create_connection()
|
||||
/home/slococo/.conda/envs/flights/lib/python3.11/site-packages/sqlalchemy/pool/base.py:393: in _create_connection
|
||||
return _ConnectionRecord(self)
|
||||
/home/slococo/.conda/envs/flights/lib/python3.11/site-packages/sqlalchemy/pool/base.py:678: in __init__
|
||||
self.__connect()
|
||||
/home/slococo/.conda/envs/flights/lib/python3.11/site-packages/sqlalchemy/pool/base.py:902: in __connect
|
||||
with util.safe_reraise():
|
||||
/home/slococo/.conda/envs/flights/lib/python3.11/site-packages/sqlalchemy/util/langhelpers.py:146: in __exit__
|
||||
raise exc_value.with_traceback(exc_tb)
|
||||
/home/slococo/.conda/envs/flights/lib/python3.11/site-packages/sqlalchemy/pool/base.py:898: in __connect
|
||||
self.dbapi_connection = connection = pool._invoke_creator(self)
|
||||
/home/slococo/.conda/envs/flights/lib/python3.11/site-packages/sqlalchemy/engine/create.py:637: in connect
|
||||
return dialect.connect(*cargs, **cparams)
|
||||
/home/slococo/.conda/envs/flights/lib/python3.11/site-packages/sqlalchemy/engine/default.py:616: in connect
|
||||
return self.loaded_dbapi.connect(*cargs, **cparams)
|
||||
/home/slococo/.conda/envs/flights/lib/python3.11/site-packages/psycopg2/__init__.py:122: in connect
|
||||
conn = _connect(dsn, connection_factory=connection_factory, **kwasync)
|
||||
E sqlalchemy.exc.OperationalError: (psycopg2.OperationalError) could not translate host name "usermanager-db" to address: Name or service not known
|
||||
E
|
||||
E (Background on this error at: https://sqlalche.me/e/20/e3q8)</error></testcase></testsuite></testsuites>
|
||||
Loading…
Reference in New Issue