8329089: Empty immutable list throws the wrong exception type for rem…

…ove(first | last) operations

Reviewed-by: rriggs

src/java.base/share/classes/java/util/ImmutableCollections.java

@@ -258,6 +258,8 @@ abstract static class AbstractImmutableList<E> extends AbstractImmutableCollecti
         @Override public void    add(int index, E element) { throw uoe(); }
         @Override public boolean addAll(int index, Collection<? extends E> c) { throw uoe(); }
         @Override public E       remove(int index) { throw uoe(); }
+        @Override public E       removeFirst() { throw uoe(); }
+        @Override public E       removeLast() { throw uoe(); }
         @Override public void    replaceAll(UnaryOperator<E> operator) { throw uoe(); }
         @Override public E       set(int index, E element) { throw uoe(); }
         @Override public void    sort(Comparator<? super E> c) { throw uoe(); }

test/jdk/java/util/Collection/MOAT.java

@@ -246,6 +246,7 @@ public static void realMain(String[] args) {
             testCollection(list);
             testImmutableList(list);
             testListMutatorsAlwaysThrow(list);
+            testImmutableListMutatorsAlwaysThrow(list);
             if (list.size() >= 1) {
                 // test subLists
                 List<Integer> headList = list.subList(0, list.size() - 1);
@@ -564,6 +565,12 @@ private static void testListMutatorsAlwaysThrow(List<Integer> c) {
                 () -> c.addAll(0, Collections.emptyList()));
     }
 
+    private static void testImmutableListMutatorsAlwaysThrow(List<Integer> c) {
+        THROWS(UnsupportedOperationException.class,
+                c::removeFirst,
+                c::removeLast);
+    }
+
     /**
      * As above, for an empty list.
      *