Merge pull request #82 from MindscapeHQ/fix-setRequestDetails-chaining

Add missing return self in set_request_details builder

Author: UberMouse

CHANGELOG.md

@@ -1,3 +1,7 @@
+## 4.2.2 (23/01/2019):
+Bugfixes
+  - Fix `set_request_details` builder method not returning self causing it to be unchainable
+
 ## 4.2.1 (18/12/2018):
 Bugfixes
   - Set version correctly in crash report sent to Raygun API ([#78](https://github.com/MindscapeHQ/raygun4py/pull/79))

python2/raygun4py/version.py

@@ -2,4 +2,4 @@
 # 1) we don't load dependencies by storing it in __init__.py
 # 2) we can import it in setup.py for the same reason
 # 3) we can import it into your module module
-__version__ = '4.2.1'
+__version__ = '4.2.2'

python3/raygun4py/raygunmsgs.py

@@ -97,6 +97,8 @@ def set_request_details(self, request):
         rg_request_details = http_utilities.build_wsgi_compliant_request(request)
         self.raygunMessage.details['request'] = rg_request_details
 
+        return self
+
 
     def set_version(self, version):
         self.raygunMessage.details['version'] = version

python3/raygun4py/version.py

@@ -2,4 +2,4 @@
 # 1) we don't load dependencies by storing it in __init__.py
 # 2) we can import it in setup.py for the same reason
 # 3) we can import it into your module module
-__version__ = '4.2.1'
+__version__ = '4.2.2'

python3/tests/test_raygunmsgs.py

@@ -109,6 +109,12 @@ def test_wsgi_standard_header_names(self):
         self.assertEqual(self.builder.raygunMessage.details['request']['headers']['Referer'],
                         "https://www.google.com/")
 
+    def test_set_request_details_allows_chaining(self):
+        self.builder \
+                .set_request_details(self.raw_wsgi_request) \
+                .set_tags(['foo', 'bar'])
+
+
 class TestRaygunErrorMessage(unittest.TestCase):
     class GrandchildError(Exception):
         pass