Skip to content
This repository has been archived by the owner on Sep 22, 2023. It is now read-only.

Tests for user, level and chapter features in backend #17

Open
wants to merge 8 commits into
base: develop
Choose a base branch
from

Conversation

artur-romao
Copy link
Member

No description provided.

Copy link
Member Author

@artur-romao artur-romao left a comment

Choose a reason for hiding this comment

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

In LevelControllerTest.java:

  • getLevelLeaderboard has an error of "Required request body is missing" but I think I'm passing it correctly...
    error1
  • submitLevelSolution has an error of 400 Bad Request httpPostRequest and I couldn't find what I'm doing wrong...
    error2

In UserControllerTest.java:
- Everything is working just fine but I struggled in changePassword() test because I can't instantiate a PasswordEncoder like I did in the comment and I've already tried with other similar encoders like BCryptPasswordEncoder but it does not return the same output. Although, I've already checked and the password is correctly being updated :)

@artur-romao artur-romao requested a review from Hugo1307 June 6, 2022 14:00
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