looks good just a couple things: 
1) could you add this test to the end? this test depends on things happening in a certain order
2) could you have user1 submit the form and check to make sure the submission goes through and the case is updated? this will test to make sure that the access via parent-child relationship works all the way through the process