Question

我正在尝试为我重构的方法编写一个Junit测试，以确保该方法仍然按预期运行。我注意到一些我无法弄清楚的奇怪行为。

为什么Java ByteArrayOutputStream.toByteArray（）返回一些 Random Bytes？我认为这可能与内存有关。数组中的11个字节始终是 random 。有什么见解吗？

以下是我重构的方法。

package foo;

import java.io.ByteArrayOutputStream;
import java.io.IOException;
import java.util.Map;
import java.util.zip.ZipEntry;
import java.util.zip.ZipOutputStream;

public class ZipUtil {

    public static byte[] createZip(Map<String, byte[]> files) throws IOException {
        final ByteArrayOutputStream bos = new ByteArrayOutputStream();
        final ZipOutputStream zipfile = new ZipOutputStream(bos);
        String fileName = null;
        ZipEntry zipentry = null;
        for (Map.Entry<String, byte[]> entry : files.entrySet()) {
            fileName = entry.getKey();
            zipentry = new ZipEntry(fileName);
            zipfile.putNextEntry(zipentry);
            zipfile.write(files.get(fileName));
        }

        zipfile.close();  
        return bos.toByteArray();  
    }  
}

测试类

package foo;

import java.util.HashMap;
import java.util.Map;

import org.junit.Assert;
import org.junit.Test;
import org.junit.runner.RunWith;
import org.powermock.core.classloader.annotations.PrepareForTest;
import org.powermock.modules.junit4.PowerMockRunner;

@RunWith(PowerMockRunner.class)
@PrepareForTest({})
public class ZipUtilTest {

    @Test
    public void test_createZip() throws Exception {

        // Setup
        Map<String, byte[]> files = new HashMap<String, byte[]>();

        byte[] byteArr = new byte[] { 49, 17, 23,
                -29, -126, 111, -72, -112, 48, 32, 91, -28, -14, 112 };

        files.put("foo.txt", byteArr);

        // Test
        byte[] result = ZipUtil.createZip(files);

        // Validations
        byte[] expectedByteArray1 = new byte[] { 80, 75, 3, 4, 20, 0, 8, 8, 8, 0, 46, -120, -11, 72, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 7,
                0, 0, 0, 102, 111, 111, 46, 116, 120, 116, 51, 20, 20, 127, -36, -108, -65, 99, -126, -127, 66, -12, -109, 79, 5, 0, 80,
                75, 7, 8, -99, 100, -122, -62, 16, 0, 0, 0, 14, 0, 0, 0, 80, 75, 1, 2, 20, 0, 20, 0, 8, 8, 8, 0, 46, -120, -11, 72, -99,
                100, -122, -62, 16, 0, 0, 0, 14, 0, 0, 0, 7, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 102, 111, 111, 46, 116,
                120, 116, 80, 75, 5, 6, 0, 0, 0, 0, 1, 0, 1, 0, 53, 0, 0, 0, 69, 0, 0, 0, 0, 0 };

        Assert.assertNotNull(result);
        Assert.assertEquals(144, result.length);

        // a few bytes are "random". So test the first 9 never random bytes
        for (int i = 0; i < 10; i++) {
            Assert.assertEquals(expectedByteArray1[i], result[i]);
        }

        // This fails
        // Assert.assertEquals(expectedByteArray1[10], result[10]);

        for (int i = 11; i < 70; i++) {
            Assert.assertEquals(expectedByteArray1[i], result[i]);
        }
    }
}

Answer 1

您的测试用例假定使用相同输入创建的连续zip文件将导致完全相同的输出字节。看起来并非如此 - zip文件规范存储从字节10开始的文件修改日期和时间（小端）。这就是从该位置开始的字节不同的原因。

https://en.wikipedia.org/wiki/Zip_(file_format)

那就是说，我不认为将zip文件中的字节与已知的zip文件进行比较是一个非常有效的单元测试。一个更有效的测试是往返旅行＆＃34; zip文件 - 从新创建的存档中提取压缩文件，并与已知输入文件进行比较，以确保它们匹配。

ByteArrayOutputStream.toByteArray（）返回随机字节？

1 个答案: