Addresss comments

Author: Simon Meng

src/s3helper.py

@@ -28,8 +28,12 @@ def get_input_artifact(event):
     )
 
     input_artifacts = event['CodePipeline.job']['data']['inputArtifacts']
-    _validate_input_artifacts(input_artifacts)
+
+    if len(input_artifacts) != 1:
+        raise RuntimeError('You should only have one input artifact. Please check the setting for the action.')
+
     artifact_to_fetch = input_artifacts[0]
+    LOG.info('artifact_to_fetch=%s', artifact_to_fetch)
 
     artifact_s3_location = artifact_to_fetch['location']['s3Location']
     bucket = artifact_s3_location['bucketName']
@@ -42,16 +46,6 @@ def get_input_artifact(event):
     return _unzip_as_string(zipped_content_as_bytes)
 
 
-def _validate_input_artifacts(input_artifacts):
-    """Validate the length of input artifacts list is 1.
-
-    Arguments:
-        input_artifacts {dict list} -- list of input artifacts
-    """
-    if len(input_artifacts) != 1:
-        raise RuntimeError('You should only have one input artifact. Please check the setting for the action.')
-
-
 def _unzip_as_string(data):
     """Unzip stream of data in bytes as string.