TryGhost/Ghost
1
0
Fork 0
mirror of https://github.com/TryGhost/Ghost.git synced 2024-11-29 15:12:58 +03:00
Code Issues Projects Releases Wiki Activity

Tests: Fix one more random failure (#9004)

Browse Source 
refs #7470

- this should fix https://github.com/TryGhost/Ghost/issues/7470#issuecomment-321016771
- the importer adds posts in parallel, but the tests read directly from the db without any order
- use findPage (findAll does not support order yet)
This commit is contained in:
Katharina Irrgang 2017-09-12 17:51:40 +02:00 committed by Hannah Wolfe
parent 4237446277
commit aef3d7f3f3
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
core/test/integration/data/importer/importers/data_spec.js
View File

@ -128,7 +128,7 @@ describe('Import', function () {
// Grab the data from tables
return Promise.all([
knex('users').select(),
knex('posts').select(),
models.Post.findPage(testUtils.context.internal),
knex('settings').select(),
knex('tags').select(),
knex('subscribers').select()
@ -139,7 +139,7 @@ describe('Import', function () {
importedData.length.should.equal(5, 'Did not get data successfully');
var users = importedData[0],
posts = importedData[1],
posts = importedData[1].posts,
settings = importedData[2],
tags = importedData[3],
subscribers = importedData[4];
@ -151,8 +151,8 @@ describe('Import', function () {
// import no longer requires all data to be dropped, and adds posts
posts.length.should.equal(exportData.data.posts.length, 'Wrong number of posts');
posts[0].status.should.eql('published');
posts[1].status.should.eql('scheduled');
posts[0].status.should.eql('scheduled');
posts[1].status.should.eql('published');
// test settings
settings.length.should.be.above(0, 'Wrong number of settings');