Merge pull request #801 from tableau/release-0.15

[Release] Sync master with v0.15 changes
## 0.15.0 (16 Feb 2021)
* Added support for python version 3.9 (#744)
* Added support for 'Get View by ID' (#750)
* Added docs and test data to MANIFEST.in file (#780)
* Added owner_id property to ProjectItem (#784)
* Added support for skipping connection check while publishing workbook (#791)
* Added support for 'Update Subscription' (#794)
* Added support for 'Get Groups for a User' (#799)
* Improved debug logging by including put/post request contents (#743)
* Improved local and active-directory group creation (#770)
* Improved 'Update Group' to match server requests/responses (#772)
* Improved SiteItem with new properties and functions (#777)
* Improved SubscriptionItem with new properties (#794)
* Improved the 'type' property of TaskItem to convert server response to enum (#796)
* Improved repository to use Github Actions for running tests/linter (#798)
* Fixed data_acceleration field causing error in workbook update payload (#741)

Author: Chris Shin

.github/workflows/run-tests.yml

@@ -0,0 +1,33 @@
+name: Python package
+
+on: [push]
+
+jobs:
+  build:
+
+    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        python-version: [3.5, 3.6, 3.7, 3.8, 3.9]
+
+    steps:
+    - uses: actions/checkout@v2
+
+    - name: Set up Python ${{ matrix.python-version }}
+      uses: actions/setup-python@v2
+      with:
+        python-version: ${{ matrix.python-version }}
+
+    - name: Install dependencies
+      run: |
+        python -m pip install --upgrade pip
+        pip install -e .[test]
+
+    - name: Lint with pycodestyle
+      run: |
+        pycodestyle tableauserverclient test samples
+
+    - name: Test with pytest
+      run: |
+        pytest test

.travis.yml

@@ -1,3 +1,20 @@
+## 0.15.0 (16 Feb 2021)
+* Added support for python version 3.9 (#744)
+* Added support for 'Get View by ID' (#750)
+* Added docs and test data to MANIFEST.in file (#780)
+* Added owner_id property to ProjectItem (#784)
+* Added support for skipping connection check while publishing workbook (#791)
+* Added support for 'Update Subscription' (#794)
+* Added support for 'Get Groups for a User' (#799)
+* Improved debug logging by including put/post request contents (#743)
+* Improved local and active-directory group creation (#770)
+* Improved 'Update Group' to match server requests/responses (#772)
+* Improved SiteItem with new properties and functions (#777)
+* Improved SubscriptionItem with new properties (#794)
+* Improved the 'type' property of TaskItem to convert server response to enum (#796)
+* Improved repository to use Github Actions for running tests/linter (#798)
+* Fixed data_acceleration field causing error in workbook update payload (#741)
+
 ## 0.14.1 (9 Dec 2020)
 * Fixed filter query issue for server version below 2020.1 (#745)
 * Fixed large workbook/datasource publish issue (#757)

CHANGELOG.md

@@ -41,6 +41,9 @@ The following people have contributed to this project to make it possible, and w
 * [Paul Vickers](https://github.com/paulvic)
 * [Madhura Selvarajan](https://github.com/maddy-at-leisure)
 * [Niklas Nevalainen](https://github.com/nnevalainen)
+* [Terrence Jones](https://github.com/tjones-commits)
+* [John Vandenberg](https://github.com/jayvdb)
+* [Lee Boynton](https://github.com/lboynton)
 
 ## Core Team
 

CONTRIBUTORS.md

@@ -2,3 +2,22 @@ include versioneer.py
 include tableauserverclient/_version.py
 include LICENSE
 include LICENSE.versioneer
+include README.md
+include CHANGELOG.md
+recursive-include docs *.md
+recursive-include samples *.py
+recursive-include samples *.txt
+recursive-include smoke *.py
+recursive-include test *.csv
+recursive-include test *.dict
+recursive-include test *.hyper
+recursive-include test *.json
+recursive-include test *.pdf
+recursive-include test *.png
+recursive-include test *.py
+recursive-include test *.tde
+recursive-include test *.tds
+recursive-include test *.tdsx
+recursive-include test *.twb
+recursive-include test *.twbx
+recursive-include test *.xml

MANIFEST.in

@@ -31,6 +31,7 @@ def main():
     parser.add_argument('--logging-level', '-l', choices=['debug', 'info', 'error'], default='error',
                         help='desired logging level (set to error by default)')
     parser.add_argument('--as-job', '-a', help='Publishing asynchronously', action='store_true')
+    parser.add_argument('--skip-connection-check', '-c', help='Skip live connection check', action='store_true')
     parser.add_argument('--site', '-S', default='', help='id (contentUrl) of site to sign into')
 
     args = parser.parse_args()
@@ -71,11 +72,13 @@ def main():
             new_workbook = TSC.WorkbookItem(default_project.id)
             if args.as_job:
                 new_job = server.workbooks.publish(new_workbook, args.filepath, overwrite_true,
-                                                   connections=all_connections, as_job=args.as_job)
+                                                   connections=all_connections, as_job=args.as_job,
+                                                   skip_connection_check=args.skip_connection_check)
                 print("Workbook published. JOB ID: {0}".format(new_job.id))
             else:
                 new_workbook = server.workbooks.publish(new_workbook, args.filepath, overwrite_true,
-                                                        connections=all_connections, as_job=args.as_job)
+                                                        connections=all_connections, as_job=args.as_job,
+                                                        skip_connection_check=args.skip_connection_check)
                 print("Workbook published. ID: {0}".format(new_workbook.id))
         else:
             error = "The default project could not be found."

samples/publish_workbook.py

@@ -15,6 +15,7 @@
 # This makes work easier for offline installs or low bandwidth machines
 needs_pytest = {'pytest', 'test', 'ptr'}.intersection(sys.argv)
 pytest_runner = ['pytest-runner'] if needs_pytest else []
+test_requirements = ['mock', 'pycodestyle', 'pytest', 'requests-mock>=1.0,<2.0']
 
 setup(
     name='tableauserverclient',
@@ -34,9 +35,8 @@
     install_requires=[
         'requests>=2.11,<3.0',
     ],
-    tests_require=[
-        'requests-mock>=1.0,<2.0',
-        'pytest',
-        'mock'
-    ]
+    tests_require=test_requirements,
+    extras_require={
+        'test': test_requirements
+    }
 )

setup.py

@@ -9,13 +9,17 @@ class GroupItem(object):
 
     tag_name = 'group'
 
-    def __init__(self, name=None):
-        self._domain_name = None
+    class LicenseMode:
+        onLogin = 'onLogin'
+        onSync = 'onSync'
+
+    def __init__(self, name=None, domain_name=None):
         self._id = None
-        self._users = None
-        self.name = name
         self._license_mode = None
         self._minimum_site_role = None
+        self._users = None
+        self.name = name
+        self.domain_name = domain_name
 
     @property
     def domain_name(self):
@@ -43,8 +47,8 @@ def license_mode(self):
         return self._license_mode
 
     @license_mode.setter
+    @property_is_enum(LicenseMode)
     def license_mode(self, value):
-        # valid values = onSync, onLogin
         self._license_mode = value
 
     @property
@@ -79,17 +83,18 @@ def from_response(cls, resp, ns):
             name = group_xml.get('name', None)
             group_item = cls(name)
             group_item._id = group_xml.get('id', None)
-            # AD groups have an extra element under this
+
+            # Domain name is returned in a domain element for some calls
+            domain_elem = group_xml.find('.//t:domain', namespaces=ns)
+            if domain_elem is not None:
+                group_item.domain_name = domain_elem.get('name', None)
+
+            # Import element is returned for both local and AD groups (2020.3+)
             import_elem = group_xml.find('.//t:import', namespaces=ns)
-            if (import_elem is not None):
-                group_item.domain_name = import_elem.get('domainName')
-                group_item.license_mode = import_elem.get('grantLicenseMode')
-                group_item.minimum_site_role = import_elem.get('siteRole')
-            else:
-                # local group, we will just have two extra attributes here
-                group_item.domain_name = 'local'
-                group_item.license_mode = group_xml.get('grantLicenseMode')
-                group_item.minimum_site_role = group_xml.get('siteRole')
+            if import_elem is not None:
+                group_item.domain_name = import_elem.get('domainName', None)
+                group_item.license_mode = import_elem.get('grantLicenseMode', None)
+                group_item.minimum_site_role = import_elem.get('siteRole', None)
 
             all_group_items.append(group_item)
         return all_group_items

tableauserverclient/models/group_item.py

@@ -74,6 +74,14 @@ def name(self):
     def name(self, value):
         self._name = value
 
+    @property
+    def owner_id(self):
+        return self._owner_id
+
+    @owner_id.setter
+    def owner_id(self, value):
+        raise NotImplementedError('REST API does not currently support updating project owner.')
+
     def is_default(self):
         return self.name.lower() == 'default'
 
@@ -86,7 +94,7 @@ def _parse_common_tags(self, project_xml, ns):
             self._set_values(None, name, description, content_permissions, parent_id)
         return self
 
-    def _set_values(self, project_id, name, description, content_permissions, parent_id):
+    def _set_values(self, project_id, name, description, content_permissions, parent_id, owner_id):
         if project_id is not None:
             self._id = project_id
         if name:
@@ -97,6 +105,8 @@ def _set_values(self, project_id, name, description, content_permissions, parent
             self._content_permissions = content_permissions
         if parent_id:
             self.parent_id = parent_id
+        if owner_id:
+            self._owner_id = owner_id
 
     def _set_permissions(self, permissions):
         self._permissions = permissions
@@ -111,9 +121,9 @@ def from_response(cls, resp, ns):
         all_project_xml = parsed_response.findall('.//t:project', namespaces=ns)
 
         for project_xml in all_project_xml:
-            (id, name, description, content_permissions, parent_id) = cls._parse_element(project_xml)
+            (id, name, description, content_permissions, parent_id, owner_id) = cls._parse_element(project_xml)
             project_item = cls(name)
-            project_item._set_values(id, name, description, content_permissions, parent_id)
+            project_item._set_values(id, name, description, content_permissions, parent_id, owner_id)
             all_project_items.append(project_item)
         return all_project_items
 
@@ -124,5 +134,8 @@ def _parse_element(project_xml):
         description = project_xml.get('description', None)
         content_permissions = project_xml.get('contentPermissions', None)
         parent_id = project_xml.get('parentProjectId', None)
+        owner_id = None
+        for owner in project_xml:
+            owner_id = owner.get('id', None)
 
-        return id, name, description, content_permissions, parent_id
+        return id, name, description, content_permissions, parent_id, owner_id