From e098561bdd60e1a5e84d6d62b3ef83a31ff3a0a9 Mon Sep 17 00:00:00 2001
From: Prafful Sharma <115104695+DraKen0009@users.noreply.github.com>
Date: Mon, 9 Dec 2024 19:46:35 +0530
Subject: [PATCH] Fixing tests and adding workflows (#3)

---
 .github/workflows/test-plugin-integration.yml | 56 +++++++++++++++++
 camera/migrations/0001_initial.py             |  6 +-
 ...121_1505.py => 0002_auto_20241206_0242.py} |  5 +-
 ...er_positionpreset_created_date_and_more.py | 23 +++++++
 {tests => camera/tests}/__init__.py           |  0
 .../tests}/test_camera_preset_apis.py         | 63 ++++++++++++++++++-
 setup.py                                      |  4 +-
 tests/test_onvif_validations.py               | 52 ---------------
 8 files changed, 148 insertions(+), 61 deletions(-)
 create mode 100644 .github/workflows/test-plugin-integration.yml
 rename camera/migrations/{0002_auto_20241121_1505.py => 0002_auto_20241206_0242.py} (88%)
 create mode 100644 camera/migrations/0003_alter_positionpreset_created_date_and_more.py
 rename {tests => camera/tests}/__init__.py (100%)
 rename {tests => camera/tests}/test_camera_preset_apis.py (65%)
 delete mode 100644 tests/test_onvif_validations.py

diff --git a/.github/workflows/test-plugin-integration.yml b/.github/workflows/test-plugin-integration.yml
new file mode 100644
index 0000000..167714c
--- /dev/null
+++ b/.github/workflows/test-plugin-integration.yml
@@ -0,0 +1,56 @@
+name: Run Tests with Docker
+
+on:
+  push:
+    branches:
+      - main
+  pull_request:
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Print branch name
+        env:
+          BRANCH_NAME: ${{ github.head_ref || github.ref_name }}
+        run: |
+          echo "Branch name: $BRANCH_NAME"
+        shell: bash
+
+      # Checkout the ohcnetwork/care repository
+      - name: Checkout ohcnetwork/care repository
+        uses: actions/checkout@v3
+        with:
+          repository: DraKen0009/care
+          ref: adding-camera-plugin
+
+
+      # Update the plug_config.py file with the required content
+      - name: Update plug_config.py
+        env:
+          BRANCH_NAME: ${{ github.head_ref || github.ref_name }}
+        run: |
+          cat > ./plug_config.py <<EOL
+          from plugs.manager import PlugManager
+          from plugs.plug import Plug
+          
+          camera_plugin = Plug(
+              name="camera",
+              package_name="git+https://github.com/ohcnetwork/care_camera_asset.git",
+              version="@${BRANCH_NAME}",
+              configs={},
+          )
+          
+          plugs = [camera_plugin]
+          manager = PlugManager(plugs)
+          EOL
+
+      # Build and start Docker containers
+      - name: Build and start Docker containers
+        run: |
+          make up
+
+      # Run Django management command tests
+      - name: Run Django management command tests
+        run: |
+          docker compose exec backend bash -c "python manage.py test camera --keepdb --parallel --shuffle"
\ No newline at end of file
diff --git a/camera/migrations/0001_initial.py b/camera/migrations/0001_initial.py
index 17690ec..0d4b746 100644
--- a/camera/migrations/0001_initial.py
+++ b/camera/migrations/0001_initial.py
@@ -1,4 +1,4 @@
-# Generated by Django 5.1.2 on 2024-11-21 09:35
+# Generated by Django 5.1.3 on 2024-12-05 21:12
 
 import care.utils.models.validators
 import django.db.models.deletion
@@ -22,12 +22,12 @@ class Migration(migrations.Migration):
             fields=[
                 ('id', models.BigAutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')),
                 ('external_id', models.UUIDField(db_index=True, default=uuid.uuid4, unique=True)),
-                ('created_date', models.DateTimeField(auto_now_add=True, db_index=True, null=True)),
-                ('modified_date', models.DateTimeField(auto_now=True, db_index=True, null=True)),
                 ('deleted', models.BooleanField(db_index=True, default=False)),
                 ('name', models.CharField(max_length=255, null=True)),
                 ('position', models.JSONField(validators=[care.utils.models.validators.JSONFieldSchemaValidator({'$schema': 'http://json-schema.org/draft-07/schema#', 'additionalProperties': False, 'properties': {'x': {'type': 'number'}, 'y': {'type': 'number'}, 'zoom': {'type': 'number'}}, 'required': ['x', 'y', 'zoom'], 'type': 'object'})])),
                 ('is_migrated', models.BooleanField(default=False)),
+                ('created_date', models.DateTimeField()),
+                ('modified_date', models.DateTimeField()),
                 ('asset_bed', models.ForeignKey(on_delete=django.db.models.deletion.PROTECT, related_name='camera_position_presets', to='facility.assetbed')),
                 ('created_by', models.ForeignKey(blank=True, null=True, on_delete=django.db.models.deletion.PROTECT, related_name='+', to=settings.AUTH_USER_MODEL)),
                 ('updated_by', models.ForeignKey(blank=True, null=True, on_delete=django.db.models.deletion.PROTECT, related_name='+', to=settings.AUTH_USER_MODEL)),
diff --git a/camera/migrations/0002_auto_20241121_1505.py b/camera/migrations/0002_auto_20241206_0242.py
similarity index 88%
rename from camera/migrations/0002_auto_20241121_1505.py
rename to camera/migrations/0002_auto_20241206_0242.py
index 856347f..83eba7e 100644
--- a/camera/migrations/0002_auto_20241121_1505.py
+++ b/camera/migrations/0002_auto_20241206_0242.py
@@ -1,8 +1,7 @@
-# Generated by Django 5.1.2 on 2024-11-21 09:35
+# Generated by Django 5.1.3 on 2024-12-05 21:12
 
 from django.db import migrations
 
-
 def migrate_camera_preset_to_position_preset(apps, schema_editor):
     try:
         CameraPreset = apps.get_model("facility", "CameraPreset")
@@ -23,6 +22,8 @@ def migrate_camera_preset_to_position_preset(apps, schema_editor):
             created_by=preset.created_by,
             updated_by=preset.updated_by,
             is_migrated=preset.is_migrated,
+            created_date=preset.created_date,
+            modified_date=preset.modified_date
         )
         for preset in camera_presets
     ]
diff --git a/camera/migrations/0003_alter_positionpreset_created_date_and_more.py b/camera/migrations/0003_alter_positionpreset_created_date_and_more.py
new file mode 100644
index 0000000..857cba7
--- /dev/null
+++ b/camera/migrations/0003_alter_positionpreset_created_date_and_more.py
@@ -0,0 +1,23 @@
+# Generated by Django 5.1.3 on 2024-12-05 21:14
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('camera', '0002_auto_20241206_0242'),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name='positionpreset',
+            name='created_date',
+            field=models.DateTimeField(auto_now_add=True, db_index=True, null=True),
+        ),
+        migrations.AlterField(
+            model_name='positionpreset',
+            name='modified_date',
+            field=models.DateTimeField(auto_now=True, db_index=True, null=True),
+        ),
+    ]
diff --git a/tests/__init__.py b/camera/tests/__init__.py
similarity index 100%
rename from tests/__init__.py
rename to camera/tests/__init__.py
diff --git a/tests/test_camera_preset_apis.py b/camera/tests/test_camera_preset_apis.py
similarity index 65%
rename from tests/test_camera_preset_apis.py
rename to camera/tests/test_camera_preset_apis.py
index 8de1f4b..832c5ef 100644
--- a/tests/test_camera_preset_apis.py
+++ b/camera/tests/test_camera_preset_apis.py
@@ -3,6 +3,8 @@
 from care.utils.tests.test_utils import TestUtils
 from rest_framework.test import APITestCase
 from rest_framework import status
+from rest_framework.exceptions import ValidationError
+from camera.utils.onvif import OnvifAsset
 
 
 class AssetBedCameraPresetViewSetTestCase(TestUtils, APITestCase):
@@ -30,7 +32,11 @@ def setUpTestData(cls):
         cls.asset_bed2 = cls.create_assetbed(cls.bed, cls.asset2)
 
     def get_base_url(self, asset_bed_id=None):
-        return f"/api/camera/assetbed/position_presets/?assetbed_external_id={asset_bed_id or self.asset_bed1.external_id}"
+        return f"/api/camera/position-presets/?assetbed_external_id={asset_bed_id or self.asset_bed1.external_id}"
+
+    def validate_invalid_meta(self, asset_class, meta):
+        with self.assertRaises(ValidationError):
+            asset_class(meta)
 
     def test_create_camera_preset_without_position(self):
         res = self.client.post(
@@ -108,14 +114,14 @@ def test_create_camera_preset_and_presence_in_various_preset_list_apis(self):
 
         # Check if preset in asset preset list
         res = self.client.get(
-            f"/api/camera/position_presets/?asset_external_id={asset_bed.asset.external_id}"
+            f"/api/camera/position-presets/?asset_external_id={asset_bed.asset.external_id}"
         )
         self.assertEqual(res.status_code, status.HTTP_200_OK)
         self.assertContains(res, preset_external_id)
 
         # Check if preset in bed preset list
         res = self.client.get(
-            f"/api/camera/position_presets/?bed_external_id={asset_bed.bed.external_id}"
+            f"/api/camera/position-presets/?bed_external_id={asset_bed.bed.external_id}"
         )
         self.assertEqual(res.status_code, status.HTTP_200_OK)
         self.assertContains(res, preset_external_id)
@@ -136,3 +142,54 @@ def test_create_camera_preset_with_same_name_in_same_bed(self):
             self.get_base_url(self.asset_bed2.external_id), data, format="json"
         )
         self.assertEqual(res.status_code, status.HTTP_400_BAD_REQUEST)
+
+
+    def test_meta_validations_for_onvif_asset(self):
+        valid_meta = {
+            "local_ip_address": "192.168.0.1",
+            "camera_access_key": "username:password:access_key",
+            "middleware_hostname": "middleware.local",
+            "insecure_connection": True,
+        }
+        onvif_asset = OnvifAsset(valid_meta)
+        self.assertEqual(onvif_asset.middleware_hostname, "middleware.local")
+        self.assertEqual(onvif_asset.host, "192.168.0.1")
+        self.assertEqual(onvif_asset.username, "username")
+        self.assertEqual(onvif_asset.password, "password")
+        self.assertEqual(onvif_asset.access_key, "access_key")
+        self.assertTrue(onvif_asset.insecure_connection)
+
+        invalid_meta_cases = [
+            # Invalid format for camera_access_key
+            {
+                "id": "123",
+                "local_ip_address": "192.168.0.1",
+                "middleware_hostname": "middleware.local",
+                "camera_access_key": "invalid_format",
+            },
+            # Missing username/password in camera_access_key
+            {
+                "local_ip_address": "192.168.0.1",
+                "middleware_hostname": "middleware.local",
+                "camera_access_key": "invalid_format",
+            },
+            # Missing middleware_hostname
+            {
+                "local_ip_address": "192.168.0.1",
+                "camera_access_key": "username:password:access_key",
+            },
+            # Missing local_ip_address
+            {
+                "middleware_hostname": "middleware.local",
+                "camera_access_key": "username:password:access_key",
+            },
+            # Invalid value for insecure_connection
+            {
+                "local_ip_address": "192.168.0.1",
+                "camera_access_key": "username:password:access_key",
+                "middleware_hostname": "middleware.local",
+                "insecure_connection": "invalid_value",
+            },
+        ]
+        for meta in invalid_meta_cases:
+            self.validate_invalid_meta(OnvifAsset, meta)
diff --git a/setup.py b/setup.py
index 41ec54d..b7bf113 100644
--- a/setup.py
+++ b/setup.py
@@ -10,7 +10,9 @@
 with open("HISTORY.rst") as history_file:
     history = history_file.read()
 
-requirements = []
+requirements = [
+    "requests"
+]
 
 test_requirements = []
 
diff --git a/tests/test_onvif_validations.py b/tests/test_onvif_validations.py
deleted file mode 100644
index 2ac7329..0000000
--- a/tests/test_onvif_validations.py
+++ /dev/null
@@ -1,52 +0,0 @@
-from camera.utils.onvif import OnvifAsset
-
-
-def test_meta_validations_for_onvif_asset(self):
-    valid_meta = {
-        "local_ip_address": "192.168.0.1",
-        "camera_access_key": "username:password:access_key",
-        "middleware_hostname": "middleware.local",
-        "insecure_connection": True,
-    }
-    onvif_asset = OnvifAsset(valid_meta)
-    self.assertEqual(onvif_asset.middleware_hostname, "middleware.local")
-    self.assertEqual(onvif_asset.host, "192.168.0.1")
-    self.assertEqual(onvif_asset.username, "username")
-    self.assertEqual(onvif_asset.password, "password")
-    self.assertEqual(onvif_asset.access_key, "access_key")
-    self.assertTrue(onvif_asset.insecure_connection)
-
-    invalid_meta_cases = [
-        # Invalid format for camera_access_key
-        {
-            "id": "123",
-            "local_ip_address": "192.168.0.1",
-            "middleware_hostname": "middleware.local",
-            "camera_access_key": "invalid_format",
-        },
-        # Missing username/password in camera_access_key
-        {
-            "local_ip_address": "192.168.0.1",
-            "middleware_hostname": "middleware.local",
-            "camera_access_key": "invalid_format",
-        },
-        # Missing middleware_hostname
-        {
-            "local_ip_address": "192.168.0.1",
-            "camera_access_key": "username:password:access_key",
-        },
-        # Missing local_ip_address
-        {
-            "middleware_hostname": "middleware.local",
-            "camera_access_key": "username:password:access_key",
-        },
-        # Invalid value for insecure_connection
-        {
-            "local_ip_address": "192.168.0.1",
-            "camera_access_key": "username:password:access_key",
-            "middleware_hostname": "middleware.local",
-            "insecure_connection": "invalid_value",
-        },
-    ]
-    for meta in invalid_meta_cases:
-        self.validate_invalid_meta(OnvifAsset, meta)