Simplify data handling

This does a couple things:

- Drop technical enforcement of acknowledging ML100K data terms
- Use temporary folder rules instead of temporary files in tests (#357)