Accomodate reviewer's comments.

Author: SurferJeffAtGoogle

appengine/mail/attachment.py

@@ -11,6 +11,7 @@
 # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 # See the License for the specific language governing permissions and
 # limitations under the License.
+
 from google.appengine.api import app_identity
 from google.appengine.api import mail
 import webapp2
@@ -20,18 +21,19 @@
 class AttachmentHandler(webapp2.RequestHandler):
     def post(self):
         f = self.request.POST['file']
-        mail.send_mail(sender=('%s@appspot.gserviceaccount.com' %
-                               app_identity.get_application_id()),
+        mail.send_mail(sender='{}@appspot.gserviceaccount.com'.format(
+            app_identity.get_application_id()),
                        to="Albert Johnson <[email protected]>",
                        subject="The doc you requested",
-                       body="""Attached is the document file you requested.
+                       body="""
+Attached is the document file you requested.
 
 The example.com Team
 """,
                        attachments=[(f.filename, f.file.read())])
 # [END send_attachment]
         self.response.content_type = 'text/plain'
-        self.response.write('Sent %s to Albert.' % f.filename)
+        self.response.write('Sent {} to Albert.'.format(f.filename))
 
     def get(self):
         self.response.content_type = 'text/html'

appengine/mail/attachment_test.py

@@ -11,6 +11,7 @@
 # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 # See the License for the specific language governing permissions and
 # limitations under the License.
+
 import attachment
 import webtest
 

appengine/mail/handle_bounced_email.py

@@ -11,6 +11,7 @@
 # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 # See the License for the specific language governing permissions and
 # limitations under the License.
+
 import logging
 
 from google.appengine.ext.webapp.mail_handlers import BounceNotificationHandler

appengine/mail/handle_bounced_email_test.py

@@ -11,6 +11,7 @@
 # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 # See the License for the specific language governing permissions and
 # limitations under the License.
+
 from google.appengine.ext.webapp.mail_handlers import BounceNotification
 
 import handle_bounced_email

appengine/mail/handle_catchall.py

@@ -11,6 +11,7 @@
 # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 # See the License for the specific language governing permissions and
 # limitations under the License.
+
 from google.appengine.api import mail
 
 import handle_incoming_email

appengine/mail/handle_incoming_email_test.py

@@ -42,11 +42,12 @@ def get(self):
     def post(self):
         print repr(self.request.POST)
         id = self.request.POST['thread_id']
-        send_example_mail('%s@appspot.gserviceaccount.com' %
-                          app_identity.get_application_id(), id)
+        send_example_mail('{}@appspot.gserviceaccount.com'.format(
+            app_identity.get_application_id()), id)
         self.response.content_type = 'text/plain'
         self.response.write(
-            'Sent an email to Albert with Reference header set to %s.' % id)
+            'Sent an email to Albert with Reference header set to {}.'
+            .format(id))
 
 
 app = webapp2.WSGIApplication([

appengine/mail/handle_owner.py

@@ -11,6 +11,7 @@
 # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 # See the License for the specific language governing permissions and
 # limitations under the License.
+
 import header
 import webtest