Skip to content

Split-off download_file_from_url method from download_attachment #407

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
herronelou wants to merge 1 commit into
base: master
Choose a base branch
from

#387 Split-off download_file_from_url method from download_attachment…

74fc43c
Select commit
Loading
Failed to load commit list.
Open

Split-off download_file_from_url method from download_attachment #407

#387 Split-off download_file_from_url method from download_attachment…
74fc43c
Select commit
Loading
Failed to load commit list.
Azure Pipelines / python-api failed Jul 16, 2025 in 1m 35s

Build #20250716.21 had test failures

19 errors / 6 warnings

Details

Tests

  • Failed: 1,440 (69.26%)
  • Passed: 639 (30.74%)
  • Other: 0 (0.00%)
  • Total: 2,079
Code coverage

  • 780 of 1984 lines covered (39.31%)

Annotations

Check failure on line 39 in Build log

See this annotation in the file changed.

@azure-pipelines azure-pipelines / python-api

Build log #L39 

Bash exited with code '1'.

Check failure on line 253 in Build log

See this annotation in the file changed.

@azure-pipelines azure-pipelines / python-api

Build log #L253 

160 test(s) failed, 231 test(s) collected.

Check failure on line 8459 in Build log

See this annotation in the file changed.

@azure-pipelines azure-pipelines / python-api

Build log #L8459 

Bash exited with code '1'.

Check failure on line 253 in Build log

See this annotation in the file changed.

@azure-pipelines azure-pipelines / python-api

Build log #L253 

160 test(s) failed, 231 test(s) collected.

Check failure on line 1 in tests/test_api.py::TestSessionTokenAuth::test_humanuser_find

See this annotation in the file changed.

@azure-pipelines azure-pipelines / python-api

tests/test_api.py::TestSessionTokenAuth::test_humanuser_find 

Test failed

Check failure on line 1 in tests/test_api.py::TestSessionTokenAuth::test_humanuser_upload_thumbnail_for_version

See this annotation in the file changed.

@azure-pipelines azure-pipelines / python-api

tests/test_api.py::TestSessionTokenAuth::test_humanuser_upload_thumbnail_for_version 

Test failed

Check failure on line 1 in tests/test_client.py::TestShotgunClientInterface::test_client_interface

See this annotation in the file changed.

@azure-pipelines azure-pipelines / python-api

tests/test_client.py::TestShotgunClientInterface::test_client_interface 

tests/base.py:138: in setUp
    super(MockTestBase, self).setUp()
tests/base.py:93: in setUp
    self.sg = api.Shotgun(
shotgun_api3/shotgun.py:704: in __init__
    self.config.set_server_params(self.base_url)
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 

self = <shotgun_api3.shotgun._Config object at 0x7f2b039e4f40>
base_url = '$(ci_site)'

    def set_server_params(self, base_url):
        """
        Set the different server related fields based on the passed in URL.
    
        This will impact the following attributes:
    
        - scheme: http or https
        - api_path: usually /api3/json
        - server: usually something.shotgunstudio.com
    
        :param str base_url: The server URL.
    
        :raises ValueError: Raised if protocol is not http or https.
        """
        self.scheme, self.server, api_base, _, _ = urllib.parse.urlsplit(base_url)
        if self.scheme not in ("http", "https"):
>           raise ValueError("base_url must use http or https got '%s'" % base_url)
E           ValueError: base_url must use http or https got '$(ci_site)'

shotgun_api3/shotgun.py:480: ValueError
Raw output 
/home/vsts/work/1/s/shotgun_api3/shotgun.py:480: ValueError: base_url must use http or https got '$(ci_site)'

Check failure on line 1 in tests/test_client.py::TestShotgunClientInterface::test_module_interface

See this annotation in the file changed.

@azure-pipelines azure-pipelines / python-api

tests/test_client.py::TestShotgunClientInterface::test_module_interface 

tests/base.py:138: in setUp
    super(MockTestBase, self).setUp()
tests/base.py:93: in setUp
    self.sg = api.Shotgun(
shotgun_api3/shotgun.py:704: in __init__
    self.config.set_server_params(self.base_url)
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 

self = <shotgun_api3.shotgun._Config object at 0x7f2b044e7ac0>
base_url = '$(ci_site)'

    def set_server_params(self, base_url):
        """
        Set the different server related fields based on the passed in URL.
    
        This will impact the following attributes:
    
        - scheme: http or https
        - api_path: usually /api3/json
        - server: usually something.shotgunstudio.com
    
        :param str base_url: The server URL.
    
        :raises ValueError: Raised if protocol is not http or https.
        """
        self.scheme, self.server, api_base, _, _ = urllib.parse.urlsplit(base_url)
        if self.scheme not in ("http", "https"):
>           raise ValueError("base_url must use http or https got '%s'" % base_url)
E           ValueError: base_url must use http or https got '$(ci_site)'

shotgun_api3/shotgun.py:480: ValueError
Raw output 
/home/vsts/work/1/s/shotgun_api3/shotgun.py:480: ValueError: base_url must use http or https got '$(ci_site)'
View more details on Azure Pipelines