Adding E2E Cypress Test to ensure the app is working correctly! #63

Mau-MD · 2022-08-05T17:37:49Z

App Context

Learning U is a platform where software engineers can go and learn new technologies in an organized way. They can track their progress and get suggestions about what courses to take in both visual and written ways.

PR Context

Right now almost all test have been manual. This is not a good practice since we can break the application without even noticing it

This PR

This adds automated tests on core features that ensures the app is working as normal.

IMPORTANT:

Since this PR was based in another one, a lot of commits appear here even if they are not related to this PR. In reaility commits start here:

How to preview a PR on Vercel

Go to /login and log in with username admin and password admin123
Go to /dashboard

… frontend inside code form

… the code form

… form

…form

…nts to delete a course

…e a course

…utton exists somewhere else in the ap

…eir own custom courses

…ew form from scratch

… inside a state to be able to render embed videos

… url

…eturn its id

…ourse

… made the course

… to the backend

vercel · 2022-08-05T17:37:51Z

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated
learning-u	❌ Failed (Inspect)		Aug 5, 2022 at 5:37PM (UTC)

nosoyilse

Amazing that you were able to add tests!

I think it is a better practice to add the test in the same PR as the functionality, but regardless is very good that you had time to add this

Mau-MD added 30 commits July 18, 2022 11:04

Fetching course progress and showing it inside popover

0a953ce

fixed a bug where advanced progress was the same as beginner progress

ca0da67

Adding feed route and dummy component

0100bf6

adding post route

13f512b

adding createPost function

8302bb2

changed userId to ParseUser

4d805e8

Adding create post route

ba3c2f9

creating getPostsByUser function

bc73f99

creating get posts created by me

fd421e8

Adding chakra ui modal component

7925b6d

Adding clipboard hook to the modal

7f8aff9

Adding modal inside course card

2f8ff38

Adding helper text and code propr

17fdc89

Adding form to input course code

9bd46b1

Adding state to conditional show course input

e849f6d

adding parse object to json util function

e4e3a7e

Adding function to clone a complete code based on its courseId

40a2940

making the function returning the course

b4d8278

adding clone course route

bf929d3

Adding name form input inside Course Code Form

5a89fd3

Adding clone course mutation that lets the user clone a course in the…

32d415e

… frontend inside code form

Adding toast for when the user succeds/fails to clone a course inside…

0e5a8e2

… the code form

Redirecting the user when the course gets cloned successfully in code…

e8b6c71

… form

Adding frontend form validation with Yup inside code form

84e29bf

Adding error when course doesn't exist when cloning a course in code …

c29e2e2

…form

Adding basic modal to display when the user wants to delete a course

149ddd1

adding logic to open and close the modal to delete a course

98e2af6

Adding a card that to display in the delete course modal when user wa…

d02730d

…nts to delete a course

Adding grid to display multiple video cards when user want's to delet…

cd1a22f

…e a course

Typo

fd735da

Mau-MD added 27 commits August 1, 2022 11:49

removing submit existing course button from feature page since that b…

edcc3db

…utton exists somewhere else in the ap

Adding create new course from scratch route and component

92300e1

Adding link to new course from scratch page

7198364

Adding inputs inside course from scratch page so people can create th…

2ae9e3c

…eir own custom courses

Adding formik to handle form input when the users wants to create a n…

3a47a8e

…ew form from scratch

adding a function to check if an url is a youtube, and it so, save it…

dd2a999

… inside a state to be able to render embed videos

Creating component to render iframe youtube video

02ad3f0

saving the embed url instead of the youtube url

b6113f5

nit

c4173a2

conditionally rendering youtube embeds when inputting a valid youtube…

be9f905

… url

adding a function to create a course from scratch given youtube urls

24a69c2

adding a function to check if a youtube url is valid, and if it is, r…

03215ee

…eturn its id

adding a route to let the users create a course given youtube urls

62921dd

avoiding fetching featured courses when fecthing user courses

207462c

linking featured course to a user so it can display who created the c…

086f062

…ourse

including user info when fetching featured courses so it can show who…

10ff772

… made the course

Adding mutation to create a new course from scratch and send the data…

13be4e6

… to the backend

Adding a text that shows who created a featured course

ecaac5f

changing createdBy text to be a link button

a968dba

adding base url to cypress config

3eac27f

adding command to login user

536ed03

adding data-test ids to multiple components

8beb41d

adding command to register a user

b98e5fd

creating tests to check if login and registration works

83ba447

Adding more test ids

398a51e

adding a test that creates a new course

2559bd2

adding tests that check if the usser can follow and make a new posr

6bd815b

nosoyilse approved these changes Aug 5, 2022

View reviewed changes

Mau-MD merged commit 6bd815b into main Aug 5, 2022

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Adding E2E Cypress Test to ensure the app is working correctly! #63

Adding E2E Cypress Test to ensure the app is working correctly! #63

Mau-MD commented Aug 5, 2022

vercel bot commented Aug 5, 2022

nosoyilse left a comment

Adding E2E Cypress Test to ensure the app is working correctly! #63

Adding E2E Cypress Test to ensure the app is working correctly! #63

Conversation

Mau-MD commented Aug 5, 2022

App Context

PR Context

This PR

Next

IMPORTANT:

How to preview a PR on Vercel

vercel bot commented Aug 5, 2022

nosoyilse left a comment

Choose a reason for hiding this comment