Skip to content
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

Jump to bottom

Update context.py to reload when using CDP remote browser - Unit Tests #686

Merged
merged 2 commits into from
Feb 12, 2025
Merged

Update context.py to reload when using CDP remote browser - Unit Tests #686

merged 2 commits into from
Feb 12, 2025

Conversation

codebeaver-ai[bot]
Copy link
Contributor

@codebeaver-ai codebeaver-ai bot commented Feb 12, 2025

CodeBeaver PR Summary

I started working from Update context.py to reload when using CDP remote browser

🔄 1 test added.
🐛 No bugs detected in your changes
🛠️ 1/2 tests passed

🔄 Test Updates

I've added 1 tests. They all pass ☑️
New Tests:

  • tests/test_context.py

No existing tests required updates.

🐛 Bug Detection

No bugs detected in your changes. Good job!

🛠️ Test Results

1/2 tests passed ⚠️

tests/test_models.py

View error 
tests/test_models.py:84: in <module>
    ChatGoogleGenerativeAI(model='gemini-2.0-flash-exp', api_key=api_key_gemini),
/usr/local/lib/python3.11/site-packages/langchain_core/load/serializable.py:125: in __init__
    super().__init__(*args, **kwargs)
/usr/local/lib/python3.11/site-packages/langchain_google_genai/chat_models.py:841: in validate_environment
    self.client = genaix.build_generative_service(
/usr/local/lib/python3.11/site-packages/langchain_google_genai/_genai_extension.py:276: in build_generative_service
    return v1betaGenerativeServiceClient(**config)
/usr/local/lib/python3.11/site-packages/google/ai/generativelanguage_v1beta/services/generative_service/client.py:667: in __init__
    self._transport = transport_init(
/usr/local/lib/python3.11/site-packages/google/ai/generativelanguage_v1beta/services/generative_service/transports/grpc.py:235: in __init__
    super().__init__(
/usr/local/lib/python3.11/site-packages/google/ai/generativelanguage_v1beta/services/generative_service/transports/base.py:100: in __init__
    credentials, _ = google.auth.default(
/usr/local/lib/python3.11/site-packages/google/auth/_default.py:719: in default
    raise exceptions.DefaultCredentialsError(_CLOUD_SDK_MISSING_CREDENTIALS)
E   google.auth.exceptions.DefaultCredentialsError: Your default credentials were not found. To set up Application Default Credentials, see https://cloud.google.com/docs/authentication/external/set-up-adc for more information.

tests/test_models.py

☂️ Coverage Improvements

Coverage improvements by file:

  • tests/test_context.py

    New coverage: 40.08%
    Improvement: +40.08%

🎨 Final Touches

  • I ran the hooks included in the pre-commit config.

Settings | Logs | CodeBeaver

@MagMueller MagMueller merged commit 97ecf70 into main Feb 12, 2025
1 check passed
@codebeaver-ai codebeaver-ai bot deleted the codebeaver/patch-1-682 branch February 12, 2025 16:39
AryamanParida pushed a commit to AryamanParida/browser-use that referenced this pull request Mar 7, 2025
@MagMueller
Merge pull request browser-use#686 from browser-use/codebeaver/patch-…
0183181 
…1-682

Update context.py to reload when using CDP remote browser - Unit Tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@MagMueller