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

Refactor functional tests to use new test utils #51

Closed
timkimadobe opened this issue Oct 26, 2023 · 0 comments · Fixed by #83
Closed

Refactor functional tests to use new test utils #51

timkimadobe opened this issue Oct 26, 2023 · 0 comments · Fixed by #83
Assignees
Labels
task Item that can be completed as a standalone code change or is part of an epic.Used by issue template

Comments

@timkimadobe
Copy link
Contributor

Task description

The existing functional tests should be refactored to use the newly available common test utilities available in: https://github.com/adobe/aepsdk-testutils-android

With the following items covered:

  1. Any changes required to the common test utils should be opened as a PR to the test utils repo, optimally in a non-breaking change manner (although before the 1.0.0 release of the test utils, larger changes should be ok)
  2. Manual dictionary/array property assertions should be refactored to use the JSON comparison tool
  3. Test utilities that exist in the test utils repo should be used over equivalent test util classes defined locally

Additional implementation details or code snippet(s)

No response

@timkimadobe timkimadobe added the task Item that can be completed as a standalone code change or is part of an epic.Used by issue template label Oct 26, 2023
@emdobrin emdobrin linked a pull request Jun 20, 2024 that will close this issue
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
task Item that can be completed as a standalone code change or is part of an epic.Used by issue template
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants