Question

我正在编写一些Spring集成测试来测试我的应用程序中的Spring Security。我使用RequestPostProcessor创建具有不同权限的测试用户。我也将它们缓存在所有测试中重用。请参阅以下代码：

public final class Users {
    public static final RequestPostProcessor
        ANONYMOUS = anonymous();

    public static final RequestPostProcessor
        PERMISSIONS_READ = buildUser(Permissions.PERMISSIONS_READ);
    public static final RequestPostProcessor
        PERMISSIONS_WRITE = buildUser(Permissions.PERMISSIONS_WRITE);
    public static final RequestPostProcessor
        PERMISSIONS_DELETE = buildUser(Permissions.PERMISSIONS_DELETE);

    public static final RequestPostProcessor
        ROLES_READ = buildUser(Permissions.ROLES_READ);
    public static final RequestPostProcessor
        ROLES_WRITE = buildUser(Permissions.ROLES_WRITE);
    public static final RequestPostProcessor
        ROLES_DELETE = buildUser(Permissions.ROLES_DELETE);

    public static final RequestPostProcessor
        USERS_READ = buildUser(Permissions.USERS_READ);
    public static final RequestPostProcessor
        USERS_WRITE = buildUser(Permissions.USERS_WRITE);
    public static final RequestPostProcessor
        USERS_DELETE = buildUser(Permissions.USERS_DELETE);


    private Users() {}

    private static RequestPostProcessor buildUser(Permissions permission) {
        return buildUser(permission.toString(), permission.toString());
    }

    private static RequestPostProcessor buildUser(String name, String... authorities) {
        return user(name).authorities(SecurityUtils.authoritiesFromStrings(authorities));
    }
}

当我在测试中使用它们时，我得到了ConcurrentModificationException。

用法：

....................
@Autowired private WebApplicationContext context;
@Autowired private Filter springSecurityFilterChain;
MockMvc mvc = MockMvcBuilders
        .webAppContextSetup(context)
        .addFilters(springSecurityFilterChain)
        .build();
....................
MockHttpServletRequestBuilder req = get("some-url");
mvc.perform(req.with(Users.ANONYMOUS))
        .andExpect(status().isFound())
        .andExpect(header().string("Location", "login-url"));
Stream.of(Users.PERMISSIONS_WRITE, Users.PERMISSIONS_DELETE,
        Users.ROLES_WRITE, Users.ROLES_DELETE,
        Users.USERS_WRITE, Users.USERS_DELETE)
        .parallel()
        .forEach(Unchecked.consumer(user -> mvc.perform(req.with(user)) //Exception is here and caused by .with(user)
                .andExpect(status().isForbidden())));
....................

例外：

....................
Caused by: java.util.ConcurrentModificationException
at java.util.ArrayList$Itr.checkForComodification(ArrayList.java:901)
at java.util.ArrayList$Itr.next(ArrayList.java:851)
at org.springframework.test.web.servlet.request.MockHttpServletRequestBuilder.postProcessRequest(MockHttpServletRequestBuilder.java:754)
at org.springframework.test.web.servlet.MockMvc.perform(MockMvc.java:145)
at com.ipan.fin.man.integrational.PermissionsTest.lambda$checkReadSecurity$5(PermissionsTest.java:162)//line with my code
....................

org.springframework.test.web.servlet.request.MockHttpServletRequestBuilder引发了异常，因为我正在缓存它（MockHttpServletRequestBuilder req）并在多个流中使用。因此，如果一个流迭代postProcessors

@Override
public MockHttpServletRequest postProcessRequest(MockHttpServletRequest request) {
    for (RequestPostProcessor postProcessor : this.postProcessors) { //here
        request = postProcessor.postProcessRequest(request);
        if (request == null) {
            throw new IllegalStateException(
                    "Post-processor [" + postProcessor.getClass().getName() + "] returned null");
        }
    }
    return request;
}

第二个将新postProcessor添加到postProcessors

@Override
public MockHttpServletRequestBuilder with(RequestPostProcessor postProcessor) {
    Assert.notNull(postProcessor, "postProcessor is required");
    this.postProcessors.add(postProcessor); //here
    return this;
}

ConcurrentModificationException将被抛出。

据我所知，在上面的代码中，由于两个原因，不允许缓存MockHttpServletRequestBuilder：

它不是线程安全的
MockHttpServletRequestBuilder.with来电添加新的RequestPostProcessor，而不是像我预期的那样替换旧的。{/ li>
我是对的吗？

P.S。使用序列流，测试工作正常，看起来MockHttpServletRequestBuilder.with的调用替换旧的RequestPostProcessor对象，因为我总是得到正确的测试结果（来自服务器的响应状态）

Answer 1

是的，您的分析是正确的：floatingPointValue并非设计为可以同时使用。

无论如何，创建MockHttpServletRequestBuilder实际上并没有明显的开销。

所以，我建议你每次只需要一个新的请求就可以创建一个新的请求，并且使用并行流并不会对这些测试产生太大的影响（如果有的话）。

总之，尽量不要过度设计您的测试。 ; - ）

此致

Sam（ Spring TestContext Framework的作者）

如果缓存MockHttpServletRequestBuilder，MockMvc.perform会抛出并行流的ConcurrentModificationException

1 个答案: