Functional test AEPTestUtils update #159
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Replace local test utils with AEPTestUtils * Update ConfigOverridesFunctionalTests.kt to use AEPTestUtils * Remove local test-utils files * Update CompletionHandlerFunctionalTests.java to use AEPTestUtils * Update ConsentStatusChangeFunctionalTests.java to use AEPTestUtils Apply lint formatting * Update ConfigOverridesFunctionalTests.kt to use static assertExactMatch * Update AEPTestUtils to the latest version * Remove unconverted functional tests * Revert "Remove unconverted functional tests" This reverts commit 2d0013a. * Update AEPTestUtils to latest version * Update CompletionHandlerFunctionalTests.java to use TestableNetworkRequest * Update ConsentStatusChangeFunctionalTests.java to use TestableNetworkRequest type Update list get operations to safe versions * Update ConsentStatusChangeFunctionalTests.java to use TestableNetworkRequest * ConsentStatusChangeFunctionalTests.java add collection size check before access * Remove Long conversion for result list size check in ConfigOverridesFunctionalTests.kt
* Update EdgeFunctionalTests.java to use AEPTestUtils * Apply lint formatting to EdgeFunctionalTests.java * Update EdgePathOverwriteTests.java to use JSON comparison APIs * Apply lint formatting for EdgePathOverwriteTests.java * Update EdgeFunctionalTests.java to use TestableNetworkRequest Replace local getPayloadJson method and usages * Update EdgePathOverwriteTests.java to use TestableNetworkRequest Replace usage of local getPayloadJson * Update testSendEvent_withXDMDataAndNullData_sendsCorrectRequestEvent to update testValue for clarity * Apply lint formatting for EdgeFunctionalTests.java and EdgePathOverwriteTests.java * Update to use stringValue for test case strings in event payloads * Update EdgeFunctionalTests and TestXDMSchema to use "test" prefixed property names Add code comments for ElementCount cases to make assertion logic clearer * Remove test case comments for JSON assertions with actual expected payloads
* Update IdentityStateFunctionalTests.java to use JSON comparison APIs * Apply lint formatting to IdentityStateFunctionalTests.java * Update NetworkResponseHandlerFunctionalTests.java to use JSON comparison APIs * Apply lint formatting for NetworkResponseHandlerFunctionalTests.java * Update NoConfigFunctionalTests.java to use JSON comparison APIs * Apply lint formatting for NoConfigFunctionalTests.java * Update RestartFunctionalTests.java to use AEPTestUtils * Apply lint formatting to RestartFunctionalTests.java * Update SampleFunctionalTests.java to use JSON comparison APIs * Apply lint formatting for SampleFunctionalTests.java * Update converted test classes to use TestableNetworkRequest and remove local getJsonPayload method * Add missing step to verify the request is not sent because the configuration state is pending and not because the identity state is not set * Functional test AEPTestUtils update (part 2) (#156) * Update EdgeFunctionalTests.java to use AEPTestUtils * Apply lint formatting to EdgeFunctionalTests.java * Update EdgePathOverwriteTests.java to use JSON comparison APIs * Apply lint formatting for EdgePathOverwriteTests.java * Update EdgeFunctionalTests.java to use TestableNetworkRequest Replace local getPayloadJson method and usages * Update EdgePathOverwriteTests.java to use TestableNetworkRequest Replace usage of local getPayloadJson * Update testSendEvent_withXDMDataAndNullData_sendsCorrectRequestEvent to update testValue for clarity * Apply lint formatting for EdgeFunctionalTests.java and EdgePathOverwriteTests.java * Update to use stringValue for test case strings in event payloads * Update EdgeFunctionalTests and TestXDMSchema to use "test" prefixed property names Add code comments for ElementCount cases to make assertion logic clearer * Remove test case comments for JSON assertions with actual expected payloads * Update AEPTestUtils to latest version * Update test case setup for testHandleExperienceEventRequest_withPendingConfigurationState_expectEventsQueueIsBlocked to set an identity state beforehand * Apply lint formatting for NoConfigFunctionalTests.java * Add test case comments for clarity * RestartFunctionalTests.java - update tearDown to reset all test helpers
# Conflicts: # code/upstream-integration-tests/build.gradle.kts
cacheung
approved these changes
Jun 26, 2024
10 tasks
10 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR brings in the functional test AEPTestUtils updates from the feature branch
feature/functional-test-aeptestutilswhich incrementally converted the functional tests for the Android Edge extension:#155
#156
#157
Related Issue
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: