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

Make IRODSTaskTestBase methods public #2002

Closed
mikkonie opened this issue Sep 13, 2024 · 1 comment
Closed

Make IRODSTaskTestBase methods public #2002

mikkonie opened this issue Sep 13, 2024 · 1 comment
Assignees
@mikkonie
Labels
app: taskflowbackend Issue in the taskflowbackend app internal Changes invisible to the user or APIs (e.g. refactoring and optimization)
Milestone
v1.0.0

Comments

@mikkonie
Copy link
Contributor

There is no logic to these being protected as they are expected to be used by other classes.

Yes, the fact that the using classes will inherit these technically makes the access ok, but it's still misleading and redundant to protect them
@mikkonie mikkonie added internal Changes invisible to the user or APIs (e.g. refactoring and optimization) app: taskflowbackend Issue in the taskflowbackend app labels Sep 13, 2024
@mikkonie mikkonie added this to the v1.0.0 milestone Sep 13, 2024
@mikkonie mikkonie self-assigned this Sep 13, 2024
mikkonie added a commit that referenced this issue Sep 13, 2024
@mikkonie
refactor tests (#1991, #2001, #2002)
3ee4e53
@mikkonie
Copy link
Contributor Author

Done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mikkonie mikkonie

Labels
app: taskflowbackend Issue in the taskflowbackend app internal Changes invisible to the user or APIs (e.g. refactoring and optimization)
Projects
None yet
Milestone
v1.0.0
Development

No branches or pull requests
1 participant
@mikkonie