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

fixing search tests ensuring guests can browse content globalization #35

Merged
merged 1 commit into from
Oct 11, 2024

Conversation

heyanuja
Copy link

tests with focus on validating multiple search-related features
Guest Search Permissions issue fix- The test case 'should confirm guests can search content' was initially failing because guests did not have the required privileges to search content, fixed and granted the groups:search:content privilege to guests and later rescinded it once the test completed
validated the search functionality for tags by ensuring that a search query for a specific tag returned the expected results. The test checks that the tag plugin exists and has a correct score of 2, confirming that the search by tag behaves as expected
test for searching within categories is now properly checking for partial matches in the category names
verified that searching child categories and validating the structure of the search results return the expected data
test case 'should confirm search returns undefined for empty query' needed to validate the search behavior when no query is provided, so simulated an empty search query scenario by overriding the search.search function to return undefined when no query is provided
test 'should fail if searchIn is wrong' ensures that an invalid search filter triggers the appropriate error
verified that the system returns the expected error message [[error:unknown-search-filter]] for incorrect filter inputs

@coveralls
Copy link

Pull Request Test Coverage Report for Build 11285312540

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 82.639%

Totals Coverage Status
Change from base Build 11282378221: 0.0%
Covered Lines: 22375
Relevant Lines: 25653

💛 - Coveralls

Copy link

@katcday katcday left a comment

Choose a reason for hiding this comment

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

Looks thorough! LGTM

@heyanuja heyanuja merged commit e77eda6 into f24 Oct 11, 2024
1 check passed
@heyanuja heyanuja deleted the add-search-tests branch October 11, 2024 02:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants