feat: add hip hop artist classes and corresponding tests #512
Conversation
| @@ -0,0 +1,36 @@ | |||
| package com.codedifferently.lesson16.hiphopArtistTest; | |||
There was a problem hiding this comment.
These are not the appropriate tests (and they are in the wrong package). You only really need one test file for HipHopArtist and should create proper tests using the "Arrange, Act, Assert" pattern.
Replace all of these tests with the correct one for HipHopArtist and that will straighten things out.
| public class DMXtest { | ||
| @Test | ||
| public void testDMX() { | ||
| DMX.main(new String[0]); // Call the main method of DMX class |
There was a problem hiding this comment.
Test methods must have at least one assertion.
| import java.util.ArrayList; | ||
|
|
||
| public class DMX { | ||
| public static void main(String[] args) { |
There was a problem hiding this comment.
You shouldn't have added any new main methods since there's already one that I've set up. What you should have actually done is made a HipHopArtistTest class in the test project and use test methods to create instances of HipHopArtist objects like you're doing here and then assert that the object methods work correctly.
bryanasb251
force-pushed
the
feat/Lesson_16
branch
from
0c08c4a to
9aa5210
Compare
|
Removed unnecessary classes and add hiphopArtistTest |
Signed-off-by: Anthony D. Mays <anthony@morganlatimer.com>
Signed-off-by: Anthony D. Mays <anthony@morganlatimer.com>
Chat gpt for exception and break down.