JBehave
  1. JBehave
  2. JBEHAVE-725

Some unit tests fail on Windows due to cr/lf encoding

    Details

    • Type: Bug Bug
    • Status: Resolved Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 3.5.4
    • Fix Version/s: 3.6
    • Component/s: Core
    • Labels:
      None
    • Environment:
      windows vista, java 1.6.0-31
    • Number of attachments :
      0

      Description

      When building jbehave-core on windows, 2 unit tests fail because the files are compared with cr/lf line endings (this may depend on the git default setting for text files) but the actual test results contain lf line endings only (since the text is never written or read).

      I ignored that until now with skipTests=true, I assume nobody else uses Windows to build jbehave, if you want, I can fix the tests.

        Activity

        Hide
        Alexander Lehmann added a comment -

        the test are shouldReportEventsToHtmlOutput and shouldReportEventsToXmlOutput

        Show
        Alexander Lehmann added a comment - the test are shouldReportEventsToHtmlOutput and shouldReportEventsToXmlOutput
        Hide
        Mauro Talevi added a comment -

        A patch for Windows would be welcome.

        Show
        Mauro Talevi added a comment - A patch for Windows would be welcome.
        Hide
        Alexander Lehmann added a comment -

        Turns out this is handled for the created file but not for the reference file, if the files are checked out with cr/lf conversion e.g. on tortoisegit, the tests fail

        Show
        Alexander Lehmann added a comment - Turns out this is handled for the created file but not for the reference file, if the files are checked out with cr/lf conversion e.g. on tortoisegit, the tests fail
        Show
        Alexander Lehmann added a comment - https://github.com/alexlehm/jbehave-core/commit/c1bc479f81e4c0440ed1089963334f6c659e87de

          People

          • Assignee:
            Unassigned
            Reporter:
            Alexander Lehmann
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: