Skip to content
This repository has been archived by the owner on Feb 14, 2024. It is now read-only.

Support retrieving the list of all users #4

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

caarmen
Copy link

@caarmen caarmen commented Dec 15, 2021

Add a findAll() function to the UserRepository.

Return an empty list if there are no users.

Add a `findAll()` function to the `UserRepository`.

Return an empty list if there are no users.
@caarmen

This comment has been minimized.

@@ -36,6 +36,9 @@ class UserRepositorySpec extends RepositorySpec {
}
}

test("find all users query") {
check(UserStatement.findAll())
}
Copy link
Author

Choose a reason for hiding this comment

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

Not sure to understand this very well :)

I also wanted to add a test similar to the "Find a user" test, but for "Find all users". But I was always ending up with a list of just one item. i guess I expected 3 items (all the test users). But I guess that's not how it works 🤷

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant