You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@echarts.apache.org by sh...@apache.org on 2021/05/17 05:55:41 UTC

[echarts] branch enhance-visual-regression-test updated (fd7a5b3 -> d864d49)

This is an automated email from the ASF dual-hosted git repository.

shenyi pushed a change to branch enhance-visual-regression-test
in repository https://gitbox.apache.org/repos/asf/echarts.git.


    from fd7a5b3  test: not throw error in sankey-ringed
     add 7d6d994  fix(clip): clipShape may be null if `clip` is set as `false`, resolves #14803.
     add 8af0c62  Merge pull request #14823 from apache/enhance-workflow
     add 9a2a525  Merge pull request #14825 from apache/enhance-workflow
     add 555bb6e  Update langEN.ts
     add 7f59518  Merge pull request #14860 from mKainzbauer/patch-1
     add 5826ca5  Merge pull request #14862 from apache/enhance-visual-regression-test
     add 5162c4f  chore: enhance the script compatibility in Windows and fix some lint errors.
     add f1f4556  chore: remove unused import and code.
     add 76c7f7a  chore: remove unused import and code.
     add 38b7b77  chore: use `npx` for script.
     add 9c788fb  Merge pull request #14872 from apache/fix-scripts
     add 12f6620  Merge pull request #14894 from apache/enhance-visual-regression-test
     add 85f97ea  Merge pull request #14906 from apache/enhance-visual-regression-test
     add 0f4408d  fix(custom): remove incremental el when switched back to normal.
     add 2400e25  chore: update zrender to latest nightly
     add 1166ac1  chore: update zrender to nightly version.
     add 466ff83  Merge pull request #14920 from apache/fix-custom-progressive
     add 404dd52  fix(legend): remove unexpected `startsWith` syntax in `LegendView`, use `lastIndexOf(search, 0) === 0` instead.
     add d9e1fcf  Merge pull request #14810 from apache/fix-grammar
     add 2d53ee6  fix(toolbox): the `show` option of customized toolbox button does not work, resolves #14405.
     add 0ac6f31  Merge pull request #14408 from apache/fix-toolbox-custom
     add a76ee5d  fix(dataZoom): type fix for startValue and endValue. close #14412
     add 23a2f7f  fix: code review comments
     add f97b7fa  Merge pull request #14775 from dileepyelleti/fix-14412
     add ca18d1e  Merge branch 'master' of https://github.com/apache/echarts into fix-line-clip
     add 816c122  test: move the test case in `line-clip.html` to `clip.html`.
     add ea5a850  Merge pull request #14813 from apache/fix-line-clip
     add 058d030  fix(tree): error when running setOption twice with diff data
     add d571d32  replace optional chaining
     add 626ae20  Merge pull request #14905 from Map1en/fix-err-when-tree-setoption-twice
     add b310a66  fix(tooltip): use kebab-case in transition property.
     add a7c223b  Merge pull request #14928 from apache/fix-tooltip-transform
     add 3f86f91  fix: resize throw error. fix #14846, fix #11395 .
     add a06096a  fix: if resize silent specified with not null/undefined, always use resize silent.
     add 7e78f24  Merge pull request #14934 from apache/fix/schedule
     new d864d49  test(visual): optimize all tests runs list

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 package-lock.json                                 |   6 +-
 package.json                                      |  10 +-
 src/chart/candlestick/candlestickVisual.ts        |   4 +-
 src/chart/custom/install.ts                       |   6 +
 src/chart/funnel/FunnelSeries.ts                  |   2 -
 src/chart/funnel/FunnelView.ts                    |   3 +-
 src/chart/helper/Line.ts                          |   2 +-
 src/chart/line/LineView.ts                        |  48 +--
 src/chart/line/install.ts                         |   5 +-
 src/chart/line/lineAnimationDiff.ts               |   2 +-
 src/chart/sankey/SankeySeries.ts                  |   6 +-
 src/chart/tree/TreeSeries.ts                      |   2 +-
 src/component/axis/CartesianAxisView.ts           |   3 +-
 src/component/axis/SingleAxisView.ts              |   1 -
 src/component/axisPointer/CartesianAxisPointer.ts |   3 +-
 src/component/axisPointer/PolarAxisPointer.ts     |   6 +-
 src/component/axisPointer/SingleAxisPointer.ts    |   3 +-
 src/component/dataZoom/AxisProxy.ts               |   4 +-
 src/component/dataZoom/DataZoomModel.ts           |   8 +-
 src/component/dataZoom/SliderZoomView.ts          |   3 +-
 src/component/legend/LegendView.ts                |   4 +-
 src/component/marker/MarkLineView.ts              |  20 +-
 src/component/marker/MarkerView.ts                |   7 +-
 src/component/radar/install.ts                    |   4 +-
 src/component/timeline/SliderTimelineModel.ts     |   4 +-
 src/component/toolbox/ToolboxView.ts              |  24 +-
 src/component/toolbox/feature/DataView.ts         |   2 +
 src/component/toolbox/feature/SaveAsImage.ts      |   4 +-
 src/component/tooltip/TooltipHTMLContent.ts       |   2 +-
 src/coord/calendar/Calendar.ts                    |   1 -
 src/coord/radar/Radar.ts                          |   3 +-
 src/core/echarts.ts                               |  18 +-
 src/data/helper/sourceHelper.ts                   |   5 +-
 src/i18n/langEN.ts                                |   4 +-
 src/model/mixin/palette.ts                        |   1 -
 test/build/mktest-tpl.html                        |   4 +-
 test/clip.html                                    |  34 +-
 test/custom-bmap-grid.html                        |   1 -
 test/custom-large.html                            | 194 +++++++++++
 test/runTest/actions/__meta__.json                |   4 +-
 test/runTest/actions/custom-bmap-grid.json        |   1 +
 test/runTest/actions/custom-large.json            |   1 +
 test/runTest/actions/tooltip-axisPointer2.json    |   2 +-
 test/runTest/client/client.js                     |   3 +
 test/runTest/client/index.html                    |   2 +-
 test/runTest/store.js                             |  20 +-
 test/toolbox-custom.html                          |  90 +++++
 test/tooltip-axisPointer2.html                    | 406 +++++++++++++++++++++-
 test/tree-setoption-twice.html                    |  80 +++++
 49 files changed, 956 insertions(+), 116 deletions(-)
 create mode 100644 test/custom-large.html
 create mode 100644 test/runTest/actions/custom-bmap-grid.json
 create mode 100644 test/runTest/actions/custom-large.json
 create mode 100644 test/toolbox-custom.html
 create mode 100644 test/tree-setoption-twice.html

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@echarts.apache.org
For additional commands, e-mail: commits-help@echarts.apache.org


[echarts] 01/01: test(visual): optimize all tests runs list

Posted by sh...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

shenyi pushed a commit to branch enhance-visual-regression-test
in repository https://gitbox.apache.org/repos/asf/echarts.git

commit d864d498aa2c7d56cc382db218b5a4eb5bbc69e0
Author: pissang <bm...@gmail.com>
AuthorDate: Mon May 17 13:54:13 2021 +0800

    test(visual): optimize all tests runs list
---
 test/runTest/client/client.js  |  3 +++
 test/runTest/client/index.html |  2 +-
 test/runTest/store.js          | 20 ++++++++++++++------
 3 files changed, 18 insertions(+), 7 deletions(-)

diff --git a/test/runTest/client/client.js b/test/runTest/client/client.js
index b761aca..f110fbd 100644
--- a/test/runTest/client/client.js
+++ b/test/runTest/client/client.js
@@ -124,6 +124,7 @@ const app = new Vue({
         // List of all runs.
         showRunsDialog: false,
         testsRuns: [],
+        loadingTestsRuns: false,
 
         pageInvisible: false,
 
@@ -354,6 +355,7 @@ const app = new Vue({
 
         showAllTestsRuns() {
             this.showRunsDialog = true;
+            this.loadingTestsRuns = true;
             socket.emit('getAllTestsRuns');
         },
 
@@ -474,6 +476,7 @@ socket.on('abort', res => {
 
 socket.on('getAllTestsRuns_return', res => {
     app.testsRuns = res.runs;
+    app.loadingTestsRuns = false;
 });
 socket.on('genTestsRunReport_return', res => {
     window.open(res.reportUrl, '_blank');
diff --git a/test/runTest/client/index.html b/test/runTest/client/index.html
index 0db49ae..5e876c3 100644
--- a/test/runTest/client/index.html
+++ b/test/runTest/client/index.html
@@ -292,7 +292,7 @@ under the License.
                     :visible.sync="showRunsDialog"
                     title="All Tests Runs"
                 >
-                    <el-table :data="testsRuns">
+                    <el-table :data="testsRuns" v-loading="loadingTestsRuns">
                         <el-table-column property="expectedVersion" label="Expected" width="160"></el-table-column>
                         <el-table-column property="actualVersion" label="Actual" width="160"></el-table-column>
                         <el-table-column property="renderer" label="Renderer" width="100"></el-table-column>
diff --git a/test/runTest/store.js b/test/runTest/store.js
index f05f3d5..5eb3e1d 100644
--- a/test/runTest/store.js
+++ b/test/runTest/store.js
@@ -203,7 +203,8 @@ module.exports.updateTestsList = async function (
 };
 
 module.exports.saveTestsList = function () {
-    fse.outputFileSync(getResultFilePath(), JSON.stringify(_tests, null, 2), 'utf-8');
+    fse.ensureDirSync(getResultBaseDir());
+    fs.writeFileSync(getResultFilePath(), JSON.stringify(_tests, null, 2), 'utf-8');
 };
 
 module.exports.mergeTestsResults = function (testsResults) {
@@ -255,11 +256,18 @@ module.exports.getAllTestsRuns = async function () {
     }
     for (let dir of dirs) {
         const params = parseRunHash(dir);
-        const resultJson = JSON.parse(fs.readFileSync(path.join(
-            RESULTS_ROOT_DIR,
-            dir,
-            RESULT_FILE_NAME
-        ), 'utf-8'));
+        let resultJson = [];
+        try {
+            resultJson = JSON.parse(fs.readFileSync(path.join(
+                RESULTS_ROOT_DIR,
+                dir,
+                RESULT_FILE_NAME
+            ), 'utf-8'));
+        }
+        catch (e) {
+            console.error('Invalid result ' + dir)
+            continue;
+        }
 
         const total = resultJson.length;
         let lastRunTime = 0;

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@echarts.apache.org
For additional commands, e-mail: commits-help@echarts.apache.org