Skip to content

test(data-workspace): add tests#26

Merged
mediremi merged 9 commits into
masterfrom
test/data-workspace
Jul 20, 2021
Merged

test(data-workspace): add tests#26
mediremi merged 9 commits into
masterfrom
test/data-workspace

Conversation

@mediremi
Copy link
Copy Markdown
Contributor

No description provided.

@mediremi mediremi marked this pull request as ready for review July 14, 2021 16:38
@mediremi mediremi requested a review from Mohammer5 July 14, 2021 16:38
@@ -0,0 +1,105 @@
import React from 'react'
import { CustomDataProvider } from '@dhis2/app-runtime'
import { render, screen, waitFor, waitForElementToBeRemoved } from '@testing-library/react'
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I used @testing-library/react instead of enzyme for these tests as the various utils provided such as waitForElementToBeRemoved and getByRole made the tests much clearer.

Comment thread src/data-workspace/data-workspace.test.js Outdated
Comment thread src/data-workspace/data-workspace.test.js
Comment thread src/data-workspace/data-workspace.test.js Outdated
@mediremi mediremi merged commit af43ecd into master Jul 20, 2021
@dhis2-bot
Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 1.8.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants