Integrated: 1633: PullRequestPollerTests#queryPaddingComment fails

[Erik Joelsson]

On Tue, 11 Oct 2022 20:12:10 GMT, Erik Joelsson <erikj at openjdk.org> wrote:

> A combination of changes in SKARA-1584 and SKARA-1606 is causing a test failure. The problem is in `TestPullRequest::review`, where we will sometimes (most often) return the internal ArrayList<Review> from the underlying store object. This is causing TestPullRequest::snapshot to not return a pure snapshot, but instead something that gets updated when the store is updated, defeating the whole idea of the snapshot.
> 
> The fix is to wrap the return value with a `List::copyOf`. I'm also applying this to `Issue::comments` as that's also part of the snapshot and using unmodifiable collections is usually preferable.

This pull request has now been integrated.

Changeset: 016c78ff
Author:    Erik Joelsson <erikj at openjdk.org>
URL:       https://git.openjdk.org/skara/commit/016c78ff64191817f57d780f5ab85240b7bd1e83
Stats:     2 lines in 2 files changed: 0 ins; 0 del; 2 mod

1633: PullRequestPollerTests#queryPaddingComment fails

Reviewed-by: ehelin

-------------

PR: https://git.openjdk.org/skara/pull/1394