From 3c2d1003b4e5cda205632171483a3a1d8ea42bc5 Mon Sep 17 00:00:00 2001 From: Ned Batchelder Date: Wed, 2 Oct 2013 09:39:07 -0400 Subject: [PATCH 1/2] Suppress request logging in the LTI test server. --- lms/djangoapps/courseware/mock_lti_server/mock_lti_server.py | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/lms/djangoapps/courseware/mock_lti_server/mock_lti_server.py b/lms/djangoapps/courseware/mock_lti_server/mock_lti_server.py index 833df3a1c3..94b6f2ecc4 100644 --- a/lms/djangoapps/courseware/mock_lti_server/mock_lti_server.py +++ b/lms/djangoapps/courseware/mock_lti_server/mock_lti_server.py @@ -13,6 +13,10 @@ class MockLTIRequestHandler(BaseHTTPRequestHandler): protocol = "HTTP/1.0" + def log_request(self, *args, **kwargs): + """Don't log requests, this is just test code.""" + pass + def do_HEAD(self): self._send_head() From ca345d92cc25467b08a4888a7860af4deb46b009 Mon Sep 17 00:00:00 2001 From: Ned Batchelder Date: Wed, 9 Oct 2013 10:03:10 -0400 Subject: [PATCH 2/2] Write to stdout to keep messages, but not pollute tests. --- .../courseware/mock_lti_server/mock_lti_server.py | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/lms/djangoapps/courseware/mock_lti_server/mock_lti_server.py b/lms/djangoapps/courseware/mock_lti_server/mock_lti_server.py index 94b6f2ecc4..83208d0102 100644 --- a/lms/djangoapps/courseware/mock_lti_server/mock_lti_server.py +++ b/lms/djangoapps/courseware/mock_lti_server/mock_lti_server.py @@ -2,6 +2,7 @@ from BaseHTTPServer import HTTPServer, BaseHTTPRequestHandler import urlparse from oauthlib.oauth1.rfc5849 import signature import mock +import sys from logging import getLogger logger = getLogger(__name__) @@ -13,9 +14,14 @@ class MockLTIRequestHandler(BaseHTTPRequestHandler): protocol = "HTTP/1.0" - def log_request(self, *args, **kwargs): - """Don't log requests, this is just test code.""" - pass + def log_message(self, format, *args): + """Log an arbitrary message.""" + # Code copied from BaseHTTPServer.py. Changed to write to sys.stdout + # so that messages won't pollute test output. + sys.stdout.write("%s - - [%s] %s\n" % + (self.client_address[0], + self.log_date_time_string(), + format % args)) def do_HEAD(self): self._send_head()