feat: TT-356 Read activities with an azure endpoint

Andrés Soto · dsgarcia8 · commit 7a6522741f60 · 2021-10-25T09:51:37.000-05:00
diff --git a/V2/azure_time_tracker/.funcignore b/V2/azure_time_tracker/.funcignore
@@ -0,0 +1,5 @@
+.git*
+.vscode
+local.settings.json
+test
+.venv
diff --git a/V2/azure_time_tracker/.gitignore b/V2/azure_time_tracker/.gitignore
@@ -0,0 +1,130 @@
+# Byte-compiled / optimized / DLL files
+__pycache__/
+*.py[cod]
+*$py.class
+
+# C extensions
+*.so
+
+# Distribution / packaging
+.Python
+build/
+develop-eggs/
+dist/
+downloads/
+eggs/
+.eggs/
+lib/
+lib64/
+parts/
+sdist/
+var/
+wheels/
+pip-wheel-metadata/
+share/python-wheels/
+*.egg-info/
+.installed.cfg
+*.egg
+MANIFEST
+
+# PyInstaller
+#  Usually these files are written by a python script from a template
+#  before PyInstaller builds the exe, so as to inject date/other infos into it.
+*.manifest
+*.spec
+
+# Installer logs
+pip-log.txt
+pip-delete-this-directory.txt
+
+# Unit test / coverage reports
+htmlcov/
+.tox/
+.nox/
+.coverage
+.coverage.*
+.cache
+nosetests.xml
+coverage.xml
+*.cover
+.hypothesis/
+.pytest_cache/
+
+# Translations
+*.mo
+*.pot
+
+# Django stuff:
+*.log
+local_settings.py
+db.sqlite3
+
+# Flask stuff:
+instance/
+.webassets-cache
+
+# Scrapy stuff:
+.scrapy
+
+# Sphinx documentation
+docs/_build/
+
+# PyBuilder
+target/
+
+# Jupyter Notebook
+.ipynb_checkpoints
+
+# IPython
+profile_default/
+ipython_config.py
+
+# pyenv
+.python-version
+
+# pipenv
+#   According to pypa/pipenv#598, it is recommended to include Pipfile.lock in version control.
+#   However, in case of collaboration, if having platform-specific dependencies or dependencies
+#   having no cross-platform support, pipenv may install dependencies that don’t work, or not
+#   install all needed dependencies.
+#Pipfile.lock
+
+# celery beat schedule file
+celerybeat-schedule
+
+# SageMath parsed files
+*.sage.py
+
+# Environments
+.env
+.venv
+env/
+venv/
+ENV/
+env.bak/
+venv.bak/
+
+# Spyder project settings
+.spyderproject
+.spyproject
+
+# Rope project settings
+.ropeproject
+
+# mkdocs documentation
+/site
+
+# mypy
+.mypy_cache/
+.dmypy.json
+dmypy.json
+
+# Pyre type checker
+.pyre/
+
+# Azure Functions artifacts
+bin
+obj
+appsettings.json
+local.settings.json
+.python_packages
diff --git a/V2/azure_time_tracker/activities/__init__.py b/V2/azure_time_tracker/activities/__init__.py
@@ -0,0 +1,20 @@
+from time_tracker.source.use_cases.use_case import get_all, get_by_id
+
+import logging
+import json
+import azure.functions as func
+
+
+def main(req: func.HttpRequest) -> func.HttpResponse:
+    logging.info(
+        'Python HTTP trigger function processed a request to get an activity.'
+    )
+    mimetype = "application/json"
+    id = req.route_params.get('id')
+
+    if id:
+        response = json.dumps(get_by_id(id))
+    else:
+        response = json.dumps(get_all())
+
+    return func.HttpResponse(body=response, status_code=200, mimetype=mimetype)
diff --git a/V2/azure_time_tracker/activities/function.json b/V2/azure_time_tracker/activities/function.json
@@ -0,0 +1,18 @@
+{
+  "scriptFile": "__init__.py",
+  "bindings": [
+    {
+      "authLevel": "anonymous",
+      "type": "httpTrigger",
+      "direction": "in",
+      "name": "req",
+      "methods": [ "get" ],
+      "route": "activities/{id:?}"
+    },
+    {
+      "type": "http",
+      "direction": "out",
+      "name": "$return"
+    }
+  ]
+}
diff --git a/V2/azure_time_tracker/activities/sample.dat b/V2/azure_time_tracker/activities/sample.dat
@@ -0,0 +1,3 @@
+{
+    "name": "Azure"
+}
diff --git a/V2/azure_time_tracker/host.json b/V2/azure_time_tracker/host.json
@@ -0,0 +1,15 @@
+{
+  "version": "2.0",
+  "logging": {
+    "applicationInsights": {
+      "samplingSettings": {
+        "isEnabled": true,
+        "excludedTypes": "Request"
+      }
+    }
+  },
+  "extensionBundle": {
+    "id": "Microsoft.Azure.Functions.ExtensionBundle",
+    "version": "[2.*, 3.0.0)"
+  }
+}
diff --git a/V2/azure_time_tracker/proxies.json b/V2/azure_time_tracker/proxies.json
@@ -0,0 +1,4 @@
+{
+  "$schema": "http://json.schemastore.org/proxies",
+  "proxies": {}
+}
diff --git a/V2/azure_time_tracker/requirements.txt b/V2/azure_time_tracker/requirements.txt
@@ -0,0 +1,5 @@
+# DO NOT include azure-functions-worker in this file
+# The Python Worker is managed by Azure Functions platform
+# Manually managing azure-functions-worker may cause unexpected issues
+
+azure-functions
diff --git a/V2/azure_time_tracker/time_tracker/source/activities_data.json b/V2/azure_time_tracker/time_tracker/source/activities_data.json
diff --git a/V2/azure_time_tracker/time_tracker/source/daos/activities_dao.py b/V2/azure_time_tracker/time_tracker/source/daos/activities_dao.py
diff --git a/V2/azure_time_tracker/time_tracker/source/daos/activities_json_dao.py b/V2/azure_time_tracker/time_tracker/source/daos/activities_json_dao.py
diff --git a/V2/azure_time_tracker/time_tracker/source/dtos/activity.py b/V2/azure_time_tracker/time_tracker/source/dtos/activity.py
diff --git a/V2/azure_time_tracker/time_tracker/source/services/activity_service.py b/V2/azure_time_tracker/time_tracker/source/services/activity_service.py
diff --git a/V2/azure_time_tracker/time_tracker/source/use_cases/use_case.py b/V2/azure_time_tracker/time_tracker/source/use_cases/use_case.py
@@ -0,0 +1,33 @@
+def get_all():
+    activity = [
+        {
+            "name": "Development",
+            "description": "Development",
+            "tenant_id": "cc925a5d-9644-4a4f-8d99-0bee49aadd05",
+            "id": "c61a4a49-3364-49a3-a7f7-0c5f2d15072b",
+            "deleted": "b4327ba6-9f96-49ee-a9ac-3c1edf525172",
+            "status": "active",
+        },
+        {
+            "name": "Management",
+            "description": "description test",
+            "tenant_id": "cc925a5d-9644-4a4f-8d99-0bee49aadd05",
+            "id": "94ec92e2-a500-4700-a9f6-e41eb7b5507c",
+            "deleted": "7cf6efe5-a221-4fe4-b94f-8945127a489a",
+            "status": "active",
+        },
+    ]
+
+    return activity
+
+
+def get_by_id(id):
+
+    return {
+        "name": "Management",
+        "description": "description test",
+        "tenant_id": "cc925a5d-9644-4a4f-8d99-0bee49aadd05",
+        "id": "94ec92e2-a500-4700-a9f6-e41eb7b5507c",
+        "deleted": "7cf6efe5-a221-4fe4-b94f-8945127a489a",
+        "status": "active",
+    }
diff --git a/V2/azure_time_tracker/time_tracker/tests/daos/activities_json_dao_test.py b/V2/azure_time_tracker/time_tracker/tests/daos/activities_json_dao_test.py
@@ -0,0 +1,85 @@
+from V2.source.daos.activities_json_dao import ActivitiesJsonDao
+from V2.source.dtos.activity import Activity
+from faker import Faker
+import json
+import pytest
+import typing
+
+
+@pytest.fixture(name='create_fake_activities')
+def _create_fake_activities(mocker) -> typing.List[Activity]:
+    def _creator(activities):
+        read_data = json.dumps(activities)
+        mocker.patch('builtins.open', mocker.mock_open(read_data=read_data))
+        return [Activity(**activity) for activity in activities]
+
+    return _creator
+
+
+def test_get_by_id__returns_an_activity_dto__when_found_one_activity_that_matches_its_id(
+    create_fake_activities,
+):
+    activities_json_dao = ActivitiesJsonDao(Faker().file_path())
+    activities = create_fake_activities(
+        [
+            {
+                "name": "test_name",
+                "description": "test_description",
+                "tenant_id": "test_tenant_id",
+                "id": "test_id",
+                "deleted": "test_deleted",
+                "status": "test_status",
+            }
+        ]
+    )
+    activity_dto = activities.pop()
+
+    result = activities_json_dao.get_by_id(activity_dto.id)
+
+    assert result == activity_dto
+
+
+def test__get_by_id__returns_none__when_no_activity_matches_its_id(
+    create_fake_activities,
+):
+    activities_json_dao = ActivitiesJsonDao(Faker().file_path())
+    create_fake_activities([])
+
+    result = activities_json_dao.get_by_id(Faker().uuid4())
+
+    assert result == None
+
+
+def test__get_all__returns_a_list_of_activity_dto_objects__when_one_or_more_activities_are_found(
+    create_fake_activities,
+):
+    activities_json_dao = ActivitiesJsonDao(Faker().file_path())
+    number_of_activities = 3
+    activities = create_fake_activities(
+        [
+            {
+                "name": "test_name",
+                "description": "test_description",
+                "tenant_id": "test_tenant_id",
+                "id": "test_id",
+                "deleted": "test_deleted",
+                "status": "test_status",
+            }
+        ]
+        * number_of_activities
+    )
+
+    result = activities_json_dao.get_all()
+
+    assert result == activities
+
+
+def test_get_all__returns_an_empty_list__when_doesnt_found_any_activities(
+    create_fake_activities,
+):
+    activities_json_dao = ActivitiesJsonDao(Faker().file_path())
+    activities = create_fake_activities([])
+
+    result = activities_json_dao.get_all()
+
+    assert result == activities
diff --git a/V2/azure_time_tracker/time_tracker/tests/services/activity_service_test.py b/V2/azure_time_tracker/time_tracker/tests/services/activity_service_test.py
@@ -0,0 +1,28 @@
+from V2.source.services.activity_service import ActivityService
+from faker import Faker
+
+
+def test__get_all__uses_the_activity_dao__to_retrieve_activities(mocker):
+    expected_activities = mocker.Mock()
+    activity_dao = mocker.Mock(
+        get_all=mocker.Mock(return_value=expected_activities)
+    )
+    activity_service = ActivityService(activity_dao)
+
+    actual_activities = activity_service.get_all()
+
+    assert activity_dao.get_all.called
+    assert expected_activities == actual_activities
+
+
+def test__get_by_id__uses_the_activity_dao__to_retrieve_one_activity(mocker):
+    expected_activity = mocker.Mock()
+    activity_dao = mocker.Mock(
+        get_by_id=mocker.Mock(return_value=expected_activity)
+    )
+    activity_service = ActivityService(activity_dao)
+
+    actual_activity = activity_service.get_by_id(Faker().uuid4())
+
+    assert activity_dao.get_by_id.called
+    assert expected_activity == actual_activity

-Original file line number
+Diff line change
@@ @@ -0,0 +1,5 @@ @@
 +.git*
 +.vscode
 +local.settings.json
 +test
 +.venv