Allow campaigns to be fetched with their related domain

Author: bendrucker

src/campaign/campaign.js

@@ -28,6 +28,10 @@ module.exports = function (bookshelf) {
 
     organization: function () {
       return this.belongsTo('Organization');
+    },
+
+    domain: function () {
+      return this.belongsTo('Domain');
     }
 
   }));

src/campaign/routes.js

@@ -21,14 +21,20 @@ module.exports = function (server) {
       handler: function (request, reply) {
         Campaign
           .where({id: request.params.id})
-          .fetch({require: true})
+          .fetch({
+            require: true,
+            withRelated: request.query.expand
+          })
           .then(reply)
           .catch(reply);
       },
       config: {
         validate: {
           params: {
             id: Joi.string().guid()
+          },
+          query: {
+            expand: Joi.array().includes(Joi.string())
           }
         },
         cache: {

test/integration/specs/campaigns.js

@@ -34,6 +34,16 @@ module.exports = function (server) {
           });
       });
 
+      it('can get the campaign with the domain', function () {
+        return server.injectThen('/campaigns/' + campaign.id + '?expand[]=domain')
+          .then(function (response) {
+            expect(response.statusCode).to.equal(200);
+            expect(JSON.parse(response.payload))
+              .to.have.deep.property('domain.id')
+              .with.length(36);
+          });
+      });
+
       it('responds with a 400 for non-uuid', function () {
         return server.injectThen('/campaigns/1')
           .then(function (response) {