reg-viz / reg-suit

:recycle: Visual Regression Testing tool

Home Page:https://reg-viz.github.io/reg-suit

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Filter is not working if ximgdiff is disabled

stevschmid opened this issue · comments

Describe the bug

If ximgdiff is disabled in regconfig.json, filtering in the generated HTML report is not working.

Reproduced step

Check out reg-cli-report-ui, disable ximgdiffConfig in develop/index.html

ximgdiffConfig: { enabled: false, workerUrl: './worker.js' },

Start the dev env via yarn start. Filtering is not working.

Cause

https://github.com/reg-viz/reg-cli-report-ui/blob/73849dfa8bfc09497706f4fd0f1b47fe36683a88/src/worker-client.ts#L44-L46

this._worker is only created if ximg diff is enabled. Since this._worker is used to relay the filtering messages, the filtering is not working if XIMGDiffConfig.enabled = false

Expected behavior

Filtering is working even with ximgdiff disabled.

Actual behavior

Filtering is not working with ximgdiff disabled.

I'll reopen this in the reg-viz/reg-cli-report-ui repo directly.