Welcome to the Treehouse Community

The Treehouse Community is a meeting place for developers, designers, and programmers of all backgrounds and skill levels to get support. Collaborate here on code errors or bugs that you need feedback on, or asking for an extra set of eyes on your latest project. Join thousands of Treehouse students and alumni in the community today. (Note: Only Treehouse students can comment or ask questions, but non-students are welcome to browse our conversations.)

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and a supportive community. Start your free trial today.

Java Unit Testing in Java How to Test Using Before to not repeat yourself

I don't understand

I am looking at this question and my mind is going blank. I tried to just take the instantiation and put it in the @Before statement but I guess that wasn't right. What should I do?

com/example/CalculatorTest.java
package com.example;

import org.junit.Before;
import org.junit.Test;

import static org.junit.Assert.*;

public class CalculatorTest {

    @Before
    public void setUp() throws Exception {
      Calculator calculator = new Calculator();
    }

    @Test
    public void addingMultipleNumbersProducesResult() throws Exception {


        int answer = calculator.addNumbers(1 ,2, 3);

        assertEquals(6, answer);
    }

    @Test
    public void addingSingleNumberTotalsAppropriately() throws Exception {


        int answer = calculator.addNumbers(1);

        assertEquals(1, answer);
    }
}
com/example/Calculator.java
package com.example;

public class Calculator {

    public int addNumbers(int... numbers) {
        int total = 0;
        for (int number : numbers) {
            total += number;
        }
        return total;
    }
}

2 Answers

Seth Kroger
Seth Kroger
56,392 Points

You need to declare calculator as an instance variable first, not as a local variable in setUp():

    private Calculator calculator;

    @Before
    public void setUp() throws Exception {
      calculator = new Calculator();
    }

The way you have declared your calculator, it is local to the setup method. As soon as setup is complete, it ceases to exist. You need to make it available to all methods of this class. How do you normally make a variable available to all methods of a class? By making it a member variable.

You should still initialize it in the Setup method. Just don't declare it there.