Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fixed few errors #10

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

BryanSJamesDev
Copy link

Here are the errors found in the code:

  1. The import statements are missing for the assertions from the JUnit framework (import static org.junit.Assert.*;).

  2. The class Dice is not defined or imported in the code. The line that creates an instance of Dice is commented out.

  3. The method Dice.roll(faces) suggests that roll is a static method of the Dice class, but it is not defined or imported in the code.

  4. The commented out line with double tolerance = expected * 0.005; suggests the use of a tolerance value, but it is not used in the assertEquals assertion.

  5. The commented out lines containing System.out.print are not terminated with a semicolon, causing a compilation error.

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.

1 participant