Code review comment for lp:~music-app-dev/music-app/use-mediascanner2.0

Revision history for this message
Victor Thompson (vthompson) wrote :

Personally, I assumed it had to do with dbus changes in mediascanner2. Isn't the app in the store/image mocking? That appears to work. I hate making concessions, but we might want to go back to copying off the Music dir, but in addition to that maybe we could run each test as follows:

1) stop mediascanner2*
2) copy/save user's Music dir
3) move test songs to ~/Music
4) start mediascanner2
5) sleep for a predefined period
6) stop mediascanner2
7) run test
8) restore user's Music dir

*Note that subsequent tests will already have mediascanner2 stopped. This will prevent the service from needlessly rescanning the users music--which could take some time. This would mean that the user should either start mediascanner2 to rebuild their music library to use the app or reboot. Everything should then work.

« Back to merge proposal