GIT_REPOSITORIES/LPP/Users/mperrinel/SciQLop-fork Commit - r830:8991b585417a

Modification of the test for the new vc request kernel

perrinel -

r830:8991b585417a

parent child

Context file:

r830:8991b585417a

core/tests/Variable/TestVariableSync.cpp +25 -23

                 iterations.push_back({std::make_shared<Synchronize>(0, syncId)});
                 iterations.push_back({std::make_shared<Synchronize>(1, syncId)});
-                // Moves var0: ranges of var0, var1 and var2 change
+                // Moves var0: ranges of var0, var1
-                auto newRange = range({12, 30}, {13, 30});
+                auto currentRange = range({12, 30}, {13, 30});
-                iterations.push_back({std::make_shared<Move>(0, newRange), {{0, newRange}, {1, newRange}}});
+                iterations.push_back(
+                    {std::make_shared<Move>(0, currentRange), {{0, currentRange}, {1, currentRange}}});
-                // Moves var1: ranges of var0, var1 and var2 change
+                // Moves var0: ranges of var0, var1
-                newRange = range({13, 0}, {14, 0});
+                auto pendingRange = range({13, 0}, {14, 0});
-                iterations.push_back({std::make_shared<Move>(0, newRange), {{0, newRange}, {1, newRange}}});
+                iterations.push_back(
+                    {std::make_shared<Move>(0, pendingRange), {{0, pendingRange}, {1, pendingRange}}});
-                // Moves var2: ranges of var0, var1 and var2 change
+                // Moves var0: ranges of var0, var1
-                newRange = range({13, 30}, {14, 30});
+                pendingRange = range({13, 30}, {14, 30});
                 iterations.push_back(
-                    {std::make_shared<Move>(0, newRange), {{0, newRange}, {1, newRange}, {2, newRange}}});
+                    {std::make_shared<Move>(0, pendingRange), {{0, pendingRange}, {1, pendingRange}}});
-                // Desyncs var2 and moves var0:
+                // moves var0:
                 // - ranges of var0 and var1 change
-                // - range of var2 doesn't change anymore
+                auto var2Range = pendingRange;
-                auto var2Range = newRange;
+                pendingRange = range({13, 45}, {14, 45});
-                newRange = range({13, 45}, {14, 45});
+                iterations.push_back(
-                iterations.push_back({std::make_shared<Synchronize>(2, syncId, false)});
+                    {std::make_shared<Move>(0, pendingRange), {{0, pendingRange}, {1, pendingRange}}});
-                iterations.push_back({std::make_shared<Move>(0, newRange), {{0, newRange}, {1, newRange}}});
                 // Shifts var0: although var1 is synchronized with var0, its range doesn't change
-                auto var1Range = newRange;
+                auto var1Range = pendingRange;
-                newRange = range({14, 45}, {15, 45});
+                pendingRange = range({14, 45}, {15, 45});
                 iterations.push_back(
-                    {std::make_shared<Move>(0, newRange, true), {{0, newRange}, {1, var1Range}}});
+                    {std::make_shared<Move>(0, pendingRange, false), {{0, pendingRange}, {1, pendingRange}}});
                 // Moves var0 through several operations:
                 // - range of var0 changes
                 //        {std::make_shared<Move>(0, newRange), {{0, oldRange}, {1, expectedRange}}});
                 // Pan left
-                moveVar0(range({14, 30}, {15, 30}), range({13, 30}, {14, 30}));
+                moveVar0(range({14, 30}, {15, 30}), range({14, 30}, {15, 30}));
                 // Pan right
-                moveVar0(range({16, 0}, {17, 0}), range({15, 0}, {16, 0}));
+                moveVar0(range({16, 0}, {17, 0}), range({16, 0}, {17, 0}));
                 // Zoom in
-                moveVar0(range({16, 30}, {16, 45}), range({15, 30}, {15, 45}));
+                moveVar0(range({16, 30}, {16, 45}), range({16, 30}, {16, 45}));
                 // Zoom out
-                moveVar0(range({16, 15}, {17, 0}), range({15, 15}, {16, 0}));
+                moveVar0(range({16, 15}, {17, 0}), range({16, 15}, {17, 0}));
-                QTest::newRow("syncWithAborting1") << syncId << initialRange << std::move(creations)
+                QTest::newRow("syncWithAborting1") << syncId << currentRange << std::move(creations)
                                                    << std::move(iterations) << 200;
             }
                         // Compares variable's range to the expected range
                         QVERIFY(variable != nullptr);
                         auto range = variable->range();
+                        qInfo() << "range vs expected range" << range << variable->range();
                         QCOMPARE(range, expectedRange);
                         // Compares variable's data with values expected for its range

plugins/mockplugin/tests/TestCosinusAcquisition.cpp +5 -2

                 auto validateVariable
                     = [dataSeries](std::shared_ptr<Variable> variable, const SqpRange &range) {
                           // Checks that the variable's range has changed
+                          qInfo() << "range vs expected range" << variable->range() << range;
                           QCOMPARE(variable->range(), range);
                           // Checks the variable's data series
                 auto variable = sqpApp->variableController().createVariable(
                     "MMS", {{COSINUS_TYPE_KEY, "scalar"}, {COSINUS_FREQUENCY_KEY, 100.}}, provider);
                 QFETCH(int, operationDelay);
                 QTest::qWait(operationDelay);
                 validateVariable(variable, initialRange);
+                QTest::qWait(operationDelay);
                 // Makes operations on the variable
                 QFETCH(std::vector<SqpRange>, operations);
                 for (const auto &operation : operations) {
                     // Asks request on the variable and waits during its execution
-                    sqpApp->variableController().onRequestDataLoading({variable}, operation, true);
+                    sqpApp->variableController().onRequestDataLoading({variable}, operation, false);
                     QTest::qWait(operationDelay);
                     validateVariable(variable, operation);
                 for (const auto &operation : operations) {
                     // Asks request on the variable and waits during its execution
-                    sqpApp->variableController().onRequestDataLoading({variable}, operation, true);
+                    sqpApp->variableController().onRequestDataLoading({variable}, operation, false);
                 }
                 QTest::qWait(operationDelay);
                 validateVariable(variable, operations.back());

General Comments 3

perrinel

|

Auto status change to "Under Review"

perrinel

|

Merge lasted acquisition developpement on main Sciqlop branch

Write
Preview

You need to be logged in to leave comments. Login now

	Site-wide shortcuts
/	Use quick search box
g h	Goto home page
g g	Goto my private gists page
g G	Goto my public gists page
g 0-9	Goto bookmarked items from 0-9
n r	New repository page
n g	New gist page

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository access permissions settings
t s	Toggle sidebar on some pages