Skip to content

Fix unit test for Hotfix#1539

Merged
bilal-alsharifi merged 2 commits intohotfix/4.12.1from
Fix-unit-test
Oct 26, 2020
Merged

Fix unit test for Hotfix#1539
bilal-alsharifi merged 2 commits intohotfix/4.12.1from
Fix-unit-test

Conversation

@JulianKast
Copy link
Copy Markdown
Contributor

@JulianKast JulianKast commented Oct 26, 2020

Fixes #1540

This PR is read for review.

Risk

This PR makes no API changes.

Testing Plan

  • I have verified that I have not introduced new warnings in this PR (or explain why below)
  • I have run the unit tests with this PR
  • I have tested this PR against Core and verified behavior (if applicable, if not applicable, explain why below).
  • I have tested Android, Java SE, and Java EE

Summary

ManagerUtilityTests.java and RPCStructTests.java were apart of the hotFix, but were developed during 5.0. The test needed to be modified slightly to work with 4.12.1.

Changelog

  • Use CharacterSet.CID2SET instead of Using CharacterSet.UTF_8 in ManagerUtilityTests.java
  • Use import android.support.test.runner.AndroidJUnit4 instead of import androidx.test.ext.junit.runners.AndroidJUnit4 in ManagerUtilityTests.java and RPCStructTests.java
  • Fix unit test in RPCRequestFactoryTests.java in relation to Lists with initial values of null are not returned properly in RPCs #1473

CLA

…cations need to be made to allow test to work.
@JulianKast JulianKast changed the title Unit test were created during 5.0 development. For the hotfix, modifi… Fix unit test for Hotfix Oct 26, 2020
@bilal-alsharifi bilal-alsharifi merged commit 0cc8537 into hotfix/4.12.1 Oct 26, 2020
@bilal-alsharifi bilal-alsharifi deleted the Fix-unit-test branch October 26, 2020 19:45
@JulianKast JulianKast mentioned this pull request Oct 26, 2020
5 tasks
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.

2 participants