tree cf84d0feca836d11d7c3512f30e5512a50d62ed5
parent 17cf2892135018b9a1032d12e294862de75bd86b
author Adam Vartanian <flooey@google.com> 1517847024 +0000
committer Adam Vartanian <flooey@google.com> 1517908611 +0000

Clean up a couple test oddities.

Some internal changes exposed existing problems in our tests.

CipherTest's test_getInstance was needlessly case-sensitive.
Algorithm names are case insensitive, so normalize them before using
them in sets in the test.

KeyGeneratorTest and SecretKeyFactoryTest both had a problem where the
first test was a failing test, which meant that the provider
infrastructure cycled through all the available providers trying to
find one that wouldn't fail and then locked into the least-preferred
provider, so we were testing the wrong thing.  Instead, lock in the
preferred provider (by calling getProvider()) before running any of
the tests, so we know we're testing the provider we intend to test.

Bug: 72860937
Test: cts -m CtsLibcoreTestCases
Change-Id: I98bcc5b34655eec9bdb4423c7739ba5ebd1d6f1d