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.

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

#2741 - Ministry Filter: Search Programs and Offerings [e2e tests] #3883

Conversation

sh16011993
Copy link
Collaborator

@sh16011993 sh16011993 commented Nov 1, 2024

As a part of this PR, the following were completed:

  • Wrote the following e2e tests:

EducationProgramAESTController(e2e)-getProgramsSummary

√ Should not retrieve any education programs when the search for the programs is made and no program statuses are searched for including no search for the inactive programs. (655 ms)
√ Should retrieve all the education programs when the search for the programs is made and all program statuses are searched for including the inactive programs. (385 ms)
√ Should retrieve all the inactive education programs when the search for the programs is made and none of the program statuses are selected except for the inactive programs. (359 ms)
√ Should retrieve all the pending, declined and inactive education programs sorted by ascending program status when the search for the programs is made and pending, declined program statuses are selected along with the inactive programs. (380 ms)
√ Should retrieve all the approved and declined education programs when the search for the programs is made and only the approved and declined program statuses are selected. (349 ms)
√ Should retrieve all the approved and pending education programs matching the program name search string when the search for the programs is made and the approved and pending program statuses are selected along with the program name search filter. (370 ms)
√ Should retrieve all the declined and inactive education programs sorted by ascending program status matching the program name search string and the location name search string when the search for the programs is made and the declined program status is selected along with the inactive programs selected and the program name and location name search filter. (623 ms)


Screenshot of the executed tests:

image
  • Fixed the issue where the checkbox doesn't show for the v-select component when the list item is unchecked.

Issue:

image

Fix:

image

@sh16011993 sh16011993 self-assigned this Nov 1, 2024
},
);

it(
Copy link
Collaborator

Choose a reason for hiding this comment

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

This one would be a good candidate to have included a status sort desc to be validated.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

There is a problem testing this specific scenario here. The reason being that there are two Pending Program Status records and they can show up in any arbitrary order resulting in the test to fail arbitrarily.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed as discussed.

Copy link
Collaborator

Choose a reason for hiding this comment

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

I see the E2Es now having two scenarios with the sortOrder=ASC but the comment was suggesting one at least with the status sort desc. Was there any reason not to have it included in the scenarios?

Copy link
Collaborator

@andrepestana-aot andrepestana-aot left a comment

Choose a reason for hiding this comment

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

Added some comments. Please take a look.

Copy link
Collaborator

@andrepestana-aot andrepestana-aot left a comment

Choose a reason for hiding this comment

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

Looks good! Thanks for checking the suggestions!

Copy link
Collaborator

@andrewsignori-aot andrewsignori-aot left a comment

Choose a reason for hiding this comment

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

Looks good 👍

@@ -56,6 +56,11 @@
font: unset !important;
}

// Temporary fix for Vuetify beta that does not display the selection component dropdown checkbox properly.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is the comment required? Is the component still beta?

Copy link
Collaborator

@andrewsignori-aot andrewsignori-aot left a comment

Choose a reason for hiding this comment

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

👍

@sh16011993 sh16011993 added this pull request to the merge queue Nov 7, 2024
@sh16011993 sh16011993 removed this pull request from the merge queue due to a manual request Nov 7, 2024
Copy link

sonarqubecloud bot commented Nov 7, 2024

Copy link

github-actions bot commented Nov 7, 2024

Backend Unit Tests Coverage Report

Totals Coverage
Statements: 22.3% ( 3697 / 16575 )
Methods: 10.31% ( 212 / 2057 )
Lines: 25.63% ( 3207 / 12513 )
Branches: 13.87% ( 278 / 2005 )

Copy link

github-actions bot commented Nov 7, 2024

E2E Workflow Workers Coverage Report

Totals Coverage
Statements: 65.43% ( 583 / 891 )
Methods: 59.26% ( 64 / 108 )
Lines: 68.54% ( 464 / 677 )
Branches: 51.89% ( 55 / 106 )

Copy link

github-actions bot commented Nov 7, 2024

E2E Queue Consumers Coverage Report

Totals Coverage
Statements: 85.92% ( 1220 / 1420 )
Methods: 87.86% ( 123 / 140 )
Lines: 86.83% ( 1035 / 1192 )
Branches: 70.45% ( 62 / 88 )

Copy link

github-actions bot commented Nov 7, 2024

E2E SIMS API Coverage Report

Totals Coverage
Statements: 66.45% ( 5642 / 8490 )
Methods: 63.9% ( 692 / 1083 )
Lines: 70.48% ( 4445 / 6307 )
Branches: 45.91% ( 505 / 1100 )

@sh16011993 sh16011993 added this pull request to the merge queue Nov 7, 2024
Merged via the queue into main with commit 942c8e3 Nov 7, 2024
20 checks passed
@sh16011993 sh16011993 deleted the 2741_ministry_filter_search_programs_and_offerings_add_location_search_e2e_tests branch November 7, 2024 00:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants