Skip to content

test: add auth controller tests#198

Open
danielsp45 wants to merge 3 commits intomainfrom
dp/add-auth-controller-tests
Open

test: add auth controller tests#198
danielsp45 wants to merge 3 commits intomainfrom
dp/add-auth-controller-tests

Conversation

@danielsp45
Copy link
Copy Markdown
Member

No description provided.

@reviewpad
Copy link
Copy Markdown

reviewpad Bot commented Sep 30, 2023

AI-Generated Summary: This pull request features multiple changes aimed at enhancing the testing of the auth controller. A new test file, auth_controller_test.exs, has been added with comprehensive sign-up, sign-in, show, sign-out, update, and create test cases. Additionally, the config/test.exs file has been updated to include a frontend URL configuration. A password parameter has been added to the User struct in the accounts_factory.ex file to facilitate testing.

Comment thread test/bokken_web/controllers/auth_controller_test.exs Outdated
Comment thread test/bokken_web/controllers/auth_controller_test.exs Outdated
Comment thread test/bokken_web/controllers/auth_controller_test.exs
Comment thread test/bokken_web/controllers/auth_controller_test.exs Outdated
Comment thread test/bokken_web/controllers/auth_controller_test.exs
Comment thread test/support/factories/accounts_factory.ex
@danielsp45 danielsp45 force-pushed the dp/add-auth-controller-tests branch from d101dd3 to fe426b1 Compare October 6, 2023 20:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants