backup targets: add test for all routes

src/routes/test/common.js

@@ -118,14 +118,15 @@ async function setupServer() {
     await database._clear();
     await appstore._setApiServerOrigin(exports.mockApiServerOrigin);
     // duplicated here since we clear the database
-    await backupTargets.add({
+    const id = await backupTargets.add({
         provider: 'filesystem',
-        label: '',
+        label: 'Default',
         config: { backupFolder: '/tmp/boxtest' },
         format: 'tgz',
         retention: { keepWithinSecs: 2 * 24 * 60 * 60 },
         schedule:  '00 00 23 * * *'
     });
+    await backupTargets.setPrimary({ id });
     await oidcServer.stop();
     await server.start();
     debug('Set up server complete');
@@ -161,7 +162,7 @@ async function setup() {
     // create an admin
     response = await superagent.post(`${serverUrl}/api/v1/users`)
         .query({ access_token: owner.token })
-        .send({ username: admin.username, email: admin.email, password: admin.password });
+        .send({ username: admin.username, email: admin.email, password: admin.password, role: 'admin' });
     expect(response.status).to.equal(201);
     admin.id = response.body.id;
     // HACK to get a token for second user (passwords are generated and the user should have gotten a password setup link...)