JIRA ticket: LIMS-2129

Summary:

All members of DLS staff have been added to the group b01-1_admin and thus Synchweb treats them as staff, which means we were never testing whether they had all_dewars permission, as #752 was not well written.

Changes:

• Swap the order of if clauses to test the all_dewars permission first

To test:

• Log in as someone from goods handling, test you can navigate via a proposal to a shipment and print shipping labels
• Test a direct link to a shipment as someone from goods handling
• Test you cannot see visits or data collections
• Log in as a 'normal' user, test you can only get to shipments from your proposal
• Test a direct link to a shipment you should not be able to see
• Log in as staff, test you can see everything as normal