Remove 'fold' from barn-agent loop.

The fold makes the code safer but I think it is at the cost of
the code being readable for someone new to the project. I've
removed it to hopefully make the steps easier to see.

This change also ensures that barn-agent doesn't failover when it's got a
'flacky' connection i.e. where it's managing to ship some files out of the
entire list. In production barn-agent can often end up running like this.

@omidaladini @johnglover 
This is the bit that was moved to prevent failovers in the case that some files could be shipped ('flakey') yes? It might be nice to mention that here as it's quite subtle.