From 1dfa29f0674e6843981410604dd4977420175adf Mon Sep 17 00:00:00 2001
From: manishT72x <manish.tanwar@xero.com>
Date: Mon, 24 Feb 2025 23:19:34 +0530
Subject: [PATCH] fix: can_refresh_access_token method accepts string as valid
 token type

---
 docs/v1/accounting/index.html        |  2 +-
 docs/v1/appstore/index.html          |  2 +-
 docs/v1/assets/index.html            |  2 +-
 docs/v1/files/index.html             |  2 +-
 docs/v1/finance/index.html           |  2 +-
 docs/v1/payroll-au/index.html        |  2 +-
 docs/v1/payroll-nz/index.html        |  2 +-
 docs/v1/payroll-uk/index.html        |  2 +-
 docs/v1/projects/index.html          |  2 +-
 setup.py                             |  2 +-
 tests/test_api_client/test_oauth2.py | 43 ++++++++++++++++++++++++++++
 xero_python/__init__.py              |  2 +-
 xero_python/api_client/oauth2.py     |  2 +-
 xero_python/docs/README.md           |  2 +-
 14 files changed, 56 insertions(+), 13 deletions(-)

diff --git a/docs/v1/accounting/index.html b/docs/v1/accounting/index.html
index 2688b484..fe71624d 100644
--- a/docs/v1/accounting/index.html
+++ b/docs/v1/accounting/index.html
@@ -6339,7 +6339,7 @@
           <nav id="scrollingNav">
             <ul class="sidenav nav nav-list">
               <li class="nav-header" data-group="Accounting"><strong>SDK: </strong><span id='sdk-name'></span></li>
-              <li class="nav-header" data-group="Accounting"><strong>VSN: </strong>7.0.0</li>
+              <li class="nav-header" data-group="Accounting"><strong>VSN: </strong>7.0.1</li>
               <li class="nav-header" data-group="Accounting"><a href="#api-Accounting">Methods</a></li>
               <li data-group="Accounting" data-name="createAccount" class="">
                 <a href="#api-Accounting-createAccount">createAccount</a>
diff --git a/docs/v1/appstore/index.html b/docs/v1/appstore/index.html
index 1f4eb9ac..01fdfa5c 100644
--- a/docs/v1/appstore/index.html
+++ b/docs/v1/appstore/index.html
@@ -1241,7 +1241,7 @@
           <nav id="scrollingNav">
             <ul class="sidenav nav nav-list">
               <li class="nav-header" data-group="AppStore"><strong>SDK: </strong><span id='sdk-name'></span></li>
-              <li class="nav-header" data-group="AppStore"><strong>VSN: </strong>7.0.0</li>
+              <li class="nav-header" data-group="AppStore"><strong>VSN: </strong>7.0.1</li>
               <li class="nav-header" data-group="AppStore"><a href="#api-AppStore">Methods</a></li>
               <li data-group="AppStore" data-name="getSubscription" class="">
                 <a href="#api-AppStore-getSubscription">getSubscription</a>
diff --git a/docs/v1/assets/index.html b/docs/v1/assets/index.html
index a16e71e9..017c355c 100644
--- a/docs/v1/assets/index.html
+++ b/docs/v1/assets/index.html
@@ -1392,7 +1392,7 @@
           <nav id="scrollingNav">
             <ul class="sidenav nav nav-list">
               <li class="nav-header" data-group="Asset"><strong>SDK: </strong><span id='sdk-name'></span></li>
-              <li class="nav-header" data-group="Asset"><strong>VSN: </strong>7.0.0</li>
+              <li class="nav-header" data-group="Asset"><strong>VSN: </strong>7.0.1</li>
               <li class="nav-header" data-group="Asset"><a href="#api-Asset">Methods</a></li>
               <li data-group="Asset" data-name="createAsset" class="">
                 <a href="#api-Asset-createAsset">createAsset</a>
diff --git a/docs/v1/files/index.html b/docs/v1/files/index.html
index 558eb317..135548e9 100644
--- a/docs/v1/files/index.html
+++ b/docs/v1/files/index.html
@@ -1170,7 +1170,7 @@
           <nav id="scrollingNav">
             <ul class="sidenav nav nav-list">
               <li class="nav-header" data-group="Files"><strong>SDK: </strong><span id='sdk-name'></span></li>
-              <li class="nav-header" data-group="Files"><strong>VSN: </strong>7.0.0</li>
+              <li class="nav-header" data-group="Files"><strong>VSN: </strong>7.0.1</li>
               <li class="nav-header" data-group="Files"><a href="#api-Files">Methods</a></li>
               <li data-group="Files" data-name="createFileAssociation" class="">
                 <a href="#api-Files-createFileAssociation">createFileAssociation</a>
diff --git a/docs/v1/finance/index.html b/docs/v1/finance/index.html
index 1c417778..12d3c887 100644
--- a/docs/v1/finance/index.html
+++ b/docs/v1/finance/index.html
@@ -2716,7 +2716,7 @@
           <nav id="scrollingNav">
             <ul class="sidenav nav nav-list">
               <li class="nav-header" data-group="Finance"><strong>SDK: </strong><span id='sdk-name'></span></li>
-              <li class="nav-header" data-group="Finance"><strong>VSN: </strong>7.0.0</li>
+              <li class="nav-header" data-group="Finance"><strong>VSN: </strong>7.0.1</li>
               <li class="nav-header" data-group="Finance"><a href="#api-Finance">Methods</a></li>
               <li data-group="Finance" data-name="getAccountingActivityAccountUsage" class="">
                 <a href="#api-Finance-getAccountingActivityAccountUsage">getAccountingActivityAccountUsage</a>
diff --git a/docs/v1/payroll-au/index.html b/docs/v1/payroll-au/index.html
index 09f0135b..32e69e0c 100644
--- a/docs/v1/payroll-au/index.html
+++ b/docs/v1/payroll-au/index.html
@@ -3412,7 +3412,7 @@
           <nav id="scrollingNav">
             <ul class="sidenav nav nav-list">
               <li class="nav-header" data-group="PayrollAu"><strong>SDK: </strong><span id='sdk-name'></span></li>
-              <li class="nav-header" data-group="PayrollAu"><strong>VSN: </strong>7.0.0</li>
+              <li class="nav-header" data-group="PayrollAu"><strong>VSN: </strong>7.0.1</li>
               <li class="nav-header" data-group="PayrollAu"><a href="#api-PayrollAu">Methods</a></li>
               <li data-group="PayrollAu" data-name="approveLeaveApplication" class="">
                 <a href="#api-PayrollAu-approveLeaveApplication">approveLeaveApplication</a>
diff --git a/docs/v1/payroll-nz/index.html b/docs/v1/payroll-nz/index.html
index b8cb75b7..39745dcb 100644
--- a/docs/v1/payroll-nz/index.html
+++ b/docs/v1/payroll-nz/index.html
@@ -4087,7 +4087,7 @@
           <nav id="scrollingNav">
             <ul class="sidenav nav nav-list">
               <li class="nav-header" data-group="PayrollNz"><strong>SDK: </strong><span id='sdk-name'></span></li>
-              <li class="nav-header" data-group="PayrollNz"><strong>VSN: </strong>7.0.0</li>
+              <li class="nav-header" data-group="PayrollNz"><strong>VSN: </strong>7.0.1</li>
               <li class="nav-header" data-group="PayrollNz"><a href="#api-PayrollNz">Methods</a></li>
               <li data-group="PayrollNz" data-name="approveTimesheet" class="">
                 <a href="#api-PayrollNz-approveTimesheet">approveTimesheet</a>
diff --git a/docs/v1/payroll-uk/index.html b/docs/v1/payroll-uk/index.html
index 792b2520..7a6c91d4 100644
--- a/docs/v1/payroll-uk/index.html
+++ b/docs/v1/payroll-uk/index.html
@@ -3517,7 +3517,7 @@
           <nav id="scrollingNav">
             <ul class="sidenav nav nav-list">
               <li class="nav-header" data-group="PayrollUk"><strong>SDK: </strong><span id='sdk-name'></span></li>
-              <li class="nav-header" data-group="PayrollUk"><strong>VSN: </strong>7.0.0</li>
+              <li class="nav-header" data-group="PayrollUk"><strong>VSN: </strong>7.0.1</li>
               <li class="nav-header" data-group="PayrollUk"><a href="#api-PayrollUk">Methods</a></li>
               <li data-group="PayrollUk" data-name="approveTimesheet" class="">
                 <a href="#api-PayrollUk-approveTimesheet">approveTimesheet</a>
diff --git a/docs/v1/projects/index.html b/docs/v1/projects/index.html
index fb6d5040..2ea5c61c 100644
--- a/docs/v1/projects/index.html
+++ b/docs/v1/projects/index.html
@@ -1462,7 +1462,7 @@
           <nav id="scrollingNav">
             <ul class="sidenav nav nav-list">
               <li class="nav-header" data-group="Project"><strong>SDK: </strong><span id='sdk-name'></span></li>
-              <li class="nav-header" data-group="Project"><strong>VSN: </strong>7.0.0</li>
+              <li class="nav-header" data-group="Project"><strong>VSN: </strong>7.0.1</li>
               <li class="nav-header" data-group="Project"><a href="#api-Project">Methods</a></li>
               <li data-group="Project" data-name="createProject" class="">
                 <a href="#api-Project-createProject">createProject</a>
diff --git a/setup.py b/setup.py
index 7a45268c..ea787609 100644
--- a/setup.py
+++ b/setup.py
@@ -48,5 +48,5 @@ def read_file(filename):
     keywords="xero python sdk API oAuth",
     name="xero_python",
     packages=find_packages(include=["xero_python", "xero_python.*"]),
-    version="7.0.0",
+    version="7.0.1",
 )
diff --git a/tests/test_api_client/test_oauth2.py b/tests/test_api_client/test_oauth2.py
index dd8eb6ee..cc2eb2c1 100644
--- a/tests/test_api_client/test_oauth2.py
+++ b/tests/test_api_client/test_oauth2.py
@@ -151,6 +151,49 @@ def test_auth2_refresh_access_token():
     assert oauth2_token.access_token == new_token["access_token"]
     assert oauth2_token.refresh_token == new_token["refresh_token"]
 
+def test_auth2_refresh_access_token_having_scope_as_string():
+    # given OAuth2Token with expired access_token
+    api_client = FakeClass()
+    api_client.set_oauth2_token = FakeMethod()
+    refresh_token = "refresh-token-value"
+    scope = (
+        "email profile openid accounting.reports.read "
+        "accounting.attachments.read accounting.settings "
+        "accounting.settings.read accounting.attachments "
+        "accounting.transactions accounting.journals.read "
+        "accounting.transactions.read accounting.contacts "
+        "accounting.contacts.read offline_access"
+    )
+    new_token = {
+        "id_token": "new-id-token-value",
+        "access_token": "new-access-token-value",
+        "expires_in": 1800,
+        "expires_at": time.time() + 1800,
+        "token_type": "Bearer",
+        "refresh_token": "new-refresh-token-value",
+        "scope": scope,
+    }
+    oauth2_token = OAuth2Token(client_id="client_id", client_secret="client_secret")
+    oauth2_token.refresh_token = refresh_token
+    oauth2_token.scope = scope
+    oauth2_token.fetch_access_token = FakeMethod(return_value=new_token)
+    # When refreshing access_token
+    assert oauth2_token.refresh_access_token(api_client=api_client)
+    # Then expected set new token function called
+    assert len(oauth2_token.fetch_access_token.calls) == 1
+    assert len(api_client.set_oauth2_token.calls) == 1
+    call_args, call_kwargs = api_client.set_oauth2_token.calls[0]
+    assert call_args == (new_token,)
+    assert call_kwargs == {}
+    # Then expected new valid access and refresh tokens set on oauth2_token
+    assert oauth2_token.expires_at == new_token["expires_at"]
+    assert oauth2_token.is_access_token_valid()
+    assert oauth2_token.id_token == new_token["id_token"]
+    assert oauth2_token.expires_in == new_token["expires_in"]
+    assert oauth2_token.token_type == new_token["token_type"]
+    assert oauth2_token.scope == new_token["scope"]
+    assert oauth2_token.access_token == new_token["access_token"]
+    assert oauth2_token.refresh_token == new_token["refresh_token"]
 
 def test_auth2_fetch_access_token():
     # Given OAuth2Token with valid refresh_token
diff --git a/xero_python/__init__.py b/xero_python/__init__.py
index ffe4266a..36aa66ac 100644
--- a/xero_python/__init__.py
+++ b/xero_python/__init__.py
@@ -2,4 +2,4 @@
 
 __author__ = """Xero Developer API"""
 __email__ = "api@xero.com"
-__version__ = "7.0.0"
+__version__ = "7.0.1"
diff --git a/xero_python/api_client/oauth2.py b/xero_python/api_client/oauth2.py
index ca7aab20..f2277425 100644
--- a/xero_python/api_client/oauth2.py
+++ b/xero_python/api_client/oauth2.py
@@ -213,7 +213,7 @@ def can_refresh_access_token(self):
         """
         return (
             self.refresh_token
-            and isinstance(self.scope, (list, tuple))
+            and isinstance(self.scope, (list, tuple, str))
             and self.client_id
             and self.client_secret
         )
diff --git a/xero_python/docs/README.md b/xero_python/docs/README.md
index f82f9b4a..eafa6285 100644
--- a/xero_python/docs/README.md
+++ b/xero_python/docs/README.md
@@ -4,7 +4,7 @@ These endpoints are related to managing authentication tokens and identity for X
 The `xero_python` package is automatically generated by the [XeroAPI SDK 2.0 Codegen](https://github.com/xero-github/xeroapi-sdk-codegen) project:
 
 - API version: 7.0.0
-- Package version: 7.0.0
+- Package version: 7.0.1
 - Build package: org.openapitools.codegen.languages.PythonClientCodegen
 For more information, please visit [https://developer.xero.com](https://developer.xero.com)