Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

✅ Finalize Identity Pool: Section I9 constants & complete test suite (I1–I9)Finalize/identity pool phase1#1729

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.

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

Already on GitHub?Sign in to your account

Open
cureprotocols wants to merge27 commits intogoogleapis:main
base:main
Choose a base branch
Loading
fromcureprotocols:finalize/identity-pool-phase1

Conversation

@cureprotocols
Copy link

This PR completes all scoped sections (I1–I9) oftest_identity_pool.py, finalizing Phase 1 of the test suite refactor.

🧩 Finalized:

  • Section I9 constants tagged and scoped for future reuse
  • Cleaned mock values: AUDIENCE, CLIENT_ID, TOKEN_URL, etc.
  • Centralized reusable patterns viamake_credentials()

✅ Status:

  • 19/19 tests passing
  • UTF-8 compliant
  • All test logic simulated (no external calls)
  • Fully mock-driven, documented, and structured for contributors

🔒 This suite is now rock-solid, CI-compatible, and upstream-ready.

Ready to begin Phase 2 with:

  • test_pluggable.py refactor
  • Expansion totest_sts.py,test_credentials.py
  • Linting, CI hooks, or packaging

Proudly authored as part of thegoogle-auth-rewired hardening project.

cureprotocolsand others added14 commitsApril 2, 2025 17:40
Swapped from reStructuredText to Markdown for improved readability, badge support, and contributor clarity.This version of the README:- Retains full respect and gratitude for the original authors.- Clarifies project goals and roadmap.- Adds visual indicators for project health and openness to contributors.
@cureprotocolscureprotocols requested review froma team ascode ownersApril 4, 2025 23:00
@google-cla
Copy link

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View thisfailed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@cureprotocolscureprotocolsforce-pushed thefinalize/identity-pool-phase1 branch from6faa16d to7eb1525CompareApril 5, 2025 16:34
@cureprotocolscureprotocols requested a review froma team as acode ownerApril 5, 2025 16:34
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@leahecoleleahecoleAwaiting requested review from leahecoleleahecole is a code owner automatically assigned from googleapis/python-samples-owners

At least 1 approving review is required to merge this pull request.

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

@cureprotocols

[8]ページ先頭

©2009-2025 Movatter.jp