Merge pull request #6 from avantifellows/linting-errors

Minor linting corrections
avantifellows · Nov 3, 2020 · 233aff9 · 233aff9
2 parents cddd938 + 5c6a517
commit 233aff9
Showing 1 changed file with 14 additions and 9 deletions.
diff --git a/utils/avanti_s3.py b/utils/avanti_s3.py
@@ -7,13 +7,16 @@
 import urllib.request
 import urllib
 import datetime
-# NOTE: public authentication details
+
 
 def get_video_title(videoId):
     """
     Gets video title from Youtube
     """
-    params = {"format": "json", "url": "https://www.youtube.com/watch?v=%s" % videoId}
+    params = {
+        "format": "json",
+        "url": "https://www.youtube.com/watch?v=%s" % videoId
+    }
     url = "https://www.youtube.com/oembed"
     query_string = urllib.parse.urlencode(params)
     url = url + "?" + query_string
@@ -23,6 +26,7 @@ def get_video_title(videoId):
         data = json.loads(response_text.decode())
     return data["title"]
 
+
 def push_response_to_s3(response_data):
     meta_data = response_data['meta']
     response = response_data['response']
@@ -40,13 +44,14 @@ def push_response_to_s3(response_data):
     file_name = f"{meta_data['object_id']}_{meta_data['student_id']}.json"
 
     # To handle windows' default backslash system
-    file_path = join(save_dir, file_name).replace("\\","/")
-
-    s3.Object(bucket, file_path).put(Body=json.dumps(response), ContentType='application/json')
+    file_path = join(save_dir, file_name).replace("\\", "/")
 
+    s3.Object(bucket, file_path).put(
+        Body=json.dumps(response), ContentType='application/json')
 
     return f"http://avanti-fellows.s3.ap-south-1.amazonaws.com/{file_path}"
 
+
 def get_resource(
         service_name='s3', region_name='ap-south-1',
         aws_access_key_id='AKIAQHKYBVIIKK3LUGF6',
@@ -92,13 +97,13 @@ def get_all_ivideo_objects(bucket='avanti-fellows', extensions=['json']):
                     s3.Object(bucket, file.key).get()['Body'].read().decode(
                         'utf-8'))
                 video_title = get_video_title(json_content["video_id"])
-                date = datetime.datetime.strftime(file.last_modified, "%Y-%m-%d")
+                date = datetime.datetime.strftime(
+                    file.last_modified, "%Y-%m-%d")
                 matching_files.append(dict({
-                    "object_id": name, "details": json_content, "title": video_title, "created": date
+                    "object_id": name, "details": json_content,
+                    "title": video_title, "created": date
                 }))
             except Exception as e:
                 pass
 
     return matching_files
-
-