test: be more lenient on jsx/tsx test coverage

[ktmud]

React modules don't have to have 100% unit-test coverages.

🏆 Enhancements

🏠 Internal

[vercel]

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/superset/superset-ui/gcq8ntdos
✅ Preview: https://superset-ui-git-fork-ktmud-codecov.superset.now.sh

[ktmud]

Kind of feel maybe we should just get rid of the patch check?

[codecov]

Codecov Report

Merging #468 into master will increase coverage by 0.11%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #468      +/-   ##
==========================================
+ Coverage   22.25%   22.36%   +0.11%     
==========================================
  Files         265      266       +1     
  Lines        6521     6549      +28     
  Branches      591      605      +14     
==========================================
+ Hits         1451     1465      +14     
- Misses       5033     5047      +14     
  Partials       37       37              

Impacted Files	Coverage Δ
plugins/preset-chart-xy/src/BoxPlot/index.ts	0.00% <0.00%> (ø)
plugins/legacy-plugin-chart-rose/src/index.js	0.00% <0.00%> (ø)
plugins/legacy-plugin-chart-sankey/src/index.js	0.00% <0.00%> (ø)
plugins/legacy-plugin-chart-treemap/src/index.js	0.00% <0.00%> (ø)
packages/superset-ui-translation/src/Translator.ts	100.00% <0.00%> (ø)
plugins/legacy-plugin-chart-world-map/src/index.js	0.00% <0.00%> (ø)
...lugins/plugin-chart-word-cloud/src/plugin/index.ts	33.33% <0.00%> (ø)
...lugins/preset-chart-xy/src/BoxPlot/legacy/index.ts	0.00% <0.00%> (ø)
...ugins/legacy-plugin-chart-pivot-table/src/index.js	0.00% <0.00%> (ø)
...ugins/preset-chart-xy/src/components/Line/Line.tsx	0.00% <0.00%> (ø)
... and 10 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0483db6...1057900. Read the comment docs.

[kristw]

100% ?

Could we combine the core-packages-ts and core-packages-tsx section then add the ignore path for control-utils's {t|j}sx?

[rusackas]

Probably not 100% since relaxing this constraint on {j|t}sx is the point. That said, the number can be negotiated/adjusted at any point. 50%? ¯\_(ツ)_/¯

[rusackas]

LGTM!

[kristw]

Not sure why

[ktmud]

Not sure why

Apparently you can't mix the use of "**" and "$". So projects/**/*.ts$ doesn't work, but projects/**/*.ts does.

[rusackas]

@kristw @ktmud Seems like this is good to go, right? My trigger finger is itchy here, as this will unblock this PR.