8000 Merge pull request #258 from RussTheAerialist/bugfix-257-empty_req_fails · sfarr15/server-client-python@b817537 · GitHub
[go: up one dir, main page]

Skip to content

Commit b817537

Browse files
author
Russell Hay
authored
Merge pull request tableau#258 from RussTheAerialist/bugfix-257-empty_req_fails
Bugfix 257 empty req fails
2 parents 564dda1 + 11ada55 commit b817537

File tree

3 files changed

+13
-1
lines changed

3 files changed

+13
-1
lines changed

CHANGELOG.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,7 @@
1+
## 0.6.1 (26 Jan 2018)
2+
3+
* Fixed #257 where refreshing extracts does not work due to a missing "self"
4+
15
## 0.6 (17 Jan 2018)
26

37
* Added support for add a datasource/workbook refresh to a schedule (#244)

tableauserverclient/server/request_factory.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -401,7 +401,7 @@ def create_req(self, subscription_item):
401401

402402
class EmptyRequest(object):
403403
@_tsrequest_wrapped
404-
def empty_req(xml_request):
404+
def empty_req(self, xml_request):
405405
pass
406406

407407

test/test_regression_tests.py

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
import unittest
2+
import tableauserverclient.server.request_factory as factory
3+
4+
5+
class BugFix257(unittest.TestCase):
6+
def test_empty_request_works(self):
7+
result = factory.EmptyRequest().empty_req()
8+
self.assertEqual(b'<tsRequest />', result)

0 commit comments

Comments
 (0)
0