chore: Configured babel plugin for removing data-test from production code

[adam-stasiak]

SUMMARY

I added babel plugin for production debug babel variant - it trims data-test from production code. It requires passing BABEL_ENV=production while building prod.
I observed that it is not applicable for inputProps:

                        <Datetime
                          inputProps={{ 'data-test': 'date-from-input' }}

and for code in templates in superset/ directory:

<!-- Modal for misc messages / alerts  -->
    <div class="misc-modal modal fade" tabindex="-1" role="dialog" aria-labelledby="myModalLabel">
      <div class="modal-dialog" role="document">
        <div class="modal-content" data-test="modal-content">
          <div class="modal-header" data-test="modal-header">
            <button type="button" class="close" data-dismiss="modal" aria-label="Close" data-test="modal-header-close-button">

It is working for

      <tr {...{ 'data-test': 'table-row' }} className="row" key={record.id}>
        {tds}
      </tr>,

So I think the next step is to wrap somehow elements from superset/ and replace inputProps with some other wrapper for elements.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

before:

after:

TEST PLAN

ADDITIONAL INFORMATION

• Has associated issue:
• Changes UI
• Requires DB Migration.
• Confirm DB Migration upgrade and downgrade tested.
• Introduces new feature or API
• Removes existing feature or API

[adam-stasiak]

retry because of fail in not related test

[codecov-io]

Codecov Report

Merging #11327 into master will decrease coverage by 2.71%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #11327      +/-   ##
==========================================
- Coverage   65.66%   62.95%   -2.72%     
==========================================
  Files         835      444     -391     
  Lines       39656    14987   -24669     
  Branches     3604     3686      +82     
==========================================
- Hits        26042     9435   -16607     
+ Misses      13505     5371    -8134     
- Partials      109      181      +72     

Flag	Coverage Δ
#cypress	?
#javascript	62.95% <ø> (+0.27%)	⬆️
#python	?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
superset-frontend/src/SqlLab/App.jsx	0.00% <0.00%> (-100.00%)	⬇️
superset-frontend/src/explore/App.jsx	0.00% <0.00%> (-100.00%)	⬇️
superset-frontend/src/dashboard/App.jsx	0.00% <0.00%> (-100.00%)	⬇️
superset-frontend/src/explore/index.jsx	0.00% <0.00%> (-100.00%)	⬇️
superset-frontend/src/dashboard/index.jsx	0.00% <0.00%> (-100.00%)	⬇️
superset-frontend/src/setup/setupColors.js	0.00% <0.00%> (-100.00%)	⬇️
superset-frontend/src/chart/ChartContainer.jsx	0.00% <0.00%> (-100.00%)	⬇️
superset-frontend/src/setup/setupFormatters.js	0.00% <0.00%> (-100.00%)	⬇️
superset-frontend/src/explore/reducers/index.js	0.00% <0.00%> (-100.00%)	⬇️
superset-frontend/src/setup/setupPluginsExtra.js	0.00% <0.00%> (-100.00%)	⬇️
... and 593 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 735123d...dc279aa. Read the comment docs.

[willbarrett]

This looks reasonable to me, but would love an opinion from @nytai or @rusackas

[nytai]

LGTM. I still think there's a case for having data-test attrs in prod, but we can add them back if that time ever comes.