Skip to content

Add fundamental unit test coverage of griddaemons login module#575

Merged
jonasbardino merged 4 commits into
nextfrom
add/fundamental-unit-test-coverage-of-griddaemons-login-module
Jun 16, 2026
Merged

Add fundamental unit test coverage of griddaemons login module#575
jonasbardino merged 4 commits into
nextfrom
add/fundamental-unit-test-coverage-of-griddaemons-login-module

Conversation

@jonasbardino

Copy link
Copy Markdown
Contributor

One of the core helpers for the work to extend support for UUID user ID as home dir in #561 .

@jonasbardino jonasbardino self-assigned this Jun 15, 2026
@jonasbardino jonasbardino added the test-only Improvements or additions solely for better test coverage - without functionality changes label Jun 15, 2026
@jonasbardino jonasbardino force-pushed the add/fundamental-unit-test-coverage-of-griddaemons-login-module branch from 627bb6b to c22a09b Compare June 15, 2026 13:59
@jonasbardino jonasbardino requested a review from a team June 15, 2026 14:41
@jonasbardino jonasbardino marked this pull request as ready for review June 15, 2026 14:42
Comment thread tests/test_mig_shared_griddaemons_login.py
Comment thread tests/test_mig_shared_griddaemons_login.py

@Martin-Rehr Martin-Rehr left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved with only a few minor comments

@jonasbardino jonasbardino merged commit 1bec578 into next Jun 16, 2026
11 checks passed
@jonasbardino jonasbardino deleted the add/fundamental-unit-test-coverage-of-griddaemons-login-module branch June 16, 2026 13:19
jonasbardino added a commit that referenced this pull request Jun 16, 2026
jonasbardino added a commit that referenced this pull request Jun 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test-only Improvements or additions solely for better test coverage - without functionality changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants