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

feat: apdex tests with script - fill up forms #9105

Merged
merged 23 commits into from May 16, 2024
Merged

feat: apdex tests with script - fill up forms #9105

merged 23 commits into from May 16, 2024

Conversation

latin-panda
Copy link
Contributor

@latin-panda latin-panda commented May 9, 2024 •
edited

Description

  • Adds functions that helps filling up forms based on the settings file
  • It includes support to fill fields that are just necessary for our current scope. But adding more field types should be following the same way done here.

medic/care-teams#102

Code review checklist

  • Readable: Concise, well named, follows the style guide, documented if necessary.
  • Documented: Configuration and user documentation on cht-docs
  • Tested: Unit and/or e2e where appropriate
  • Internationalised: All user facing text
  • Backwards compatible: Works with existing data and configuration or includes a migration. Any breaking changes documented in the release notes.

Compose URLs

If Build CI hasn't passed, these may 404:

License

The software is provided under AGPL-3.0. Contributions to this project are accepted under the same license.

@latin-panda
adding settings provider to read from file
2858381
@latin-panda
adding settings provider to read from file
97a3ac2
@latin-panda
adding some docs
acad7a1
@latin-panda
Removing functions that we don't need just now.
1205a21
@latin-panda
refactored capabilities settings
9da257c
@latin-panda
removing functions we dont need for now
aaa5cc8
@latin-panda
Adds privacy policy check
04d459f
@latin-panda
support many capabilities
2ad6668
@latin-panda
support many capabilities
4ab4a7d
@latin-panda
client list retry 40 times
9108191
@latin-panda
adding more navigation
44b6c1e
@latin-panda
adding more navigation
6036935
@latin-panda
making it nice
a4fb278
@latin-panda
Merge branch 'apdex-automation-tests' of https://github.com/medic/cht…
b206cf8 
…-core into apdex-script-navigation-and-retry

# Conflicts:
#	tests/performance/apdex-score/README.md
#	tests/performance/apdex-score/settings-provider.js
#	tests/performance/apdex-score/test/specs/num-1/test.apdex.js
@latin-panda
Sonar
5425d08
@latin-panda
dummy change to trigger CI
3aa7f30
@latin-panda
Merge branch 'apdex-script-navigation-and-retry' of https://github.co…
51e9f77 
…m/medic/cht-core into apdex-script-forms
@latin-panda
script to open, assert forms and nav form pages.
cf0813c
@latin-panda latin-panda linked an issue May 9, 2024 that may be closed by this pull request
Script to use the settings and fill up forms medic/care-teams#102
Closed
@latin-panda latin-panda changed the base branch from apdex-automation-tests to apdex-script-navigation-and-retry May 9, 2024 14:33
Base automatically changed from apdex-script-navigation-and-retry to apdex-automation-tests May 9, 2024 14:37
@latin-panda
Merge branch 'apdex-automation-tests' of https://github.com/medic/cht…
0d2877c 
…-core into apdex-script-forms

# Conflicts:
#	tests/performance/apdex-score/README.md
#	tests/performance/apdex-score/test/page-objects/contacts.page.js
#	tests/performance/apdex-score/test/page-objects/page.js
#	tests/performance/apdex-score/test/specs/num-1/test.apdex.js
@latin-panda
helper functions for UI interaction
2ce1302
@latin-panda
updating readme
053bbb6
@latin-panda
form
51d419e
@latin-panda
documentation
aafde1b
@latin-panda latin-panda marked this pull request as ready for review May 15, 2024 11:28
@latin-panda
Copy link
Contributor Author

@ralfudx @lorerod please review, this is the last piece to integrate the settings file. After this I will go back to another task in Care Teams and let you take it to the finish line, but if you have any questions or blocker let me know, I'll try my best to help!

ralfudx
ralfudx approved these changes May 15, 2024
View reviewed changes
Copy link

@ralfudx ralfudx left a comment

Choose a reason for hiding this comment

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

This looks good - I like that you added comments in the read me to describe some of the fields

@latin-panda
Copy link
Contributor Author

Thanks @ralfudx
@lorerod I'm going to proceed and merge, but please let me know if you have feedback and I will open a new PR to fix them.

@latin-panda latin-panda merged commit 5697137 into apdex-automation-tests May 16, 2024
31 checks passed
@latin-panda latin-panda deleted the apdex-script-forms branch May 16, 2024 04:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ralfudx ralfudx ralfudx approved these changes

@lorerod lorerod Awaiting requested review from lorerod

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Script to use the settings and fill up forms
2 participants
@latin-panda @ralfudx