java - 需要 JUnit 测试方面的帮助

Question

我是 JUnit 的新手，所以我遇到了一些麻烦。我目前正在运行 JUint 4，并且我正在尝试编写一个专门应该失败的测试，但它仍然成功通过。我需要你的帮助，因为我觉得我写得正确，但它们没有按预期工作。这是我的主要类(class):

import java.util.ArrayList;
import java.util.Iterator;
import java.util.List;

public class PrimeNumbers implements Iterable<Integer> {
    private List<Integer> primes = new ArrayList<Integer>();

    /*
     * creates a list of n prime numbers
     *
     * @param n - the number of primes to compute silently treats negative
     * arguments as zero
     */
    public void computePrimes(int n) {
        int count = 1; // count of primes
        int number = 2; // number tested for primeness
        boolean isPrime; // is this number a prime

        while (count <= n) {
            isPrime = true;
            for (int divisor = 2; divisor <= number / 2; divisor++) {
                if (number % divisor == 0) {
                    isPrime = false;
                    break; // for loop
                }
            }
            if (isPrime && (number % 10 != 9)) { // THIS IS THE FAULT!!!
                primes.add(number);
                count++;
            }
            number++;
        }
    }

    @Override
    public Iterator<Integer> iterator() {
        return primes.iterator();
    }

    @Override
    public String toString() {
        return primes.toString();
    }

}

这是我的测试类:

import org.junit.Test;
import java.util.ArrayList;
import java.util.Iterator;
import java.util.List;
import static org.junit.Assert.*;

public class PrimeNumbersTest {

    @Test
    public void test(){
        //instantiate prime object, create primes list, and iterator
        PrimeNumbers prime = new PrimeNumbers();
        List<Integer> primeList = new ArrayList<Integer>();

        //array holding 19 so I can compare the objects
        int[] primeArray = new int[]{2,3,5,7,11,13,17,19};
        int i = 0;
        int n = 8;
        //call the method
        prime.computePrimes(n);
        Iterator<Integer> primeIterator = primeList.iterator();
        //loop to iterate through the list and then check last item if it is equal to 19
        while(primeIterator.hasNext()){
            if(!(primeIterator.equals(primeArray[i]))) {
                assertTrue("incorrect", primeIterator.equals(primeArray[i]));
            }
            i++;
            primeIterator.next();
        }
    }
}

谢谢大家的帮助!

Answer 1

看一下这两行:

List<Integer> primeList = new ArrayList<Integer>();
...
Iterator<Integer> primeIterator = primeList.iterator();

您创建一个空列表，然后测试该列表的内容。由于该列表中没有任何内容，因此您的代码永远不会进入 while 循环，因此永远不会调用测试断言。

如果你更换这个...

Iterator<Integer> primeIterator = primeList.iterator();

...有了这个...

Iterator<Integer> primeIterator = prime.iterator();

...然后您将测试由 PrimeNumbers 类创建的迭代器。这一定是您想要做的，但应该注意的是，此调用将始终返回 false:primeIterator.equals(primeArray[i]) (因为它是比较迭代器，而不是内部的值)迭代器，针对“预期”数组)，因此您的测试将在 while 循环的第一次迭代中失败。这将是一个假阴性。

看起来您正试图证明 PrimeNumbers (已被编码为不返回 19)在被要求返回前 8 个时不返回 19质数。如果是这样，那么以下可能是一种更紧凑且更具体的方法:

@Test
public void test() {
    //instantiate prime object, create primes list, and iterator
    PrimeNumbers prime = new PrimeNumbers();

    int n = 8;
    int unexpectedPrimeNumber = 19;

    // call the method
    prime.computePrimes(n);

    // gather the generated prime numbers into a list for easier assertion
    List<Integer> primeList = new ArrayList<>();
    prime.iterator().forEachRemaining(integer -> primeList.add(integer));

    assertEquals("Should contain " + n + " prime numbers!", n, primeList.size());
    assertFalse("Should not contain the unexpected prime number: " + unexpectedPrimeNumber, primeList.contains(unexpectedPrimeNumber));
}