Skip to content

Conversation

@busunkim96
Copy link
Contributor

No description provided.

@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Nov 18, 2021
@busunkim96 busunkim96 requested a review from tseaver November 18, 2021 16:10
@tseaver
Copy link
Contributor

tseaver commented Nov 18, 2021

@busunkim96 I don't understand the systest failures. They all blow up with:

______________ ERROR at setup of test_refresh_no_scopes[urllib3] _______________

service_account_file = '/tmpfs/src/github/google-auth-library-python/system_tests/system_tests_sync/../data/service_account.json'

    @pytest.fixture
    def credentials(service_account_file):
>       yield service_account.Credentials.from_service_account_file(service_account_file)

system_tests_sync/test_service_account.py:25:
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _
.nox/service_account_sync-3-7/lib/python3.7/site-packages/google/oauth2/service_account.py:239: in from_service_account_file
    filename, require=["client_email", "token_uri"]
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _

filename = '/tmpfs/src/github/google-auth-library-python/system_tests/system_tests_sync/../data/service_account.json'
require = ['client_email', 'token_uri']

    def from_filename(filename, require=None):
        """Reads a Google service account JSON file and returns its parsed info.

        Args:
            filename (str): The path to the service account .json file.
            require (Sequence[str]): List of keys required to be present in the
                info.

        Returns:
            Tuple[ Mapping[str, str], google.auth.crypt.Signer ]: The verified
                info and a signer instance.
        """
>       with io.open(filename, "r", encoding="utf-8") as json_file:
E       FileNotFoundError: [Errno 2] No such file or directory: '/tmpfs/src/github/google-auth-library-python/system_tests/system_tests_sync/../data/service_account.json'

@busunkim96
Copy link
Contributor Author

Oops I accidentally deleted the other secret files before tarring in the last commit. Fixed in 30094a6

@tseaver tseaver merged commit 3c3fbf4 into main Nov 18, 2021
@tseaver tseaver deleted the update-refresh-token branch November 18, 2021 20:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants