Add scan_date to import settings if overridden (#14502)

* Add scan_date to import settings if overridden

* Fix datetime JSON serialization in import_settings

Convert scan_date to ISO format string before storing in import_settings
dict, which gets JSON-serialized. Raw datetime objects cause
"TypeError: Object of type datetime is not JSON serializable".

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* Add unit tests for scan_date in import_settings

Tests both cases:
- User supplies scan_date: verifies it's stored as ISO string
- No scan_date supplied: verifies it's stored as None

Both tests also verify import_settings remains JSON-serializable.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* Fix timezone import in scan_date test

Use datetime.timezone.utc instead of django.utils.timezone.utc
which doesn't exist.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* Fix ruff lint: import sorting and use datetime.UTC alias

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: Maffooch

dojo/importers/base_importer.py

@@ -463,6 +463,7 @@ def update_import_history(
         import_settings["close_old_findings"] = self.close_old_findings_toggle
         import_settings["push_to_jira"] = self.push_to_jira
         import_settings["tags"] = self.tags
+        import_settings["scan_date"] = self.scan_date.isoformat() if self.scan_date_override else None
         if settings.V3_FEATURE_LOCATIONS:
             # Add the list of locations that were added exclusively at import time
             if len(self.endpoints_to_add) > 0:

unittests/test_update_import_history.py

@@ -1,4 +1,6 @@
+import json
 import logging
+from datetime import UTC, datetime
 from unittest.mock import patch
 
 from django.contrib.auth.models import User as DjangoUser
@@ -149,3 +151,29 @@ def test_precheck_filters_out_deleted_findings_allows_successful_bulk(self):
         expected = (len(new_findings) - 1) + (len(closed_findings) - 1)
         created = Test_Import_Finding_Action.objects.filter(test_import=test_import).count()
         self.assertEqual(created, expected)
+
+    def test_import_settings_scan_date_when_user_supplies_scan_date(self):
+        """When the user supplies a scan_date, import_settings should contain the ISO-formatted date."""
+        user_scan_date = datetime(2025, 6, 15, 12, 0, 0, tzinfo=UTC)
+        self.importer.scan_date = user_scan_date
+        self.importer.scan_date_override = True
+
+        new_findings = self._create_findings(1)
+        test_import = self.importer.update_import_history(new_findings=new_findings)
+
+        settings = test_import.import_settings
+        # Verify import_settings is JSON-serializable (the original bug)
+        json.dumps(settings)
+        self.assertEqual(settings["scan_date"], user_scan_date.isoformat())
+
+    def test_import_settings_scan_date_when_no_scan_date_supplied(self):
+        """When no scan_date override is provided, import_settings should have scan_date as None."""
+        self.importer.scan_date_override = False
+
+        new_findings = self._create_findings(1)
+        test_import = self.importer.update_import_history(new_findings=new_findings)
+
+        settings = test_import.import_settings
+        # Verify import_settings is JSON-serializable
+        json.dumps(settings)
+        self.assertIsNone(settings["scan_date"])