List型を引数に取るコンストラクタを利用した際にlistの操作がPatternsに影響する

[lambig]

改修について

現状

Pattensはlistを受け取った際にそのlistオブジェクトをそのままインスタンスに保持する。
そのため、受け取ったlistがmutableな場合には以下の操作で定義されたPatternsの挙動が改変できてしまう。

破壊例

            var list = new ArrayList<>(
                    List.<Tuple2<Predicate<Integer>, Function<Integer, String>>>of(
                            when(equalsTo(3), then("b")), // このパターンが消されてしまうので
                            when(equalsTo(4), thenSupply(() -> "c")),
                            when(i -> i > 0, thenApply(Object::toString)), // 3はこちらで処理されるようになる
                            when(i -> i < 0, thenApply(sequenceOf((UnaryOperator<Integer>) i -> i + 1, Number::longValue, Object::toString)))));
            Patterns<Integer, String> target = Patterns.of(list);
            list.remove(0);

対応

これを防ぐため、Patternsは受け取ったリストのコピーを保持するように改修する必要がある。

単体テスト

実際にPatternsを破壊するテストケースが実装されているので、
そのケースによってPatternsが破壊されなくなることを確認したらそのケース名のみを変更すればよい。