public List<Modification> modifications(ConsoleResult result) {
        List<Modification> modifications = new ArrayList<>();
        for (String change : result.output()) {
            if (!StringUtils.isBlank(change)) {
                String description = "";
                try {
                    long revision = revisionFromChange(change);
                    description = p4Client.describe(revision);
                    modifications.add(modificationFromDescription(description, result));
                } catch (P4OutputParseException e) {
                    LOG.error("Error parsing changes for {}", this);
                    LOG.error("---- change ---------");
                    LOG.error(result.replaceSecretInfo(change));
                    LOG.error("---- description ----");
                    LOG.error(result.replaceSecretInfo(description));
                    LOG.error("---------------------");
                } catch (RuntimeException e) {
                    throw (RuntimeException) result.smudgedException(e);
                }
            }
        }
        return modifications;
    }
@Test
    void shouldIgnoreEmptyLinesInChanges() {
        final String output =
                """
                        Change 539921 on 2008/09/24 by abc@SomeRefinery_abc_sa1-sgr-xyz-001 'more work in progress on ABC un'
                        Change 539920 on 2008/09/24 by abc@SomeRefinery_abc_sa1-sgr-xyz-001 'Fixed pipeline for abc-new-sale'
                        Change 539919 on 2008/09/24 by james@SomeRefinery_def_sa1-sgr-xyz-001 'Assignment stage to send ok and'
                        Change 539918 on 2008/09/24 by abc@SomeRefinery_abc_sa1-sgr-xyz-001 'More refactoring and code clean'
                        Change 539917 on 2008/09/24 by thomas@tom_ws_stuff 'added client name for Arza SW'
                        Change 539916 on 2008/09/24 by alex@SA2-COUNT-LAX-001 'sending the context further '
                        Change 539915 on 2008/09/24 by ellen@ellen-box-1 'TT 678 cause:Old code try to'
                        Change 539914 on 2008/09/24 by valerie@ExcelSheets_New 'update sheet comments '

                        Change 539913 on 2008/09/24 by perforce@SOM-NAME-HERE-HOST-1 'apply new version numbers via A'
                        Change 539912 on 2008/09/24 by lance@lance-box-1 'Corrected a typo. '

                        Change 539911 on 2008/09/24 by lance@lance-box-1 'corrected a typo. '
                        Change 539910 on 2008/09/24 by josh@josh_box_1 'Changes to remove hacks I had m'
                        Change 539909 on 2008/09/24 by thomas@tom_ws_stuff 'Added Arza SW to add request '
                        Change 539908 on 2008/09/24 by padma@Padma '1. Fix #2644 : When the FSOi'
                        Change 539907 on 2008/09/24 by berlin@Dans_Box 'Added GetDocumentMetadataForEdi'
                        Change 539906 on 2008/09/24 by lance@lance-box-1 'Added detail aboutPFP. '
                        Change 539904 on 2008/09/24 by lance@lance-box-1 'Added a detail about PFP. '
                        Change 539903 on 2008/09/24 by nitya@docs_box 'Updated for lam, am 20080923'
                        Change 539902 on 2008/09/24 by abc@SomeRefinery_abc_sa1-sgr-xyz-001 'Work in progress '
                        Change 539901 on 2008/09/24 by anil@anil-box-1 'Added all columns of AA_TASK to'
                        """;

        when(p4Client.describe(any(Long.class))).thenReturn("""
                Change 539921 by abc@SomeRefinery_abc_sa1-sgr-xyz-001 on 2008/09/24 12:10:00

                \tmore work in progress on ABC unit test

                Affected files ...
                """);

        List<Modification> modifications = parser.modifications(new ConsoleResult(0, List.of(output.split("\n")), new ArrayList<>(), new ArrayList<>(), new ArrayList<>()));
        assertThat(modifications.size()).isEqualTo(20);
    }