CHO: Remove commented out code and update docstring

terraform/storage.tf

@@ -1,4 +1,4 @@
-# Add a static bucket (public contents)
+# Add a bucket to store cloud function source code in.
 resource "google_storage_bucket" "twined_gcp_source" {
   name                        = "twined-gcp"
   location                    = "EU"
@@ -13,21 +13,11 @@ resource "google_storage_bucket" "twined_gcp_source" {
 }
 
 
-# Make static bucket contents public
+# Make bucket contents public.
 resource "google_storage_bucket_iam_binding" "static_assets_object_viewer" {
   bucket = google_storage_bucket.twined_gcp_source.name
   role   = "roles/storage.objectViewer"
   members = [
     "allUsers"
   ]
 }
-
-
-## Allow operating service account to generate signed upload urls
-#resource "google_storage_bucket_iam_binding" "media_assets_object_admin" {
-#  bucket = google_storage_bucket.media_assets.name
-#  role   = "roles/storage.objectAdmin"
-#  members = [
-#    "serviceAccount:${google_service_account.server_service_account.email}"
-#  ]
-#}

tests/mocks.py

@@ -6,7 +6,7 @@ class MockCloudEvent:
     data: dict
 
 
-# This class is duplicated from `data-gateway` (see ATTRIBUTIONS.md).
+# This class is duplicated from `aerosense-ai/data-gateway` (see ATTRIBUTIONS.md).
 class MockBigQueryClient:
     def __init__(self, expected_query_results=None):
         self.expected_query_results = expected_query_results or [[]]
@@ -51,7 +51,7 @@ def query(self, query, *args, **kwargs):
         return result
 
 
-# This class is duplicated from `data-gateway` (see ATTRIBUTIONS.md).
+# This class is duplicated from `aerosense-ai/data-gateway` (see ATTRIBUTIONS.md).
 class MockQueryResult:
     def __init__(self, result):
         self._result = result

tests/test_event_handler.py

@@ -10,14 +10,9 @@
 REPOSITORY_ROOT = os.path.dirname(os.path.dirname(__file__))
 
 
-# # Manually add the cloud_functions package to the path (its imports have to be done in a certain way for Google Cloud
-# # Functions to accept them that doesn't work when running/testing the package locally).
-# sys.path.insert(0, os.path.abspath(os.path.join(REPOSITORY_ROOT, "functions", "event_handler")))
-# from functions.event_handler.main import store_pub_sub_event_in_bigquery
-
-
 class TestEventHandler(unittest.TestCase):
     def test_store_pub_sub_event_in_bigquery(self):
+        """Test that the `event_handler` cloud function can receive, parse, and store an event in BigQuery."""
         cloud_event = MockCloudEvent(
             data={
                 "message": {