[FIX] mail: tests: 'load more messages on scroll' works with phantomjs

Before this commit, this test was failing due to phantomjs assuming that
there could be more messages to display on the screen.

The issues comes from the bad support of flexbox with PhantomJS, and this
test broke when we changed a DIV display with 'flex' instead of 'block'.

For instance, this is the expected values:

    thread clientHeight:   960
    thread scrollHeight:  1629
    message clientHeight:   50

With 30 messages, 50 x 30 = 1500 < 1629 and 960 < 1629, so there are
enough messages loaded with this screen and we are not at the top of
the thread (this is the case when clientHeight = scrollHeight).

However, with phantomJS we have the following values:

    thread clientHeight:   944
    thread scrollHeight:   944
    message clientHeight:  259

As a result, with 30 messages, 50 x 30 = 1500 > 944 (looks like
messages do not vertically collide ?) and 944 = 944 so it assumes
we are at the top of the thread, so it fetches for more messages.

This commit fixes this test by enforcing a special rendering of the
thread for phantomJS.

Unfortunately, we can't simply make the hack on the test only. We
have to make discuss and thread aware of this because they should
adapt their rendering right away

1 files changed, 8 insertions, 4 deletions

diff --git a/addons/mail/static/src/js/discuss.js b/addons/mail/static/src/js/discuss.js
index d3ed2b06a1d..ede167aa6a5 100644
--- a/addons/mail/static/src/js/discuss.js
+++ b/addons/mail/static/src/js/discuss.js
@@ -196,10 +196,10 @@ var Discuss = AbstractAction.extend(ControlPanelMixin, {
     /**
      * @override
      * @param {Object} [options]
-    //  * @param {boolean} [options.phantomjs=false] if set, rendering of non-empty
-    //  *   thread will use 'block' display instead of 'flex', because phantomjs
-    //  *   sucks with flexbox... TODO: remove this option when we no longer use
-    //  *   phantomJS
+     * @param {boolean} [options.phantomjs=false] if set, rendering of non-empty
+     *   thread will use 'block' display instead of 'flex', because phantomjs
+     *   sucks with flexbox... TODO: remove this option when we no longer use
+     *   phantomJS
      */
     init: function (parent, action, options) {
         this._super.apply(this, arguments);
@@ -223,6 +223,9 @@ var Discuss = AbstractAction.extend(ControlPanelMixin, {
         this._selectedMessage = null;
         this._throttledUpdateThreads = _.throttle(
             this._updateThreads.bind(this), 100, { leading: false });
+
+        // TODO: remove this option when we no longer use phantomJS
+        this._withPhantomJS = this.options.phantomjs || false;
     },
     /**
      * @override
@@ -456,6 +459,7 @@ var Discuss = AbstractAction.extend(ControlPanelMixin, {
             displayEmailIcons: false,
             displayReplyIcons: true,
             displayBottomThreadFreeSpace: true,
+            withPhantomJS: this._withPhantomJS, // TODO: remove this option when we no longer use phantomJS
         };
     },
     /**