Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Test/pn-11507 - Implement sender dashboard unit tests #1261

Open
wants to merge 17 commits into
base: develop
Choose a base branch
from
Open

Test/pn-11507 - Implement sender dashboard unit tests #1261

wants to merge 17 commits into from

Conversation

mflauti
Copy link
Contributor

@mflauti mflauti commented Jun 19, 2024 •
edited
Loading

Short description

This PR contains sender dashboard ta code

List of changes proposed in this pull request

  • StatisticsDataFactoryManager.test.ts
  • StatisticsDataFactory.test.ts to verify that the factory can instantiate every StatisticsData concrete class and parses data properly

How to test

  • Run pa webapp tests and verify everything passes without errors
  • Verify on sonarcloud the new code increases the test coverage appropriately

@AndreaCimini90 AndreaCimini90 self-assigned this Jun 19, 2024
mflauti and others added 11 commits June 21, 2024 17:56
@mflauti
test: add unit tests for utility/StatisticsData module
e8f75d0 
test: add statistics mock data
@mflauti
test: add tests for statistics components
12901d9 
test: change testSelect method in common test-utils to allow selecting element using data-testid and role=combobox
@mflauti
chore: add vite, tsconfig, nvm, eslint, prettier, babel, sonar basic …
44993c9 
…config to pn-data-viz

chore: setup test environment for pn-data-viz
test: draft test and mocks for pn-data-viz
@mflauti
Merge branch 'develop' into test/PN-11507
f4e4539
@AndreaCimini90
removed useless code and done some rework
098c8bf
@AndreaCimini90
Merge remote-tracking branch 'origin/develop' into test/PN-11507
b86c3fb
@AndreaCimini90
fixed resize problems
8f8c3b7
@AndreaCimini90
fixed data-viz tests
92c0d9c
@AndreaCimini90
fixed pa tests (part 1)
8b5675a
@AndreaCimini90
Merge remote-tracking branch 'origin/develop' into test/PN-11507
9cdc52f
@AndreaCimini90
fixed tests (part2)
511bfb5
@AndreaCimini90 AndreaCimini90 changed the title DRAFT / DO NOT MERGE - Test/pn-11507 - Implement sender dashboard unit tests Test/pn-11507 - Implement sender dashboard unit tests Jul 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ZorbaDimatteo ZorbaDimatteo Awaiting requested review from ZorbaDimatteo ZorbaDimatteo is a code owner

@AndreaCimini90 AndreaCimini90 Awaiting requested review from AndreaCimini90 AndreaCimini90 is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@ZorbaDimatteo ZorbaDimatteo

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@mflauti @ZorbaDimatteo @AndreaCimini90