rusackas commented on code in PR #29118:
URL: https://github.com/apache/superset/pull/29118#discussion_r1643329349
##########
superset-frontend/src/dashboard/components/PropertiesModal/PropertiesModal.test.jsx:
##########
@@ -16,10 +16,15 @@
* specific language governing permissions and limitations
* under the License.
*/
-import { render } from 'spec/helpers/testing-library';
+import { mount } from 'enzyme';
+import { Provider } from 'react-redux';
Review Comment:
My mistake in the first place! Indeed, we don't want to migrate RTL tests to
Jest/Enzyme tests, so this would be a step backward. For _existing_ J/E tests,
we can use `styledMount`, but we ultimately want to move away from Jest/Enzyme
entirely (over time).
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
