get_last_entries_from_db: double json decode (ansible#258)

* get_last_entries_from_db: double json decode

the settings model json-encodes everything when saving to the db,
but we're already passing it a json.dumps output

that's how it works for a while now, not changing that,
but updating the parser to cope with the double json

and updating mock db accordingly

Fixes:

    File "/var/lib/awx/venv/awx/lib64/python3.11/site-packages/metrics_utility/base/collector.py", line 155, in last_gathered_entry_for
    return self.last_gathered_entries.get(key)', ' ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^', "AttributeError: 'str' object has no attribute 'get'

* adjust test accordingly

Author: himdel

metrics_utility/automation_controller_billing/helpers.py

@@ -29,7 +29,8 @@ def get_last_entries_from_db() -> Dict:
             result = cursor.fetchone()
 
             if result and result[0]:
-                return json.loads(result[0], object_hook=datetime_hook)  # This is the JSON value
+                json_in_json = json.loads(result[0])
+                return json.loads(json_in_json, object_hook=datetime_hook)  # This is the JSON value
     except Exception as e:
         logger.error(f'Error getting AUTOMATION_ANALYTICS_LAST_ENTRIES from database: {e}')
     return {}

metrics_utility/test/test_automation_controller_billing_helpers.py

@@ -70,7 +70,7 @@ def test_successful_entries_retrieval(self, mock_connection):
         # Setup
         mock_cursor = MagicMock()
         mock_connection.cursor.return_value.__enter__.return_value = mock_cursor
-        test_json = '{"config": "2024-01-01T00:00:00Z", "hosts": "2024-01-03T00:00:00Z", "jobs": "2024-01-02T00:00:00Z"}'
+        test_json = '"{\\"config\\": \\"2024-01-01T00:00:00Z\\", \\"hosts\\": \\"2024-01-03T00:00:00Z\\", \\"jobs\\": \\"2024-01-02T00:00:00Z\\"}"'
         mock_cursor.fetchone.return_value = (test_json,)
         # Execute
         result = get_last_entries_from_db()
@@ -168,7 +168,8 @@ def test_functions_work_with_real_data(self, mock_connection):
             ('SUBSCRIPTION_NAME', '"Red Hat AAP"'),
             ('ABC', '"1.2.3"'),
         ]
-        mock_cursor.fetchone.return_value = ('{"config": "2024-01-01T00:00:00Z", "jobs": "2024-01-02T00:00:00Z"}',)  # Last entries result
+        test_json = '"{\\"config\\": \\"2024-01-01T00:00:00Z\\", \\"jobs\\": \\"2024-01-02T00:00:00Z\\"}"'  # Last entries result
+        mock_cursor.fetchone.return_value = (test_json,)
 
         # Execute all functions
         license_info, settings_info = get_config_and_settings_from_db()

tools/docker/conf_setting.sql

@@ -6,4 +6,4 @@ INSERT INTO public.conf_setting (created, modified, key, value) VALUES
   (now(), now(), 'INSTALL_UUID', '"00000000-0000-0000-0000-000000000000"'),
   (now(), now(), 'LICENSE', '{"license_type": "UNLICENSED", "product_name": "AWX", "subscription_name": null, "valid_key": false}'),
   (now(), now(), 'TOWER_URL_BASE', '"https://platformhost"'),
-  (now(), now(), 'AUTOMATION_ANALYTICS_LAST_ENTRIES', '{"config": "2024-01-01T10:00:00Z", "jobs": "2024-01-02T15:30:00Z"}');
+  (now(), now(), 'AUTOMATION_ANALYTICS_LAST_ENTRIES', '"{\"config\": \"2024-01-01T10:00:00Z\", \"jobs\": \"2024-01-02T15:30:00Z\"}"');