:fire: more linting

backend/db/migrations/20210524105357_create_users_table.js

@@ -1,15 +1,13 @@
-exports.up = function(knex) {
-    return knex.schema
-        .createTable('users', function(table) {
-            table.increments('user_id').primary()
-            table.string('user_name', 32).notNullable()
-            table.string('user_email', 90).notNullable()
-            table.boolean('is_admin').notNullable()
-            table.boolean('is_poster').notNullable()
-        })
+exports.up = function (knex) {
+    return knex.schema.createTable('users', function (table) {
+        table.increments('user_id').primary()
+        table.string('user_name', 32).notNullable()
+        table.string('user_email', 90).notNullable()
+        table.boolean('is_admin').notNullable()
+        table.boolean('is_poster').notNullable()
+    })
 }
 
-exports.down = function(knex) {
-    return knex.schema
-        .dropTable('users')
-}
+exports.down = function (knex) {
+    return knex.schema.dropTable('users')
+}

backend/db/migrations/20210527173933_create_profiles_table.js

@@ -1,12 +1,10 @@
-exports.up = function(knex) {
-    return knex.schema
-        .createTable('profiles', function(table) {
-            table.increments('profile_id').primary()
-            table.integer('user_id').notNullable()
-        })
+exports.up = function (knex) {
+    return knex.schema.createTable('profiles', function (table) {
+        table.increments('profile_id').primary()
+        table.integer('user_id').notNullable()
+    })
 }
 
-exports.down = function(knex) {
-    return knex.schema
-        .dropTable('profiles')
+exports.down = function (knex) {
+    return knex.schema.dropTable('profiles')
 }

backend/db/migrations/20210527174416_create_response_keys_table.js

@@ -1,13 +1,11 @@
-exports.up = function(knex) {
-    return knex.schema
-        .createTable('response_keys', function(table) {
-            table.increments('response_key_id').primary()
-            table.string('response_key_name').notNullable()
-            table.string('response_key_description')
-        })
+exports.up = function (knex) {
+    return knex.schema.createTable('response_keys', function (table) {
+        table.increments('response_key_id').primary()
+        table.string('response_key_name').notNullable()
+        table.string('response_key_description')
+    })
 }
 
-exports.down = function(knex) {
-    return knex.schema
-        .dropTable('response_keys')
+exports.down = function (knex) {
+    return knex.schema.dropTable('response_keys')
 }

backend/db/migrations/20210527174422_create_responses_table.js

@@ -1,14 +1,12 @@
-exports.up = function(knex) {
-    return knex.schema
-        .createTable('responses', function(table) {
-            table.increments('response_id').primary()
-            table.integer('profile_id').notNullable()
-            table.integer('response_key_id').notNullable()
-            table.string('val').notNullable()
-        })
+exports.up = function (knex) {
+    return knex.schema.createTable('responses', function (table) {
+        table.increments('response_id').primary()
+        table.integer('profile_id').notNullable()
+        table.integer('response_key_id').notNullable()
+        table.string('val').notNullable()
+    })
 }
 
-exports.down = function(knex) {
-    return knex.schema
-        .dropTable('responses')
+exports.down = function (knex) {
+    return knex.schema.dropTable('responses')
 }

backend/db/migrations/20210527174434_create_groupings_table.js

@@ -1,13 +1,11 @@
-exports.up = function(knex) {
-    return knex.schema
-        .createTable('groupings', function(table) {
-            table.increments('grouping_id').primary()
-            table.string('grouping_name', 128).notNullable()
-            table.string('grouping_type', 128).notNullable() // Don't over normalize
-        })
+exports.up = function (knex) {
+    return knex.schema.createTable('groupings', function (table) {
+        table.increments('grouping_id').primary()
+        table.string('grouping_name', 128).notNullable()
+        table.string('grouping_type', 128).notNullable() // Don't over normalize
+    })
 }
 
-exports.down = function(knex) {
-    return knex.schema
-        .dropTable('groupings')
+exports.down = function (knex) {
+    return knex.schema.dropTable('groupings')
 }

backend/db/migrations/20210527174440_create_memberships_table.js

@@ -1,16 +1,14 @@
-exports.up = function(knex) {
-    return knex.schema
-        .createTable('memberships', function(table) {
-            table.increments('membership_id').primary()
-            table.integer('user_id').notNullable() // From
-            table.integer('grouping_id').notNullable() // To
-            table.string('membership_type', 128).notNullable() // Don't over normalize
-            table.boolean('can_edit').notNullable()
-            table.boolean('is_active').notNullable()
-        })
+exports.up = function (knex) {
+    return knex.schema.createTable('memberships', function (table) {
+        table.increments('membership_id').primary()
+        table.integer('user_id').notNullable() // From
+        table.integer('grouping_id').notNullable() // To
+        table.string('membership_type', 128).notNullable() // Don't over normalize
+        table.boolean('can_edit').notNullable()
+        table.boolean('is_active').notNullable()
+    })
 }
 
-exports.down = function(knex) {
-    return knex.schema
-        .dropTable('memberships')
+exports.down = function (knex) {
+    return knex.schema.dropTable('memberships')
 }

backend/db/migrations/20210527174448_create_messages_table.js

@@ -1,14 +1,12 @@
-exports.up = function(knex) {
-    return knex.schema
-        .createTable('messages', function(table) {
-            table.increments('message_id').primary()
-            table.integer('user_id').notNullable()
-            table.integer('grouping_id').notNullable()
-            table.string('message', 255).notNullable()
-        })
+exports.up = function (knex) {
+    return knex.schema.createTable('messages', function (table) {
+        table.increments('message_id').primary()
+        table.integer('user_id').notNullable()
+        table.integer('grouping_id').notNullable()
+        table.string('message', 255).notNullable()
+    })
 }
 
-exports.down = function(knex) {
-    return knex.schema
-        .dropTable('messages')
-}
+exports.down = function (knex) {
+    return knex.schema.dropTable('messages')
+}

backend/db/seeds/01-users.js

@@ -1,10 +1,11 @@
-const mock = require('../mock');
+const mock = require('../mock')
 
-exports.seed = function(knex) {
-  // Deletes ALL existing entries
-  return knex('users').truncate()
-    .then(function () {
-      // Inserts seed entries
-      return knex('users').insert(mock.users);
-    });
-};
+exports.seed = function (knex) {
+    // Deletes ALL existing entries
+    return knex('users')
+        .truncate()
+        .then(function () {
+            // Inserts seed entries
+            return knex('users').insert(mock.users)
+        })
+}

backend/db/seeds/02-profiles.js

@@ -1,10 +1,11 @@
-const mock = require('../mock');
+const mock = require('../mock')
 
-exports.seed = function(knex) {
-  // Deletes ALL existing entries
-  return knex('profiles').truncate()
-    .then(function () {
-      // Inserts seed entries
-      return knex('profiles').insert(mock.profiles);
-    });
-};
+exports.seed = function (knex) {
+    // Deletes ALL existing entries
+    return knex('profiles')
+        .truncate()
+        .then(function () {
+            // Inserts seed entries
+            return knex('profiles').insert(mock.profiles)
+        })
+}

backend/db/seeds/03-response_keys.js

@@ -1,10 +1,11 @@
-const mock = require('../mock');
+const mock = require('../mock')
 
-exports.seed = function(knex) {
-  // Deletes ALL existing entries
-  return knex('response_keys').truncate()
-    .then(function () {
-      // Inserts seed entries
-      return knex('response_keys').insert(mock.response_keys);
-    });
-};
+exports.seed = function (knex) {
+    // Deletes ALL existing entries
+    return knex('response_keys')
+        .truncate()
+        .then(function () {
+            // Inserts seed entries
+            return knex('response_keys').insert(mock.response_keys)
+        })
+}

backend/db/seeds/05-memberships.js

@@ -1,10 +1,11 @@
-const mock = require('../mock');
+const mock = require('../mock')
 
-exports.seed = function(knex) {
-  // Deletes ALL existing entries
-  return knex('memberships').truncate()
-    .then(function () {
-      // Inserts seed entries
-      return knex('memberships').insert(mock.memberships);
-    });
-};
+exports.seed = function (knex) {
+    // Deletes ALL existing entries
+    return knex('memberships')
+        .truncate()
+        .then(function () {
+            // Inserts seed entries
+            return knex('memberships').insert(mock.memberships)
+        })
+}

backend/db/seeds/06-groupings.js

@@ -1,10 +1,11 @@
-const mock = require('../mock');
+const mock = require('../mock')
 
-exports.seed = function(knex) {
-  // Deletes ALL existing entries
-  return knex('groupings').truncate()
-    .then(function () {
-      // Inserts seed entries
-      return knex('groupings').insert(mock.groupings);
-    });
-};
+exports.seed = function (knex) {
+    // Deletes ALL existing entries
+    return knex('groupings')
+        .truncate()
+        .then(function () {
+            // Inserts seed entries
+            return knex('groupings').insert(mock.groupings)
+        })
+}

backend/db/seeds/07-messages.js

@@ -1,10 +1,11 @@
-const mock = require('../mock');
+const mock = require('../mock')
 
-exports.seed = function(knex) {
-  // Deletes ALL existing entries
-  return knex('messages').truncate()
-    .then(function () {
-      // Inserts seed entries
-      return knex('messages').insert(mock.messages);
-    });
-};
+exports.seed = function (knex) {
+    // Deletes ALL existing entries
+    return knex('messages')
+        .truncate()
+        .then(function () {
+            // Inserts seed entries
+            return knex('messages').insert(mock.messages)
+        })
+}

backend/lib/plugins/profile.js

@@ -8,6 +8,7 @@ const ProfileService = require('../services/profile')
 
 const ProfileProfilesRoute = require('../routes/profile/profiles')
 const ProfileCreateRoute = require('../routes/profile/create')
+const ProfileUpdateRoute = require('../routes/profile/update')
 
 module.exports = {
     name: 'profile-plugin',
@@ -27,5 +28,6 @@ module.exports = {
 
         await server.route(ProfileProfilesRoute)
         await server.route(ProfileCreateRoute)
+        await server.route(ProfileUpdateRoute)
     },
 }

backend/lib/services/profile.js

@@ -46,7 +46,6 @@ module.exports = class ProfileService extends Schmervice.Service {
                 profile.response_keys.push(response.response_key_id)
                 profile.responses.push(response)
             })
-
             return profile
         })
     }
@@ -55,11 +54,9 @@ module.exports = class ProfileService extends Schmervice.Service {
      * Save responses in a profile
      * @param {number} userId
      * @param {Array} responses
-     * @returns
+     * @returns {object}
      */
     async saveResponsesCreateProfileFor(userId, responses, txn) {
-        const { Profile, Response } = this.server.models()
-
         const profile = await Profile.query(txn).insert({
             user_id: userId,
         })
@@ -74,6 +71,34 @@ module.exports = class ProfileService extends Schmervice.Service {
         //** Work around for HAPI returning profile_id as id */
         return { user_id: profile.user_id, profile_id: profile.id }
     }
+
+    /** Update responses in place
+     * @param {number} profileId
+     * @param {Array} responses
+     * @returns {Array} updated responses
+     */
+    async updateResponsesInProfile(profileId, responses, txn) {
+        const { Profile, Response } = this.server.models()
+        for (const responseToSave of responses) {
+            await Response.query(txn)
+                .update({
+                    response_id: responseToSave.response_id,
+                    profile_id: responseToSave.profile_id,
+                    response_key_id: responseToSave.response_key_id,
+                    val: responseToSave.val,
+                })
+                .where({
+                    profile_id: profileId,
+                })
+                .where({
+                    response_id: responseToSave.response_id,
+                })
+        }
+        return await Response.query(txn).where({
+            profile_id: profileId,
+        })
+    }
+
     /**
      * Delete a profile
      * @param {number} userId

backend/tests/example.test.js

@@ -8,12 +8,12 @@ const routeBase = '../lib/routes'
 const route = require(routeBase + '/example/base')
 
 test('path / should return ok on GET', async t => {
-  const server = Hapi.server()
+    const server = Hapi.server()
 
-  server.route(route)
+    server.route(route)
 
-  const { statusCode, payload } = await server.inject(route.path)
+    const { statusCode, payload } = await server.inject(route.path)
 
-  t.is(payload, '{"test":"hello, world"}')
-  t.is(statusCode, 200)
-})
+    t.is(payload, '{"test":"hello, world"}')
+    t.is(statusCode, 200)
+})