-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
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 private, local only, API for usage in E2E tests #1429
Conversation
b21dbcf
to
0e8975a
Compare
e534171
to
2ed16e0
Compare
b939f9c
to
3d56c3a
Compare
4bc78c1
to
cade0a4
Compare
7025d31
to
2991258
Compare
frontend/tests/utils/privateApi.ts
Outdated
@@ -0,0 +1,21 @@ | |||
// @ts-ignore |
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.
@alejsdev what do you think of having this?
currently without this running the build test would fail. The other option would be to ignore the test files when running build, which maybe is better?
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.
We could ignore the test files while running the build :)
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.
@alejsdev done! I had to make a new config file that ignores the whole test folder :)
4e0ed8f
to
bda9841
Compare
24e98f8
to
3cb25ed
Compare
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.
Nice! A couple of small comments.
|
||
assert user | ||
assert user.email == "[email protected]" | ||
assert user.full_name == "Pollo Listo" |
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.
🤣 🐔
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.
Nice! 🚀 🐔 🎉
No description provided.