Skip to content

Bugfixes on tests and github actions runner for Windows #10196

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 15 commits into
base: master
Choose a base branch
from
Open
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fixed EditorConsoleTest on Windows
For some reason the consolePane.getText() method returns the document
text with single CR translated into OS native CR+LF. This will influence
test outcome since we check various combinations of CR and LF.
  • Loading branch information
cmaglie committed Jun 19, 2020
commit ea25c35ddb5243bde35e89dcf59b8799f4d857c0
8 changes: 7 additions & 1 deletion app/src/processing/app/EditorConsole.java
Original file line number Diff line number Diff line change
Expand Up @@ -231,7 +231,13 @@ public void insertString(String str, SimpleAttributeSet attributes) throws BadLo
}

public String getText() {
return consoleTextPane.getText();
try {
return document.getText(0, document.getLength());
} catch (BadLocationException e) {
// Should never happen...
e.printStackTrace();
return "";
}
}

}
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy