• Maksim Moskvitin's avatar
    Reland "Add test for keystore key rotation" · c8f79f6a
    Maksim Moskvitin authored
    This is a reland of b9c8d354
    
    CL was reverted because of test failures on many builders. The reason
    for this failures is ModifyEntitySpecifics() being wrapped in DCHECK(),
    which makes new test extremely flaky. Solution: unwrap it.
    
    Original change's description:
    > Add test for keystore key rotation
    >
    > New test mimics server-side behavior for triggering keystore key
    > rotation: generation of new keystore key with touching the Nigori
    > node. After that client should receive updated keystore keys and the
    > same Nigori node as it already has (modulo metadata update). The client
    > should commit the new Nigori node with rotated keystore key and this
    > verified by checking |encryption_keybag| encryption key name.
    >
    > Bug: 922900
    > Change-Id: I91b86bbb0f68a55fdcc10049fa90afa70dc32155
    > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1887611
    > Reviewed-by: Mikel Astiz <mastiz@chromium.org>
    > Commit-Queue: Maksim Moskvitin <mmoskvitin@google.com>
    > Auto-Submit: Maksim Moskvitin <mmoskvitin@google.com>
    > Cr-Commit-Position: refs/heads/master@{#712502}
    
    Bug: 922900
    Change-Id: I994af8befefb8207cedd37e53022329258455221
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1899497
    Commit-Queue: Maksim Moskvitin <mmoskvitin@google.com>
    Commit-Queue: Mikel Astiz <mastiz@chromium.org>
    Reviewed-by: default avatarMikel Astiz <mastiz@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#712683}
    c8f79f6a
encryption_helper.cc 9.19 KB