Apply clang-format to source tree
[alexxy/gromacs.git] / src / gromacs / analysisdata / tests / mock_datamodule.cpp
index b2551ca65508b29a204dd8539e40b7d12e45a041..05046f5d40e4a5e4edfa9176a69877411b729229 100644 (file)
@@ -71,65 +71,65 @@ namespace test
  */
 class MockAnalysisDataModule::Impl
 {
-    public:
-        //! Initializes a mock object with the given flags.
-        explicit Impl(int flags);
-
-        /*! \brief
-         * Callback used to initialize reference data checks
-         *
-         * Called in response to dataStarted().
-         * Records the source data for later use (for access to data properties).
-         */
-        void startReferenceData(AbstractAnalysisData *data);
-        /*! \brief
-         * Callback used to check frame start against reference data.
-         *
-         * Called to check parameters and order of calls to frameStarted().
-         * In addition to reference data checks, this method checks statically
-         * that the new frame matches \a frameIndex_.
-         */
-        void startReferenceFrame(const AnalysisDataFrameHeader &header);
-        /*! \brief
-         * Callback used to check frame points against reference data.
-         *
-         * Called to check parameters and order of calls to pointsAdded().
-         */
-        void checkReferencePoints(const AnalysisDataPointSetRef &points);
-        /*! \brief
-         * Callback used to check frame finish against reference data.
-         *
-         * Called to check parameters and order of calls to frameFinished().
-         */
-        void finishReferenceFrame(const AnalysisDataFrameHeader &header);
-        /*! \brief
-         * Callback used to check serial frame finish with reference data.
-         *
-         * Called to check parameters and order of calls to
-         * frameFinishedSerial().
-         * \a frameIndex_ is incremented here.
-         */
-        void finishReferenceFrameSerial(int frameIndex);
-
-        /*! \brief
-         * Reference data checker to use for checking frames.
-         *
-         * Must be initialized if startReferenceFrame() is called.
-         */
-        TestReferenceChecker         rootChecker_;
-        /*! \brief
-         * Reference data checker to use to check the current frame.
-         *
-         * Initialized between startReferenceFrame() and finishReferenceFrame()
-         * calls.
-         */
-        TestReferenceChecker         frameChecker_;
-        //! Source data.
-        const AbstractAnalysisData  *source_;
-        //! Flags that will be returned by the mock module.
-        int                          flags_;
-        //! Index of the current/next frame.
-        int                          frameIndex_;
+public:
+    //! Initializes a mock object with the given flags.
+    explicit Impl(int flags);
+
+    /*! \brief
+     * Callback used to initialize reference data checks
+     *
+     * Called in response to dataStarted().
+     * Records the source data for later use (for access to data properties).
+     */
+    void startReferenceData(AbstractAnalysisData* data);
+    /*! \brief
+     * Callback used to check frame start against reference data.
+     *
+     * Called to check parameters and order of calls to frameStarted().
+     * In addition to reference data checks, this method checks statically
+     * that the new frame matches \a frameIndex_.
+     */
+    void startReferenceFrame(const AnalysisDataFrameHeader& header);
+    /*! \brief
+     * Callback used to check frame points against reference data.
+     *
+     * Called to check parameters and order of calls to pointsAdded().
+     */
+    void checkReferencePoints(const AnalysisDataPointSetRef& points);
+    /*! \brief
+     * Callback used to check frame finish against reference data.
+     *
+     * Called to check parameters and order of calls to frameFinished().
+     */
+    void finishReferenceFrame(const AnalysisDataFrameHeader& header);
+    /*! \brief
+     * Callback used to check serial frame finish with reference data.
+     *
+     * Called to check parameters and order of calls to
+     * frameFinishedSerial().
+     * \a frameIndex_ is incremented here.
+     */
+    void finishReferenceFrameSerial(int frameIndex);
+
+    /*! \brief
+     * Reference data checker to use for checking frames.
+     *
+     * Must be initialized if startReferenceFrame() is called.
+     */
+    TestReferenceChecker rootChecker_;
+    /*! \brief
+     * Reference data checker to use to check the current frame.
+     *
+     * Initialized between startReferenceFrame() and finishReferenceFrame()
+     * calls.
+     */
+    TestReferenceChecker frameChecker_;
+    //! Source data.
+    const AbstractAnalysisData* source_;
+    //! Flags that will be returned by the mock module.
+    int flags_;
+    //! Index of the current/next frame.
+    int frameIndex_;
 };
 
 namespace
@@ -140,8 +140,7 @@ namespace
  *
  * \ingroup module_analysisdata
  */
-void checkReferenceDataPoint(TestReferenceChecker    *checker,
-                             const AnalysisDataValue &value)
+void checkReferenceDataPoint(TestReferenceChecker* checker, const AnalysisDataValue& value)
 {
     TestReferenceChecker compound(checker->checkCompound("DataValue", nullptr));
     compound.checkReal(value.value(), "Value");
@@ -155,56 +154,45 @@ void checkReferenceDataPoint(TestReferenceChecker    *checker,
     }
 }
 
-}       // namespace
+} // namespace
 
-MockAnalysisDataModule::Impl::Impl(int flags)
-    : source_(nullptr), flags_(flags), frameIndex_(0)
-{
-}
+MockAnalysisDataModule::Impl::Impl(int flags) : source_(nullptr), flags_(flags), frameIndex_(0) {}
 
 
-void
-MockAnalysisDataModule::Impl::startReferenceData(AbstractAnalysisData *data)
+void MockAnalysisDataModule::Impl::startReferenceData(AbstractAnalysisData* data)
 {
     source_ = data;
 }
 
 
-void
-MockAnalysisDataModule::Impl::startReferenceFrame(
-        const AnalysisDataFrameHeader &header)
+void MockAnalysisDataModule::Impl::startReferenceFrame(const AnalysisDataFrameHeader& header)
 {
-    GMX_RELEASE_ASSERT(rootChecker_,
-                       "Root checker not set, but reference data used");
+    GMX_RELEASE_ASSERT(rootChecker_, "Root checker not set, but reference data used");
     EXPECT_FALSE(frameChecker_.isValid());
     EXPECT_EQ(frameIndex_, header.index());
-    frameChecker_ = rootChecker_.checkCompound("DataFrame",
-                                               formatString("Frame%d", frameIndex_).c_str());
+    frameChecker_ =
+            rootChecker_.checkCompound("DataFrame", formatString("Frame%d", frameIndex_).c_str());
     frameChecker_.checkReal(header.x(), "X");
 }
 
 
-void
-MockAnalysisDataModule::Impl::checkReferencePoints(
-        const AnalysisDataPointSetRef &points)
+void MockAnalysisDataModule::Impl::checkReferencePoints(const AnalysisDataPointSetRef& points)
 {
     EXPECT_TRUE(frameChecker_.isValid());
     if (frameChecker_)
     {
-        TestReferenceChecker checker(
-                frameChecker_.checkCompound("DataValues", nullptr));
+        TestReferenceChecker checker(frameChecker_.checkCompound("DataValues", nullptr));
         checker.checkInteger(points.columnCount(), "Count");
         if (checker.checkPresent(source_->dataSetCount() > 1, "DataSet"))
         {
             checker.checkInteger(points.dataSetIndex(), "DataSet");
         }
-        const int  sourceColumnCount = source_->columnCount(points.dataSetIndex());
-        const bool bAllColumns       = (points.firstColumn() == 0
-                                        && points.columnCount() == sourceColumnCount);
+        const int sourceColumnCount = source_->columnCount(points.dataSetIndex());
+        const bool bAllColumns = (points.firstColumn() == 0 && points.columnCount() == sourceColumnCount);
         if (checker.checkPresent(!bAllColumns, "FirstColumn"))
         {
             checker.checkInteger(points.firstColumn(), "FirstColumn");
-            checker.checkInteger(points.lastColumn(),  "LastColumn");
+            checker.checkInteger(points.lastColumn(), "LastColumn");
         }
 
         AnalysisDataValuesRef::const_iterator value;
@@ -216,9 +204,7 @@ MockAnalysisDataModule::Impl::checkReferencePoints(
 }
 
 
-void
-MockAnalysisDataModule::Impl::finishReferenceFrame(
-        const AnalysisDataFrameHeader &header)
+void MockAnalysisDataModule::Impl::finishReferenceFrame(const AnalysisDataFrameHeader& header)
 {
     EXPECT_TRUE(frameChecker_.isValid());
     EXPECT_EQ(frameIndex_, header.index());
@@ -226,8 +212,7 @@ MockAnalysisDataModule::Impl::finishReferenceFrame(
 }
 
 
-void
-MockAnalysisDataModule::Impl::finishReferenceFrameSerial(int frameIndex)
+void MockAnalysisDataModule::Impl::finishReferenceFrameSerial(int frameIndex)
 {
     EXPECT_FALSE(frameChecker_.isValid());
     EXPECT_EQ(frameIndex_, frameIndex);
@@ -248,8 +233,7 @@ namespace
  * \param[in] header    Frame header to check.
  * \param[in] refFrame  Data to check against.
  */
-void checkHeader(const AnalysisDataFrameHeader    &header,
-                 const AnalysisDataTestInputFrame &refFrame)
+void checkHeader(const AnalysisDataFrameHeader& header, const AnalysisDataTestInputFrame& refFrame)
 {
     EXPECT_EQ(refFrame.index(), header.index());
     EXPECT_FLOAT_EQ(refFrame.x(), header.x());
@@ -263,19 +247,18 @@ void checkHeader(const AnalysisDataFrameHeader    &header,
  * \param[in] refPoints    Data to check against.
  * \param[in] columnOffset Offset of first column of \p points in \p refPoints.
  */
-void checkPoints(const AnalysisDataPointSetRef       &points,
-                 const AnalysisDataTestInputPointSet &refPoints,
+void checkPoints(const AnalysisDataPointSetRef&       points,
+                 const AnalysisDataTestInputPointSetrefPoints,
                  int                                  columnOffset)
 {
     for (int i = 0; i < points.columnCount(); ++i)
     {
         const int column = points.firstColumn() - refPoints.firstColumn() + i + columnOffset;
-        EXPECT_FLOAT_EQ(refPoints.y(column),
-                        points.y(i))
-        << "  Column: " << i+1 << " / " << points.columnCount()
-        << " (+" << points.firstColumn() << ")\n"
-        << "Ref. col: " << column+1 << " / " << refPoints.size()
-        << " (+" << refPoints.firstColumn() << ", offs " << columnOffset << ")";
+        EXPECT_FLOAT_EQ(refPoints.y(column), points.y(i))
+                << "  Column: " << i + 1 << " / " << points.columnCount() << " (+"
+                << points.firstColumn() << ")\n"
+                << "Ref. col: " << column + 1 << " / " << refPoints.size() << " (+"
+                << refPoints.firstColumn() << ", offs " << columnOffset << ")";
     }
 }
 
@@ -285,15 +268,14 @@ void checkPoints(const AnalysisDataPointSetRef       &points,
  * \param[in] frame     Frame to check.
  * \param[in] refFrame  Data to check against.
  */
-void checkFrame(const AnalysisDataFrameRef       &frame,
-                const AnalysisDataTestInputFrame &refFrame)
+void checkFrame(const AnalysisDataFrameRef& frame, const AnalysisDataTestInputFrame& refFrame)
 {
     checkHeader(frame.header(), refFrame);
     ASSERT_EQ(refFrame.pointSetCount(), frame.pointSetCount());
     for (int i = 0; i < frame.pointSetCount(); ++i)
     {
-        const AnalysisDataPointSetRef       &points    = frame.pointSet(i);
-        const AnalysisDataTestInputPointSet &refPoints = refFrame.pointSet(i);
+        const AnalysisDataPointSetRef&       points    = frame.pointSet(i);
+        const AnalysisDataTestInputPointSetrefPoints = refFrame.pointSet(i);
         EXPECT_EQ(refPoints.firstColumn(), points.firstColumn());
         checkPoints(points, refPoints, 0);
     }
@@ -307,28 +289,25 @@ void checkFrame(const AnalysisDataFrameRef       &frame,
  */
 class StaticDataFrameHeaderChecker
 {
-    public:
-        /*! \brief
-         * Constructs a checker against a given input data frame.
-         *
-         * \param[in] frame Frame to check against.
-         *
-         * \p frame must exist for the lifetime of this object.
-         */
-        StaticDataFrameHeaderChecker(const AnalysisDataTestInputFrame *frame)
-            : frame_(frame)
-        {
-        }
-
-        //! Function call operator for the functor.
-        void operator()(const AnalysisDataFrameHeader &header) const
-        {
-            SCOPED_TRACE(formatString("Frame %d", frame_->index()));
-            checkHeader(header, *frame_);
-        }
+public:
+    /*! \brief
+     * Constructs a checker against a given input data frame.
+     *
+     * \param[in] frame Frame to check against.
+     *
+     * \p frame must exist for the lifetime of this object.
+     */
+    StaticDataFrameHeaderChecker(const AnalysisDataTestInputFrame* frame) : frame_(frame) {}
+
+    //! Function call operator for the functor.
+    void operator()(const AnalysisDataFrameHeader& header) const
+    {
+        SCOPED_TRACE(formatString("Frame %d", frame_->index()));
+        checkHeader(header, *frame_);
+    }
 
-    private:
-        const AnalysisDataTestInputFrame *frame_;
+private:
+    const AnalysisDataTestInputFrame* frame_;
 };
 
 /*! \brief
@@ -339,47 +318,48 @@ class StaticDataFrameHeaderChecker
  */
 class StaticDataPointsChecker
 {
-    public:
-        /*! \brief
-         * Constructs a checker against a given input data frame and point set.
-         *
-         * \param[in] frame    Frame to check against.
-         * \param[in] points   Point set in \p frame to check against.
-         * \param[in] firstcol Expected first column.
-         * \param[in] n        Expected number of columns.
-         *
-         * \p firstcol and \p n are used to create a checker that only expects
-         * to be called for a subset of columns.
-         * \p frame and \p points must exist for the lifetime of this object.
-         */
-        StaticDataPointsChecker(const AnalysisDataTestInputFrame *frame,
-                                const AnalysisDataTestInputPointSet *points,
-                                int firstcol, int n)
-            : frame_(frame), points_(points), firstcol_(firstcol), n_(n)
-        {
-        }
+public:
+    /*! \brief
+     * Constructs a checker against a given input data frame and point set.
+     *
+     * \param[in] frame    Frame to check against.
+     * \param[in] points   Point set in \p frame to check against.
+     * \param[in] firstcol Expected first column.
+     * \param[in] n        Expected number of columns.
+     *
+     * \p firstcol and \p n are used to create a checker that only expects
+     * to be called for a subset of columns.
+     * \p frame and \p points must exist for the lifetime of this object.
+     */
+    StaticDataPointsChecker(const AnalysisDataTestInputFrame*    frame,
+                            const AnalysisDataTestInputPointSet* points,
+                            int                                  firstcol,
+                            int                                  n) :
+        frame_(frame),
+        points_(points),
+        firstcol_(firstcol),
+        n_(n)
+    {
+    }
 
-        //! Function call operator for the functor.
-        void operator()(const AnalysisDataPointSetRef &points) const
-        {
-            SCOPED_TRACE(formatString("Frame %d, point set %d",
-                                      frame_->index(), points_->index()));
-            EXPECT_EQ(points_->dataSetIndex(), points.dataSetIndex());
-            const int expectedFirstColumn
-                = std::max(0, points_->firstColumn() - firstcol_);
-            const int expectedLastColumn
-                = std::min(n_ - 1, points_->lastColumn() - firstcol_);
-            EXPECT_EQ(expectedFirstColumn, points.firstColumn());
-            EXPECT_EQ(expectedLastColumn,  points.lastColumn());
-            checkHeader(points.header(), *frame_);
-            checkPoints(points, *points_, firstcol_);
-        }
+    //! Function call operator for the functor.
+    void operator()(const AnalysisDataPointSetRef& points) const
+    {
+        SCOPED_TRACE(formatString("Frame %d, point set %d", frame_->index(), points_->index()));
+        EXPECT_EQ(points_->dataSetIndex(), points.dataSetIndex());
+        const int expectedFirstColumn = std::max(0, points_->firstColumn() - firstcol_);
+        const int expectedLastColumn  = std::min(n_ - 1, points_->lastColumn() - firstcol_);
+        EXPECT_EQ(expectedFirstColumn, points.firstColumn());
+        EXPECT_EQ(expectedLastColumn, points.lastColumn());
+        checkHeader(points.header(), *frame_);
+        checkPoints(points, *points_, firstcol_);
+    }
 
-    private:
-        const AnalysisDataTestInputFrame    *frame_;
-        const AnalysisDataTestInputPointSet *points_;
-        int                                  firstcol_;
-        int                                  n_;
+private:
+    const AnalysisDataTestInputFrame*    frame_;
+    const AnalysisDataTestInputPointSet* points_;
+    int                                  firstcol_;
+    int                                  n_;
 };
 
 /*! \brief
@@ -390,25 +370,22 @@ class StaticDataPointsChecker
  */
 class DataStorageRequester
 {
-    public:
-        /*! \brief
-         * Constructs a functor that requests the given amount of storage.
-         *
-         * \param[in] count  Number of frames of storage to request, or
-         *      -1 for all frames.
-         *
-         * \see AbstractAnalysisData::requestStorage()
-         */
-        explicit DataStorageRequester(int count) : count_(count) {}
-
-        //! Function call operator for the functor.
-        void operator()(AbstractAnalysisData *data) const
-        {
-            EXPECT_TRUE(data->requestStorage(count_));
-        }
-
-    private:
-        int                     count_;
+public:
+    /*! \brief
+     * Constructs a functor that requests the given amount of storage.
+     *
+     * \param[in] count  Number of frames of storage to request, or
+     *      -1 for all frames.
+     *
+     * \see AbstractAnalysisData::requestStorage()
+     */
+    explicit DataStorageRequester(int count) : count_(count) {}
+
+    //! Function call operator for the functor.
+    void operator()(AbstractAnalysisData* data) const { EXPECT_TRUE(data->requestStorage(count_)); }
+
+private:
+    int count_;
 };
 
 /*! \brief
@@ -420,65 +397,66 @@ class DataStorageRequester
  */
 class StaticDataPointsStorageChecker
 {
-    public:
-        /*! \brief
-         * Constructs a checker for a given frame.
-         *
-         * \param[in] source     Data object that is being checked.
-         * \param[in] data       Test input data to check against.
-         * \param[in] frameIndex Frame index for which this functor expects
-         *      to be called.
-         * \param[in] pointSetIndex Point set for which this functor expects
-         *      to be called.
-         * \param[in] storageCount How many past frames should be checked for
-         *      storage (-1 = check all frames).
-         *
-         * This checker works as StaticDataPointsChecker, but additionally
-         * checks that previous frames can be accessed using access methods
-         * in AbstractAnalysisData and that correct data is returned.
-         *
-         * \p source and \p data must exist for the lifetime of this object.
-         */
-        StaticDataPointsStorageChecker(AbstractAnalysisData        *source,
-                                       const AnalysisDataTestInput *data,
-                                       int frameIndex, int pointSetIndex,
-                                       int storageCount)
-            : source_(source), data_(data),
-              frameIndex_(frameIndex), pointSetIndex_(pointSetIndex),
-              storageCount_(storageCount)
-        {
-        }
+public:
+    /*! \brief
+     * Constructs a checker for a given frame.
+     *
+     * \param[in] source     Data object that is being checked.
+     * \param[in] data       Test input data to check against.
+     * \param[in] frameIndex Frame index for which this functor expects
+     *      to be called.
+     * \param[in] pointSetIndex Point set for which this functor expects
+     *      to be called.
+     * \param[in] storageCount How many past frames should be checked for
+     *      storage (-1 = check all frames).
+     *
+     * This checker works as StaticDataPointsChecker, but additionally
+     * checks that previous frames can be accessed using access methods
+     * in AbstractAnalysisData and that correct data is returned.
+     *
+     * \p source and \p data must exist for the lifetime of this object.
+     */
+    StaticDataPointsStorageChecker(AbstractAnalysisData*        source,
+                                   const AnalysisDataTestInput* data,
+                                   int                          frameIndex,
+                                   int                          pointSetIndex,
+                                   int                          storageCount) :
+        source_(source),
+        data_(data),
+        frameIndex_(frameIndex),
+        pointSetIndex_(pointSetIndex),
+        storageCount_(storageCount)
+    {
+    }
 
-        //! Function call operator for the functor.
-        void operator()(const AnalysisDataPointSetRef &points) const
+    //! Function call operator for the functor.
+    void operator()(const AnalysisDataPointSetRef& points) const
+    {
+        SCOPED_TRACE(formatString("Frame %d", frameIndex_));
+        const AnalysisDataTestInputFrame&    refFrame  = data_->frame(frameIndex_);
+        const AnalysisDataTestInputPointSet& refPoints = refFrame.pointSet(pointSetIndex_);
+        EXPECT_EQ(refPoints.firstColumn(), points.firstColumn());
+        EXPECT_EQ(refPoints.size(), points.columnCount());
+        checkHeader(points.header(), refFrame);
+        checkPoints(points, refPoints, 0);
+        for (int past = 1; (storageCount_ < 0 || past <= storageCount_) && past <= frameIndex_; ++past)
         {
-            SCOPED_TRACE(formatString("Frame %d", frameIndex_));
-            const AnalysisDataTestInputFrame    &refFrame  = data_->frame(frameIndex_);
-            const AnalysisDataTestInputPointSet &refPoints = refFrame.pointSet(pointSetIndex_);
-            EXPECT_EQ(refPoints.firstColumn(), points.firstColumn());
-            EXPECT_EQ(refPoints.size(), points.columnCount());
-            checkHeader(points.header(), refFrame);
-            checkPoints(points, refPoints, 0);
-            for (int past = 1;
-                 (storageCount_ < 0 || past <= storageCount_) && past <= frameIndex_;
-                 ++past)
-            {
-                int   index = frameIndex_ - past;
-                SCOPED_TRACE(formatString("Checking storage of frame %d", index));
-                ASSERT_NO_THROW_GMX({
-                                        AnalysisDataFrameRef frame = source_->getDataFrame(index);
-                                        ASSERT_TRUE(frame.isValid());
-                                        checkFrame(frame, data_->frame(index));
-                                    });
-            }
+            int index = frameIndex_ - past;
+            SCOPED_TRACE(formatString("Checking storage of frame %d", index));
+            ASSERT_NO_THROW_GMX({
+                AnalysisDataFrameRef frame = source_->getDataFrame(index);
+                ASSERT_TRUE(frame.isValid());
+                checkFrame(frame, data_->frame(index));
+            });
         }
+    }
 
-    private:
-        AbstractAnalysisData        *source_;
-        const AnalysisDataTestInput *data_;
-        int                          frameIndex_;
-        int                          pointSetIndex_;
-        int                          storageCount_;
+private:
+    AbstractAnalysisData*        source_;
+    const AnalysisDataTestInput* data_;
+    int                          frameIndex_;
+    int                          pointSetIndex_;
+    int                          storageCount_;
 };
 
 /*! \brief
@@ -488,7 +466,7 @@ class StaticDataPointsStorageChecker
  * object was an AnalysisDataModuleSerial object: forward the call to
  * MockAnalysisDataModule::dataStarted() and return false.
  */
-void setSerialExpectationForParallelDataStarted(MockAnalysisDataModule *mock)
+void setSerialExpectationForParallelDataStarted(MockAnalysisDataModulemock)
 {
     using ::testing::_;
     using ::testing::AtMost;
@@ -497,23 +475,17 @@ void setSerialExpectationForParallelDataStarted(MockAnalysisDataModule *mock)
     using ::testing::Return;
     using ::testing::WithArg;
     EXPECT_CALL(*mock, parallelDataStarted(_, _))
-        .Times(AtMost(1))
-        .WillOnce(DoAll(WithArg<0>(Invoke(mock, &MockAnalysisDataModule::dataStarted)),
-                        Return(false)));
+            .Times(AtMost(1))
+            .WillOnce(DoAll(WithArg<0>(Invoke(mock, &MockAnalysisDataModule::dataStarted)), Return(false)));
 }
 
-}       // anonymous namespace
+} // anonymous namespace
 
 
-MockAnalysisDataModule::MockAnalysisDataModule(int flags)
-    : impl_(new Impl(flags))
-{
-}
+MockAnalysisDataModule::MockAnalysisDataModule(int flags) : impl_(new Impl(flags)) {}
 
 
-MockAnalysisDataModule::~MockAnalysisDataModule()
-{
-}
+MockAnalysisDataModule::~MockAnalysisDataModule() {}
 
 
 int MockAnalysisDataModule::flags() const
@@ -522,10 +494,9 @@ int MockAnalysisDataModule::flags() const
 }
 
 
-void
-MockAnalysisDataModule::setupStaticCheck(const AnalysisDataTestInput &data,
-                                         AbstractAnalysisData        *source,
-                                         bool                         bParallel)
+void MockAnalysisDataModule::setupStaticCheck(const AnalysisDataTestInput& data,
+                                              AbstractAnalysisData*        source,
+                                              bool                         bParallel)
 {
     impl_->flags_ |= efAllowMulticolumn | efAllowMultipoint | efAllowMultipleDataSets;
 
@@ -538,42 +509,39 @@ MockAnalysisDataModule::setupStaticCheck(const AnalysisDataTestInput &data,
     {
 #ifndef __clang_analyzer__
         ::testing::Expectation init =
-            EXPECT_CALL(*this, parallelDataStarted(source, _))
-                .WillOnce(Return(true));
+                EXPECT_CALL(*this, parallelDataStarted(source, _)).WillOnce(Return(true));
         ::testing::ExpectationSet framesFinished;
         ::testing::Expectation    prevFinish;
         for (int row = 0; row < data.frameCount(); ++row)
         {
-            ::testing::InSequence frameSequence;
-            const AnalysisDataTestInputFrame &frame = data.frame(row);
+            ::testing::InSequence             frameSequence;
+            const AnalysisDataTestInputFrameframe = data.frame(row);
             EXPECT_CALL(*this, frameStarted(Property(&AnalysisDataFrameHeader::index, row)))
-                .After(init)
-                .WillOnce(Invoke(StaticDataFrameHeaderChecker(&frame)));
+                    .After(init)
+                    .WillOnce(Invoke(StaticDataFrameHeaderChecker(&frame)));
             for (int ps = 0; ps < frame.pointSetCount(); ++ps)
             {
-                const AnalysisDataTestInputPointSet &points = frame.pointSet(ps);
+                const AnalysisDataTestInputPointSetpoints = frame.pointSet(ps);
                 StaticDataPointsChecker              checker(&frame, &points, 0,
-                                                             data.columnCount(points.dataSetIndex()));
+                                                data.columnCount(points.dataSetIndex()));
                 EXPECT_CALL(*this, pointsAdded(Property(&AnalysisDataPointSetRef::frameIndex, row)))
-                    .WillOnce(Invoke(checker));
+                        .WillOnce(Invoke(checker));
             }
             EXPECT_CALL(*this, frameFinished(Property(&AnalysisDataFrameHeader::index, row)))
-                .WillOnce(Invoke(StaticDataFrameHeaderChecker(&frame)));
+                    .WillOnce(Invoke(StaticDataFrameHeaderChecker(&frame)));
             ::testing::Expectation finish;
             if (row > 0)
             {
-                finish = EXPECT_CALL(*this, frameFinishedSerial(row))
-                        .After(prevFinish);
+                finish = EXPECT_CALL(*this, frameFinishedSerial(row)).After(prevFinish);
             }
             else
             {
                 finish = EXPECT_CALL(*this, frameFinishedSerial(row));
             }
             framesFinished += finish;
-            prevFinish      = finish;
+            prevFinish = finish;
         }
-        EXPECT_CALL(*this, dataFinished())
-            .After(framesFinished);
+        EXPECT_CALL(*this, dataFinished()).After(framesFinished);
 #endif
     }
     else
@@ -583,18 +551,16 @@ MockAnalysisDataModule::setupStaticCheck(const AnalysisDataTestInput &data,
         EXPECT_CALL(*this, dataStarted(source));
         for (int row = 0; row < data.frameCount(); ++row)
         {
-            const AnalysisDataTestInputFrame &frame = data.frame(row);
-            EXPECT_CALL(*this, frameStarted(_))
-                .WillOnce(Invoke(StaticDataFrameHeaderChecker(&frame)));
+            const AnalysisDataTestInputFrame& frame = data.frame(row);
+            EXPECT_CALL(*this, frameStarted(_)).WillOnce(Invoke(StaticDataFrameHeaderChecker(&frame)));
             for (int ps = 0; ps < frame.pointSetCount(); ++ps)
             {
-                const AnalysisDataTestInputPointSet &points = frame.pointSet(ps);
+                const AnalysisDataTestInputPointSetpoints = frame.pointSet(ps);
                 StaticDataPointsChecker              checker(&frame, &points, 0,
-                                                             data.columnCount(points.dataSetIndex()));
+                                                data.columnCount(points.dataSetIndex()));
                 EXPECT_CALL(*this, pointsAdded(_)).WillOnce(Invoke(checker));
             }
-            EXPECT_CALL(*this, frameFinished(_))
-                .WillOnce(Invoke(StaticDataFrameHeaderChecker(&frame)));
+            EXPECT_CALL(*this, frameFinished(_)).WillOnce(Invoke(StaticDataFrameHeaderChecker(&frame)));
             EXPECT_CALL(*this, frameFinishedSerial(row));
         }
         EXPECT_CALL(*this, dataFinished());
@@ -602,10 +568,10 @@ MockAnalysisDataModule::setupStaticCheck(const AnalysisDataTestInput &data,
 }
 
 
-void
-MockAnalysisDataModule::setupStaticColumnCheck(
-        const AnalysisDataTestInput &data,
-        int firstcol, int n, AbstractAnalysisData * /*source*/)
+void MockAnalysisDataModule::setupStaticColumnCheck(const AnalysisDataTestInput& data,
+                                                    int                          firstcol,
+                                                    int                          n,
+                                                    AbstractAnalysisData* /*source*/)
 {
     impl_->flags_ |= efAllowMulticolumn | efAllowMultipoint | efAllowMultipleDataSets;
 
@@ -617,31 +583,27 @@ MockAnalysisDataModule::setupStaticColumnCheck(
     EXPECT_CALL(*this, dataStarted(_));
     for (int row = 0; row < data.frameCount(); ++row)
     {
-        const AnalysisDataTestInputFrame &frame = data.frame(row);
-        EXPECT_CALL(*this, frameStarted(_))
-            .WillOnce(Invoke(StaticDataFrameHeaderChecker(&frame)));
+        const AnalysisDataTestInputFrame& frame = data.frame(row);
+        EXPECT_CALL(*this, frameStarted(_)).WillOnce(Invoke(StaticDataFrameHeaderChecker(&frame)));
         for (int ps = 0; ps < frame.pointSetCount(); ++ps)
         {
-            const AnalysisDataTestInputPointSet &points = frame.pointSet(ps);
-            if (points.lastColumn() >= firstcol
-                && points.firstColumn() <= firstcol + n - 1)
+            const AnalysisDataTestInputPointSet& points = frame.pointSet(ps);
+            if (points.lastColumn() >= firstcol && points.firstColumn() <= firstcol + n - 1)
             {
                 EXPECT_CALL(*this, pointsAdded(_))
-                    .WillOnce(Invoke(StaticDataPointsChecker(&frame, &points, firstcol, n)));
+                        .WillOnce(Invoke(StaticDataPointsChecker(&frame, &points, firstcol, n)));
             }
         }
-        EXPECT_CALL(*this, frameFinished(_))
-            .WillOnce(Invoke(StaticDataFrameHeaderChecker(&frame)));
+        EXPECT_CALL(*this, frameFinished(_)).WillOnce(Invoke(StaticDataFrameHeaderChecker(&frame)));
         EXPECT_CALL(*this, frameFinishedSerial(row));
     }
     EXPECT_CALL(*this, dataFinished());
 }
 
 
-void
-MockAnalysisDataModule::setupStaticStorageCheck(
-        const AnalysisDataTestInput &data,
-        int storageCount, AbstractAnalysisData *source)
+void MockAnalysisDataModule::setupStaticStorageCheck(const AnalysisDataTestInput& data,
+                                                     int                          storageCount,
+                                                     AbstractAnalysisData*        source)
 {
     GMX_RELEASE_ASSERT(data.isMultipoint() == source->isMultipoint(),
                        "Mismatching multipoint properties");
@@ -652,33 +614,27 @@ MockAnalysisDataModule::setupStaticStorageCheck(
     using ::testing::Invoke;
 
     setSerialExpectationForParallelDataStarted(this);
-    EXPECT_CALL(*this, dataStarted(source))
-        .WillOnce(Invoke(DataStorageRequester(storageCount)));
+    EXPECT_CALL(*this, dataStarted(source)).WillOnce(Invoke(DataStorageRequester(storageCount)));
     for (int row = 0; row < data.frameCount(); ++row)
     {
-        const AnalysisDataTestInputFrame &frame = data.frame(row);
-        EXPECT_CALL(*this, frameStarted(_))
-            .WillOnce(Invoke(StaticDataFrameHeaderChecker(&frame)));
+        const AnalysisDataTestInputFrame& frame = data.frame(row);
+        EXPECT_CALL(*this, frameStarted(_)).WillOnce(Invoke(StaticDataFrameHeaderChecker(&frame)));
         for (int pointSet = 0; pointSet < frame.pointSetCount(); ++pointSet)
         {
-            StaticDataPointsStorageChecker checker(source, &data, row, pointSet,
-                                                   storageCount);
+            StaticDataPointsStorageChecker checker(source, &data, row, pointSet, storageCount);
             EXPECT_CALL(*this, pointsAdded(_)).WillOnce(Invoke(checker));
         }
-        EXPECT_CALL(*this, frameFinished(_))
-            .WillOnce(Invoke(StaticDataFrameHeaderChecker(&frame)));
+        EXPECT_CALL(*this, frameFinished(_)).WillOnce(Invoke(StaticDataFrameHeaderChecker(&frame)));
         EXPECT_CALL(*this, frameFinishedSerial(row));
     }
     EXPECT_CALL(*this, dataFinished());
 }
 
 
-void
-MockAnalysisDataModule::setupReferenceCheck(const TestReferenceChecker &checker,
-                                            AbstractAnalysisData       *source)
+void MockAnalysisDataModule::setupReferenceCheck(const TestReferenceChecker& checker,
+                                                 AbstractAnalysisData*       source)
 {
-    impl_->flags_ |= efAllowMulticolumn | efAllowMultipoint | efAllowMissing
-        | efAllowMultipleDataSets;
+    impl_->flags_ |= efAllowMulticolumn | efAllowMultipoint | efAllowMissing | efAllowMultipleDataSets;
 
     impl_->rootChecker_ = TestReferenceChecker(checker);
     // Google Mock does not support checking the order fully, because
@@ -691,22 +647,22 @@ MockAnalysisDataModule::setupReferenceCheck(const TestReferenceChecker &checker,
     using ::testing::Invoke;
 
     setSerialExpectationForParallelDataStarted(this);
-    Expectation dataStart = EXPECT_CALL(*this, dataStarted(source))
-            .WillOnce(Invoke(impl_.get(), &Impl::startReferenceData));
+    Expectation dataStart =
+            EXPECT_CALL(*this, dataStarted(source)).WillOnce(Invoke(impl_.get(), &Impl::startReferenceData));
     Expectation frameStart = EXPECT_CALL(*this, frameStarted(_))
-            .After(dataStart)
-            .WillRepeatedly(Invoke(impl_.get(), &Impl::startReferenceFrame));
+                                     .After(dataStart)
+                                     .WillRepeatedly(Invoke(impl_.get(), &Impl::startReferenceFrame));
     Expectation pointsAdd = EXPECT_CALL(*this, pointsAdded(_))
-            .After(dataStart)
-            .WillRepeatedly(Invoke(impl_.get(), &Impl::checkReferencePoints));
+                                    .After(dataStart)
+                                    .WillRepeatedly(Invoke(impl_.get(), &Impl::checkReferencePoints));
     Expectation frameFinish = EXPECT_CALL(*this, frameFinished(_))
-            .After(dataStart)
-            .WillRepeatedly(Invoke(impl_.get(), &Impl::finishReferenceFrame));
-    Expectation frameFinishSerial = EXPECT_CALL(*this, frameFinishedSerial(_))
-            .After(dataStart)
-            .WillRepeatedly(Invoke(impl_.get(), &Impl::finishReferenceFrameSerial));
-    EXPECT_CALL(*this, dataFinished())
-        .After(frameStart, pointsAdd, frameFinish, frameFinishSerial);
+                                      .After(dataStart)
+                                      .WillRepeatedly(Invoke(impl_.get(), &Impl::finishReferenceFrame));
+    Expectation frameFinishSerial =
+            EXPECT_CALL(*this, frameFinishedSerial(_))
+                    .After(dataStart)
+                    .WillRepeatedly(Invoke(impl_.get(), &Impl::finishReferenceFrameSerial));
+    EXPECT_CALL(*this, dataFinished()).After(frameStart, pointsAdd, frameFinish, frameFinishSerial);
 }
 
 } // namespace test