diff --git a/rm-server/test/java/org/alfresco/module/org_alfresco_module_rm/test/issue/RM1008Test.java b/rm-server/test/java/org/alfresco/module/org_alfresco_module_rm/test/issue/RM1008Test.java
index ed09fba0ac..d1086fc6c9 100644
--- a/rm-server/test/java/org/alfresco/module/org_alfresco_module_rm/test/issue/RM1008Test.java
+++ b/rm-server/test/java/org/alfresco/module/org_alfresco_module_rm/test/issue/RM1008Test.java
@@ -40,44 +40,44 @@ import org.alfresco.util.GUID;
 
 /**
  * System test for RM-1008
- * 
+ *
  * @author Roy Wetherall
  * @since 2.1
  */
-public class RM1008Test extends BaseRMTestCase 
-{    
+public class RM1008Test extends BaseRMTestCase
+{
     private String myUser;
-    
+
     @Override
     protected void initServices()
     {
         super.initServices();
     }
-    
+
     @Override
     protected boolean isRecordTest()
     {
         return true;
     }
-    
+
     @Override
     protected boolean isUserTest()
     {
         return true;
     }
-    
+
     @Override
     protected void setupTestUsersImpl(NodeRef filePlan)
     {
         super.setupTestUsersImpl(filePlan);
-        
+
         myUser = GUID.generate();
         createPerson(myUser);
         filePlanRoleService.assignRoleToAuthority(filePlan, FilePlanRoleService.ROLE_USER, myUser);
     }
-   
+
     public void testContainers() throws Exception
-    {        
+    {
         doTestInTransaction(new Test<Void>()
         {
             @Override
@@ -87,20 +87,20 @@ public class RM1008Test extends BaseRMTestCase
                 assertNotNull(holdContainer);
                 NodeRef transferContainer = filePlanService.getTransferContainer(filePlan);
                 assertNotNull(transferContainer);
-                
+
                 Capability viewRecords = capabilityService.getCapability("ViewRecords");
                 assertNotNull(viewRecords);
-                
+
                 assertEquals(AccessStatus.ALLOWED, viewRecords.hasPermission(holdContainer));
                 assertEquals(AccessStatus.ALLOWED, viewRecords.hasPermission(transferContainer));
-                
+
                 assertEquals(AccessStatus.ALLOWED, permissionService.hasPermission(holdContainer, RMPermissionModel.FILING));
                 assertEquals(AccessStatus.ALLOWED, permissionService.hasPermission(transferContainer, RMPermissionModel.FILING));
-                
+
                 return null;
             }
         }, rmAdminName);
-        
+
         doTestInTransaction(new Test<Void>()
         {
             @Override
@@ -110,23 +110,23 @@ public class RM1008Test extends BaseRMTestCase
                 assertNotNull(holdContainer);
                 NodeRef transferContainer = filePlanService.getTransferContainer(filePlan);
                 assertNotNull(transferContainer);
-                
+
                 Capability viewRecords = capabilityService.getCapability("ViewRecords");
                 assertNotNull(viewRecords);
-                
+
                 assertEquals(AccessStatus.ALLOWED, viewRecords.hasPermission(holdContainer));
                 assertEquals(AccessStatus.ALLOWED, viewRecords.hasPermission(transferContainer));
-                
+
                 assertEquals(AccessStatus.DENIED, permissionService.hasPermission(holdContainer, RMPermissionModel.FILING));
                 assertEquals(AccessStatus.DENIED, permissionService.hasPermission(transferContainer, RMPermissionModel.FILING));
-                
+
                 return null;
             }
         }, myUser);
     }
-    
+
     public void testHold()
-    {        
+    {
         final NodeRef hold = doTestInTransaction(new Test<NodeRef>()
         {
             @Override
@@ -138,7 +138,7 @@ public class RM1008Test extends BaseRMTestCase
                 return holds.iterator().next();
             }
         }, rmAdminName);
-        
+
         doTestInTransaction(new Test<Void>()
         {
             @Override
@@ -146,82 +146,83 @@ public class RM1008Test extends BaseRMTestCase
             {
                 Capability viewRecords = capabilityService.getCapability("ViewRecords");
                 assertNotNull(viewRecords);
-                
+
                 assertEquals(AccessStatus.ALLOWED, viewRecords.hasPermission(hold));
                 assertEquals(AccessStatus.ALLOWED, permissionService.hasPermission(hold, RMPermissionModel.FILING));
-                
+
                 return null;
             }
         }, rmAdminName);
-        
+
         doTestInTransaction(new Test<Void>()
         {
             @Override
             public Void run()
-            {     
+            {
                 Capability viewRecords = capabilityService.getCapability("ViewRecords");
                 assertNotNull(viewRecords);
-                
+
                 assertEquals(AccessStatus.ALLOWED, viewRecords.hasPermission(hold));
                 assertEquals(AccessStatus.DENIED, permissionService.hasPermission(hold, RMPermissionModel.FILING));
-                
+
                 return null;
             }
         }, myUser);
-        
+
         doTestInTransaction(new Test<Void>()
         {
             @Override
             public Void run()
             {
                 filePlanPermissionService.setPermission(filePlan, myUser, FILING);
-                
+
                 return null;
             }
         }, rmAdminName);
-        
+
         doTestInTransaction(new Test<Void>()
         {
             @Override
             public Void run()
-            {     
+            {
                 Capability viewRecords = capabilityService.getCapability("ViewRecords");
                 assertNotNull(viewRecords);
-                
+
                 assertEquals(AccessStatus.ALLOWED, viewRecords.hasPermission(hold));
-                assertEquals(AccessStatus.ALLOWED, permissionService.hasPermission(hold, RMPermissionModel.FILING));
-                
+                assertEquals(AccessStatus.ALLOWED, permissionService.hasPermission(hold, RMPermissionModel.READ_RECORDS));
+                assertEquals(AccessStatus.DENIED, permissionService.hasPermission(hold, RMPermissionModel.FILING));
+
                 return null;
             }
         }, myUser);
-        
+
         doTestInTransaction(new Test<Void>()
         {
             @Override
             public Void run()
             {
                 filePlanPermissionService.deletePermission(filePlan, myUser, FILING);
-                
+
                 return null;
             }
         }, rmAdminName);
-        
+
         doTestInTransaction(new Test<Void>()
         {
             @Override
             public Void run()
-            {     
+            {
                 Capability viewRecords = capabilityService.getCapability("ViewRecords");
                 assertNotNull(viewRecords);
-                
+
                 assertEquals(AccessStatus.ALLOWED, viewRecords.hasPermission(hold));
                 assertEquals(AccessStatus.DENIED, permissionService.hasPermission(hold, RMPermissionModel.FILING));
-                
+
                 return null;
             }
         }, myUser);
     }
-    
+
     public void testTransfer()
     {
         final NodeRef transferFolder = doTestInTransaction(new Test<NodeRef>()
@@ -235,39 +236,39 @@ public class RM1008Test extends BaseRMTestCase
                 dsProps.put(PROP_DISPOSITION_AUTHORITY, "test");
                 dsProps.put(PROP_DISPOSITION_INSTRUCTIONS, "test");
                 dsProps.put(PROP_RECORD_LEVEL_DISPOSITION, false);
-                DispositionSchedule dispositionSchedule = dispositionService.createDispositionSchedule(transferCat, dsProps);                
-                
+                DispositionSchedule dispositionSchedule = dispositionService.createDispositionSchedule(transferCat, dsProps);
+
                 Map<QName, Serializable> adParams = new HashMap<QName, Serializable>(3);
                 adParams.put(PROP_DISPOSITION_ACTION_NAME, "cutoff");
                 adParams.put(PROP_DISPOSITION_DESCRIPTION, "test");
-                adParams.put(PROP_DISPOSITION_PERIOD, "immediately|0"); 
-                
+                adParams.put(PROP_DISPOSITION_PERIOD, "immediately|0");
+
                 dispositionService.addDispositionActionDefinition(dispositionSchedule, adParams);
-                
+
                 adParams = new HashMap<QName, Serializable>(3);
                 adParams.put(PROP_DISPOSITION_ACTION_NAME, "transfer");
                 adParams.put(PROP_DISPOSITION_DESCRIPTION, "test");
-                adParams.put(PROP_DISPOSITION_PERIOD, "immediately|0");            
-                
+                adParams.put(PROP_DISPOSITION_PERIOD, "immediately|0");
+
                 dispositionService.addDispositionActionDefinition(dispositionSchedule, adParams);
 
                 return rmService.createRecordFolder(transferCat, "transferFolder");
             }
         });
-        
+
         final NodeRef transfer = doTestInTransaction(new Test<NodeRef>()
         {
             @Override
             public NodeRef run()
-            {                
+            {
                 actionService.executeRecordsManagementAction(transferFolder, "cutoff");
                 actionService.executeRecordsManagementAction(transferFolder, "transfer");
-                
+
                 NodeRef transferContainer = filePlanService.getTransferContainer(filePlan);
                 List<ChildAssociationRef> childAssocs = nodeService.getChildAssocs(transferContainer, ContentModel.ASSOC_CONTAINS, RegexQNamePattern.MATCH_ALL);
-                return childAssocs.get(0).getChildRef();                
+                return childAssocs.get(0).getChildRef();
             }
-            
+
             @Override
             public void test(NodeRef result) throws Exception
             {
@@ -275,7 +276,7 @@ public class RM1008Test extends BaseRMTestCase
                 assertEquals(TYPE_TRANSFER, nodeService.getType(result));
             }
         });
-        
+
         doTestInTransaction(new Test<Void>()
         {
             @Override
@@ -283,82 +284,83 @@ public class RM1008Test extends BaseRMTestCase
             {
                 Capability viewRecords = capabilityService.getCapability("ViewRecords");
                 assertNotNull(viewRecords);
-                
+
                 assertEquals(AccessStatus.ALLOWED, viewRecords.hasPermission(transfer));
                 assertEquals(AccessStatus.ALLOWED, permissionService.hasPermission(transfer, RMPermissionModel.FILING));
-                
+
                 return null;
             }
         }, rmAdminName);
-        
+
         doTestInTransaction(new Test<Void>()
         {
             @Override
             public Void run()
-            {     
+            {
                 Capability viewRecords = capabilityService.getCapability("ViewRecords");
                 assertNotNull(viewRecords);
-                
+
                 assertEquals(AccessStatus.ALLOWED, viewRecords.hasPermission(transfer));
                 assertEquals(AccessStatus.DENIED, permissionService.hasPermission(transfer, RMPermissionModel.FILING));
-                
+
                 return null;
             }
         }, myUser);
-        
+
         doTestInTransaction(new Test<Void>()
         {
             @Override
             public Void run()
             {
                 filePlanPermissionService.setPermission(filePlan, myUser, FILING);
-                
+
                 return null;
             }
         }, rmAdminName);
-        
+
         doTestInTransaction(new Test<Void>()
         {
             @Override
             public Void run()
-            {     
+            {
                 Capability viewRecords = capabilityService.getCapability("ViewRecords");
                 assertNotNull(viewRecords);
-                
+
                 assertEquals(AccessStatus.ALLOWED, viewRecords.hasPermission(transfer));
-                assertEquals(AccessStatus.ALLOWED, permissionService.hasPermission(transfer, RMPermissionModel.FILING));
-                
+                assertEquals(AccessStatus.ALLOWED, permissionService.hasPermission(transfer, RMPermissionModel.READ_RECORDS));
+                assertEquals(AccessStatus.DENIED, permissionService.hasPermission(transfer, RMPermissionModel.FILING));
+
                 return null;
             }
         }, myUser);
-        
+
         doTestInTransaction(new Test<Void>()
         {
             @Override
             public Void run()
             {
                 filePlanPermissionService.deletePermission(filePlan, myUser, FILING);
-                
+
                 return null;
             }
         }, rmAdminName);
-        
+
         doTestInTransaction(new Test<Void>()
         {
             @Override
             public Void run()
-            {     
+            {
                 Capability viewRecords = capabilityService.getCapability("ViewRecords");
                 assertNotNull(viewRecords);
-                
+
                 assertEquals(AccessStatus.ALLOWED, viewRecords.hasPermission(transfer));
                 assertEquals(AccessStatus.DENIED, permissionService.hasPermission(transfer, RMPermissionModel.FILING));
-                
+
                 return null;
             }
         }, myUser);
 
-         
+
     }
 
 }