-
Notifications
You must be signed in to change notification settings - Fork 19
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add test: connected user can be retrieved #247
Conversation
PR Review ChecklistDo not edit the content of this comment. The PR reviewer should simply update this comment by ticking each review item below, as they get completed. Trivial Change
Code
Architecture
|
connection/user.feature
Outdated
@@ -36,6 +36,14 @@ Feature: Connection Users | |||
Given users delete: user | |||
Then users not contains: user | |||
|
|||
Scenario: users can get themselves |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is not really in our 'style' grammatically? Bit informal - try calling this 'connected user is retrievable' and call the step 'Then get connected user' instead of 'user get self'?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agreed, done.
…cted user (#413) ## What is the goal of this PR? We've added a behaviour test step `get connected user` in line with changes made in typedb/typedb-behaviour#247 ## What are the changes implemented in this PR? Added user step: `get connected user`.
What is the goal of this PR?
We've added a behaviour test scenario testing that the connected user is retrievable. This checks that the client provides a way to get the currently logged in user.
What are the changes implemented in this PR?
connected user is retrievable
.