You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@texera.apache.org by "Arq0n (via GitHub)" <gi...@apache.org> on 2026/06/19 01:22:49 UTC
[PR] test(auth): add SessionUser spec [texera]
Arq0n opened a new pull request, #5785:
URL: https://github.com/apache/texera/pull/5785
## Purpose
Add dedicated unit coverage for `SessionUser`, which wraps a generated jOOQ `User` POJO and exposes it as a `java.security.Principal`.
## Summary of Changes
- Added `SessionUserSpec.scala` under `common/auth/src/test/scala/org/apache/texera/auth/`.
- Verifies that `SessionUser`:
- delegates `Principal#getName` to the wrapped user
- delegates `getUid`, `getEmail`, and `getGoogleId`
- returns the same wrapped `User` instance from `getUser`
- checks roles by exact `UserRoleEnum` equality through `isRoleOf`
## How Was This PR Tested?
```bash
sbt "Auth / testOnly org.apache.texera.auth.SessionUserSpec"
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@texera.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] test(auth): add SessionUser spec [texera]
Posted by "aglinxinyuan (via GitHub)" <gi...@apache.org>.
aglinxinyuan commented on PR #5785:
URL: https://github.com/apache/texera/pull/5785#issuecomment-4749132944
The task is already taken.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@texera.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] test(auth): add SessionUser spec [texera]
Posted by "aglinxinyuan (via GitHub)" <gi...@apache.org>.
aglinxinyuan closed pull request #5785: test(auth): add SessionUser spec
URL: https://github.com/apache/texera/pull/5785
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@texera.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] test(auth): add SessionUser spec [texera]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #5785:
URL: https://github.com/apache/texera/pull/5785#issuecomment-4747472628
<!-- texera:welcome-first-time-contributor -->
👋 Thanks for your first contribution to Texera, @Arq0n!
If you're looking for a good place to start, browse issues labeled [`starter-task`](https://github.com/apache/texera/issues?q=is%3Aissue+is%3Aopen+label%3Astarter-task); they're scoped to be approachable for newcomers.
You can drive common housekeeping yourself by commenting one of these commands on its own line:
- **Issues.** Comment `/take` to assign an open issue to yourself, or `/untake` to release it. You can find unclaimed work with the search filter `is:issue is:open no:assignee`.
- **Sub-issues.** To link issues into a parent/child hierarchy, comment `/sub-issue #5166 #5222` on the parent to attach those children (or `/unsub-issue #5166 #5222` to detach them). From a child issue, comment `/parent-issue #5166` to set its parent, or `/unparent-issue` to clear it (the current parent is detected automatically). References may be written as `#5166` or as a bare `5166`; cross-repository references are not supported.
- **Pull requests (author only).** Comment `/request-review @user` to request a review from someone, or `/unrequest-review @user` to withdraw that request.
Each command must match exactly: `/take this` will not work, only `/take` does. For the full contribution flow, see [CONTRIBUTING.md](https://github.com/apache/texera/blob/main/CONTRIBUTING.md).
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@texera.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] test(auth): add SessionUser spec [texera]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #5785:
URL: https://github.com/apache/texera/pull/5785#issuecomment-4747472740
<!-- texera:template-compliance -->
👋 Thanks for opening this pull request, @Arq0n!
It looks like the pull request description doesn't quite follow our template yet:
- The **What changes were proposed in this PR?** section is missing; please keep the template's headings.
- The **How was this PR tested?** section is missing; please keep the template's headings.
- The **Was this PR authored or co-authored using generative AI tooling?** section is missing; please keep the template's headings.
Filling out the template helps reviewers understand and triage your contribution faster. Please edit the description to complete it. This message will disappear automatically once the template is followed.
You can find the template prompts by editing the description, or see [CONTRIBUTING.md](https://github.com/apache/texera/blob/main/CONTRIBUTING.md) for the full contribution flow.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@texera.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org