что не так? Валидатор ругается на последнее требование

Если ты хочешь сделать Nullsafe метод то переверни условия под if jpg.equals(null) вот досюда ты кстати не дойдешь вообще. У null нельзя вызвать методы, ты получишь NPE. Сделай по аналогии

if(ImageTypes.BMP.equals(jpg)){
            return new BmpReader();
        }

Проверь все типи из enum и в else можешь кидать IllegalArgument Кроме того, раз у тебя входной параметр уже enum то можно вообще сравнивать через ==

public static ImageReader getImageReader(ImageTypes type) {
    if (ImageTypes.JPG == type) {
        return new JpgReader();
    } else if (ImageTypes.PNG == type) {
        return new PngReader();
    } else if (ImageTypes.BMP == type) {
        return new BmpReader();
    } else {
        throw new IllegalArgumentException();
    }
}

В 17й Java еще вот так можно :) Но это валидатор не примет:

public static ImageReader getImageReader(ImageTypes type) {
    return switch (type) {
        case JPG -> new JpgReader();
        case PNG -> new PngReader();
        case BMP -> new BmpReader();
        default -> throw new IllegalArgumentException();
    };
}

if (ImageTypes.JPG == type) {
        return new JpgReader();

if (type == ImageTypes.JPG) {
       return new JpgReader();