[ 
https://issues.apache.org/jira/browse/HDDS-3182?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17786314#comment-17786314
 ] 

Abhishek Pal edited comment on HDDS-3182 at 11/15/23 11:18 AM:
---------------------------------------------------------------

As of now Enzyme would mostly be going EOL for React since v17 (not officially, 
but for v17 there isn't an official adapter yet, and for v18 things don't look 
that good either)
Though we are using Recon with React v16 - which has Enzyme adapter present, 
later down the line we are planning to upgrade React and we would again need to 
go through migrating these tests to a compatible library.

I suggest using the [React Testing 
Library|https://testing-library.com/docs/react-testing-library/intro] for these 
tests along with [Jest|https://jestjs.io/].
Any suggestions would be welcome regarding this.

One reason I want to work with this is because we have enabled [dependabot 
upgrades|https://issues.apache.org/jira/browse/HDDS-9171] for Ozone, and having 
unit tests in place, preferably running in the CI actions would allow us to 
safely merge the minor upgrades.

[~erose] [~vivekratnavel] [~adoroszlai]


was (Author: JIRAUSER291782):
As of now Enzyme would mostly be going EOL for React since v17 (not officially, 
but for v17 there isn't an official adapter yet, and for v18 things don't look 
that good either)
Though we are using Recon with React v16 - which has Enzyme adapter present, 
later down the line we are planning to upgrade React and we would again need to 
go through migrating these tests to a compatible library.

I suggest using the [React Testing 
Library|https://testing-library.com/docs/react-testing-library/intro] for these 
tests along with [Jest|https://jestjs.io/].
Any suggestions would be welcome regarding this.

One reason I want to work with this is because we have enabled [dependabot 
upgrades|https://issues.apache.org/jira/browse/HDDS-9171] for Ozone, and having 
unit tests in place, preferably running in the CI actions would allow us to 
safely merge the minor upgrades.

[~erose][~vivekratnavel][~adoroszlai]

> Add unit tests to Recon Frontend
> --------------------------------
>
>                 Key: HDDS-3182
>                 URL: https://issues.apache.org/jira/browse/HDDS-3182
>             Project: Apache Ozone
>          Issue Type: Task
>          Components: Ozone Recon
>    Affects Versions: 0.5.0
>            Reporter: Vivek Ratnavel Subramanian
>            Assignee: Abhishek Pal
>            Priority: Major
>              Labels: Triaged
>
> Add enzyme and jest libraries as dev dependencies and improve code coverage 
> with unit tests in Recon UI.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to