测试弹簧MVC控制器

Question

I am testing my controller using mockito junit framework.

I got stuck in the controller layer where I get null pointer exception in UserDetails interface.

I tried to override the interface but still I face the problem.

This is my default controller only for userDetails interface to get the user information.

public class DefaultController {


    public UserDetails getUserDetails(){
        UserDetails userDetails=null;
        Authentication auth = SecurityContextHolder.getContext().getAuthentication();
        if (!(auth instanceof AnonymousAuthenticationToken)) {
               userDetails = (UserDetails) auth.getPrincipal();
        }
        return userDetails;
    }

    public Set<Integer> getUserCSRoles(){
        Set<Integer> csroles=new HashSet<Integer>();
        UserDetails userDetails=getUserDetails();
        if(userDetails!=null)
        for (GrantedAuthority grantedAuthority : userDetails.getAuthorities()) {
            if(grantedAuthority.getAuthority().split("_").length>1){
            if(grantedAuthority.getAuthority().split("_")[1].toString().equalsIgnoreCase("MANAGER")){
                csroles.add(1);
            }else if(grantedAuthority.getAuthority().split("_")[1].toString().equalsIgnoreCase("SHAREDSERVICE")){
                csroles.add(2);
            }else if(grantedAuthority.getAuthority().split("_")[1].toString().equalsIgnoreCase("ADMIN")){
                csroles.add(3);
            }else if(grantedAuthority.getAuthority().split("_")[1].toString().equalsIgnoreCase("SUPERADMIN")){
                csroles.add(3);
            }
            }
        }
        return csroles;
    }

This is my managaer controller :

@RequestMapping(value="new",method=RequestMethod.POST)
    public String testMethod(@ModelAttribute("employeeDetails") EmployeeDetails details,HttpSession httpSession, Model model){
        UserDetails userDetails = getUserDetails();
        model.addAttribute("details", details);
        return "redirect:newemployee";
    }

And the for testing the manager this is what I have done:

@Test
    public void testnewEmployeePost() throws Exception{
        UserDetails userDetails = new UserDetails() {

            @Override
            public boolean isEnabled() {
                // TODO Auto-generated method stub
                return true;
            }

            @Override
            public boolean isCredentialsNonExpired() {
                // TODO Auto-generated method stub
                return true;
            }

            @Override
            public boolean isAccountNonLocked() {
                // TODO Auto-generated method stub
                return true;
            }

            @Override
            public boolean isAccountNonExpired() {
                // TODO Auto-generated method stub
                return true;
            }

            @Override
            public String getUsername() {
                // TODO Auto-generated method stub
                return "nitesh";
            }

            @Override
            public String getPassword() {
                // TODO Auto-generated method stub
                return "nitesh";
            }

            @Override
            public Collection<? extends GrantedAuthority> getAuthorities() {
                GrantedAuthority authority = new GrantedAuthority() {

                    @Override
                    public String getAuthority() {
                        // TODO Auto-generated method stub
                        return "ROLE_USER";
                    }
                };
                List<GrantedAuthority> grantedAuthorities = new ArrayList<GrantedAuthority>();
                grantedAuthorities.add(authority);
                return grantedAuthorities;
            }
        };

        EmployeeDetails employeeDetails = new EmployeeDetails();
        employeeDetails.setPanCardNo("lower");
        employeeDetails.setFirstName("Nitesh");

        Mockito.when(managerController.getUserDetails()).thenReturn(userDetails);

        mockMvc.perform(post("/manager/new")
                .contentType(MediaType.ALL)
                .sessionAttr("employeeDetails", employeeDetails)
                )   
                .andExpect(model().attribute("employeeDetails", instanceOf(EmployeeDetails.class)));
    }

And the exception thrown is :

java.lang.NullPointerException
    at com.rectrix.ool.controller.DefaultController.getUserDetails(DefaultController.java:24)
    at com.rectrix.ool.test.controller.TestManagerController.testnewEmployeePost(TestManagerController.java:172)
    at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
    at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
    at java.lang.reflect.Method.invoke(Method.java:606)
    at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
    at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
    at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47)
    at org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
    at org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:26)
    at org.springframework.test.context.junit4.statements.RunBeforeTestMethodCallbacks.evaluate(RunBeforeTestMethodCallbacks.java:74)
    at org.springframework.test.context.junit4.statements.RunAfterTestMethodCallbacks.evaluate(RunAfterTestMethodCallbacks.java:83)
    at org.springframework.test.context.junit4.statements.SpringRepeat.evaluate(SpringRepeat.java:72)
    at org.springframework.test.context.junit4.SpringJUnit4ClassRunner.runChild(SpringJUnit4ClassRunner.java:231)
    at org.springframework.test.context.junit4.SpringJUnit4ClassRunner.runChild(SpringJUnit4ClassRunner.java:88)
    at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
    at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
    at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
    at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
    at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
    at org.springframework.test.context.junit4.statements.RunBeforeTestClassCallbacks.evaluate(RunBeforeTestClassCallbacks.java:61)
    at org.springframework.test.context.junit4.statements.RunAfterTestClassCallbacks.evaluate(RunAfterTestClassCallbacks.java:71)
    at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
    at org.springframework.test.context.junit4.SpringJUnit4ClassRunner.run(SpringJUnit4ClassRunner.java:174)
    at org.eclipse.jdt.internal.junit4.runner.JUnit4TestReference.run(JUnit4TestReference.java:50)
    at org.eclipse.jdt.internal.junit.runner.TestExecution.run(TestExecution.java:38)
    at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.runTests(RemoteTestRunner.java:459)
    at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.runTests(RemoteTestRunner.java:675)
    at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.run(RemoteTestRunner.java:382)
    at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.main(RemoteTestRunner.java:192)

Kindly guide me to overcome this issue

Answer 1

You don't need to create an annonymous UserDetails instance. You can use User which is an implementation of the UserDetails already available in Spring Security. Anyway it is not even required. And you are trying to mock the Parent Class DefaultController of the ManagerController . I have never seen or heard the ability to mock a is a relationship. What you should do is import dependency

<dependency>
    <groupId>org.springframework.security</groupId>
    <artifactId>spring-security-test</artifactId>
    <version>4.0.4.RELEASE</version>
</dependency>

And rewrite your test as following to mock an authenticated user.

 import static org.springframework.security.test.web.servlet.request.SecurityMockMvcRequestPostProcessors.user;

    @Test
    public void testnewEmployeePost() throws Exception{


        EmployeeDetails employeeDetails = new EmployeeDetails();
        employeeDetails.setPanCardNo("lower");
        employeeDetails.setFirstName("Nitesh");

        mockMvc.perform(post("/manager/new")
            .with(user("nitesh")
            .password("nitesh")
            .authorities(AuthorityUtils.createAuthorityList("ROLE_USER"))
        .contentType(MediaType.ALL)
        .sessionAttr("employeeDetails", employeeDetails))   
        .andExpect(model().attribute("employeeDetails", instanceOf(EmployeeDetails.class)));
    }