From 3dedfc7d61f36fb0a179d5fcb2ec9a520c7a7cca Mon Sep 17 00:00:00 2001 From: kirrg001 Date: Wed, 21 Feb 2018 23:44:58 +0100 Subject: [PATCH] Fixed default mobiledoc handling in our data generator no issue - the handling here was not correct - if you've passed no mobiledoc, it wasn't adding mobiledoc and an undefined html value - we need a default mobiledoc+html value in case you don't pass the values within the test cases --- core/test/utils/fixtures/data-generator.js | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/core/test/utils/fixtures/data-generator.js b/core/test/utils/fixtures/data-generator.js index 6ad306132b..653420557d 100644 --- a/core/test/utils/fixtures/data-generator.js +++ b/core/test/utils/fixtures/data-generator.js @@ -410,14 +410,22 @@ DataGenerator.forKnex = (function () { overrides = overrides || {}; var newObj = _.cloneDeep(overrides), - mobiledoc = JSON.parse(overrides.mobiledoc || '{}'); + mobiledocObj; + + if (!newObj.mobiledoc) { + newObj.mobiledoc = DataGenerator.markdownToMobiledoc('## markdown'); + } + + if (!newObj.html) { + mobiledocObj = JSON.parse(newObj.mobiledoc); + newObj.html = mobiledocObj.cards && mobiledocObj.cards[0][1].markdown; + } return _.defaults(newObj, { id: ObjectId.generate(), uuid: uuid.v4(), title: 'title', status: 'published', - html: mobiledoc.cards && mobiledoc.cards[0][1].markdown, featured: true, page: false, author_id: DataGenerator.Content.users[0].id,