Conversation
Delete compromised file
| test("Ensure that Video Card is rendered", () => { | ||
|
|
||
|
|
||
| }); | ||
|
|
||
| test("Ensure that Thumbnail in Video Card match with the one in the mock item", () => { | ||
|
|
||
|
|
||
| }); | ||
|
|
||
| test("Ensure that Title in Video Card match with the mock item selected", () => { | ||
|
|
||
|
|
||
| }); | ||
|
|
||
| test("Ensure that Title in Video Card match with the mock item selected", () => { | ||
|
|
||
|
|
||
| }); |
There was a problem hiding this comment.
Comment this section if you will add those scenarios in another PR
| describe('Header Test', () => { | ||
|
|
||
|
|
||
| test("Ensure all App is getting rendered", () => { | ||
|
|
||
|
|
||
| }); | ||
| }) No newline at end of file |
There was a problem hiding this comment.
Comment this test scenario if you are not using it
| describe('App Test', () => { | ||
|
|
||
| test("Ensure all videos in the mock file are getting rendered", () => { | ||
|
|
||
|
|
||
| }); | ||
|
|
||
| }) No newline at end of file |
There was a problem hiding this comment.
Comment this test scenario if you are not using it
| test("Ensure that main video is rendered", () => { | ||
|
|
||
|
|
||
| }); | ||
|
|
||
| test("Ensure that Title in Video Card match with the mock item selected", () => { | ||
|
|
||
|
|
||
| }); | ||
|
|
||
| test("Ensure that Description is displayed selected", () => { | ||
|
|
||
|
|
||
| }); |
There was a problem hiding this comment.
Comment this tests if you are going to add it in another PR
| test("Ensure that Video Card is rendered", () => { | ||
| }); | ||
|
|
||
| test("Ensure that Thumbnail in Video Card match with the one in the mock item", () => { | ||
| }); | ||
|
|
||
| test("Ensure that Title in Video Card match with the mock item selected", () => { | ||
|
|
||
| }); | ||
|
|
||
| test("Ensure that Title in Video Card match with the mock item selected", () => { |
There was a problem hiding this comment.
Comment this tests if you are going to add in another PR
| import styled from 'styled-components'; | ||
|
|
||
|
|
||
| const MenuButton = styled.button` |
There was a problem hiding this comment.
This component seems to be duplicated from src/components/Header/LoginControl/LoginControl.component.jsx
| } | ||
|
|
||
| return( | ||
| <form onSubmit={onSubmitQuery}> |
There was a problem hiding this comment.
The search method is triggered only once, it is not taking more inputs for search.
|
|
||
|
|
||
|
|
||
| const VideoCard =(props) =>{ |
There was a problem hiding this comment.
You can use destructuring to improve readability on the props.
const VideoCard = ({ id, snippet, setShowDetail }) => {}| `; | ||
|
|
||
| function VideoDetail(props){ | ||
| console.log(props) |
|
|
||
|
|
||
|
|
||
| const VideoList =(props) => { |
There was a problem hiding this comment.
Try to use destructuring on each component you create, it would improve readability
2 participants
Update to display videos using youtube API v3, update css and views from previous verison, and add some tests