Skip to content

Time - Haben and Jocelyn#15

Open
wangjoc wants to merge 15 commits into
Ada-C13:masterfrom
wangjoc:master
Open

Time - Haben and Jocelyn#15
wangjoc wants to merge 15 commits into
Ada-C13:masterfrom
wangjoc:master

Conversation

@wangjoc

@wangjoc wangjoc commented Apr 26, 2020

Copy link
Copy Markdown

Inspiration Board

Congratulations! You're submitting your assignment!

Comprehension Questions

Question Answer
What is the flow of data from input on the form to being a new card on the board? The flow of data starts from the NewCardForm component where it is collected from the through the two input text fields. Once the Add New card is clicked, the onClick event for the button is triggered, which calls on the createCard function from the parent component of Board. The createCard function posts the message and emoji to the relevant board hosted in heroku, which then makes it available to be pulled down and presented in the Board.
What function did you use to make the GET request from the API to get the list of cards? Why use that function? We used the get.then method through axios to receive the information and the map it to a card. We used this function because the get request would provide us with a response that we then needed to parse out further to get the relevant info.
How do snapshot tests differ from unit tests? Snapshot test differ than unit test because you write them after the code has been written. To update the tests, you need to create new snapshots. It's important to avoid automatically updating the snapshots after you make a change in code to make sure that the changes you pushed through are actually correct (or else you may end up with snapshot that captures the app incorrectly.
What purpose does Enzyme serve in testing a React app? Enzyme makes it easie to test react component's output. It allows you to manipulate, traverse, and simulate runtime given the output.

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.

2 participants