From 0ebccc5c65cd050aca35bb21fb26da25120d3bd5 Mon Sep 17 00:00:00 2001
From: Jason Williams <jaswilli@gmail.com>
Date: Fri, 1 Aug 2014 05:04:40 +0000
Subject: [PATCH] Fix tests to ensure correct post is being checked

Refs #3473
- Change tests to not assume that all inserted fixture data
  will end up with the same millisecond-precision time for
  results sorting.  If a test is set up to check the contents
  of a specific fixture extract it explictly from the results.
---
 .../integration/model/model_posts_spec.js     | 21 ++++++++++++++-----
 1 file changed, 16 insertions(+), 5 deletions(-)

diff --git a/core/test/integration/model/model_posts_spec.js b/core/test/integration/model/model_posts_spec.js
index d9a7a3a860..857bf00015 100644
--- a/core/test/integration/model/model_posts_spec.js
+++ b/core/test/integration/model/model_posts_spec.js
@@ -21,6 +21,10 @@ describe('Post Model', function () {
 
         should.exist(PostModel);
 
+        function extractFirstPost(posts) {
+            return _.filter(posts, { id: 1 })[0];
+        }
+
         function checkFirstPostData(firstPost) {
             should.not.exist(firstPost.author_id);
             firstPost.author.should.be.an.Object;
@@ -54,8 +58,14 @@ describe('Post Model', function () {
                 .then(function (results) {
                     should.exist(results);
                     results.length.should.be.above(0);
-                    firstPost = results.models[0].toJSON();
-                    checkFirstPostData(firstPost);
+                    posts = results.models.map(function (model) {
+                        return model.toJSON();
+                    });
+
+                    // the first post in the result is not always the post at
+                    // position 0 in the fixture data so we need to use extractFirstPost
+                    // to get the post with id: 1
+                    checkFirstPostData(extractFirstPost(posts));
 
                     done();
                 }).catch(done);
@@ -86,9 +96,10 @@ describe('Post Model', function () {
                     results.meta.pagination.pages.should.equal(1);
                     results.posts.length.should.equal(4);
 
-                    firstPost = results.posts[0];
-
-                    checkFirstPostData(firstPost);
+                    // the first post in the result is not always the post at
+                    // position 0 in the fixture data so we need to use extractFirstPost
+                    // to get the post with id: 1
+                    checkFirstPostData(extractFirstPost(results.posts));
 
                     done();
                 }).catch(done);