JPlagOptions alternative constructors should be removed

[brodmo]

At the moment, the JPlagOptions record has about 15 alternative constructors of the following form:

public JPlagOptions withLanguageOption(Language language) {
    return new JPlagOptions(language, minimumTokenMatch, submissionDirectories, oldSubmissionDirectories, baseCodeSubmissionDirectory,
            subdirectoryName, fileSuffixes, exclusionFileName, similarityMetric, similarityThreshold, maximumNumberOfComparisons,
            clusteringOptions, debugParser);
}

These constructors are obviously horrible in terms of maintainability. They are currently used for testing. They should be removed and some other way to construct options for the tests introduced (e.g. a builder). TestBase.java, which is not in good shape at the moment either, would also be simplified as a result.

[brodmo]

Somewhat relatedly, the readability of the JPlagOptions field definitions could be greatly improved by utilizing this spotless feature:

public record JPlagOptions(
        // spotless:off
        Language language,
        Integer minimumTokenMatch,
        Set<File> submissionDirectories,
        Set<File> oldSubmissionDirectories,
        File baseCodeSubmissionDirectory,
        String subdirectoryName,
        List<String> fileSuffixes,
        String exclusionFileName,
        SimilarityMetric similarityMetric,
        double similarityThreshold,
        int maximumNumberOfComparisons,
        ClusteringOptions clusteringOptions,
        boolean debugParser
        // spotless:on
) {

rather than

public record JPlagOptions(Language language, Integer minimumTokenMatch, Set<File> submissionDirectories, Set<File> oldSubmissionDirectories,
        File baseCodeSubmissionDirectory, String subdirectoryName, List<String> fileSuffixes, String exclusionFileName,
        SimilarityMetric similarityMetric, double similarityThreshold, int maximumNumberOfComparisons, ClusteringOptions clusteringOptions,
        boolean debugParser) {

[brodmo]

It's also possible to set eclipse's join_wrapped_lines setting to false, which would make spotless generally much more lenient in such cases

[dfuchss]

Simply use // if you want to force line breaks :)