[ https://issues.apache.org/jira/browse/OPENNLP-1592?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17864482#comment-17864482 ]
ASF GitHub Bot commented on OPENNLP-1592: ----------------------------------------- rzo1 commented on PR #154: URL: https://github.com/apache/opennlp-sandbox/pull/154#issuecomment-2219615339 Wondering if checkstyle is active for sandbox? Might be good to have it for future PRs. > Provide tests for MetaSummarizer > -------------------------------- > > Key: OPENNLP-1592 > URL: https://issues.apache.org/jira/browse/OPENNLP-1592 > Project: OpenNLP > Issue Type: Test > Components: Summarizer > Affects Versions: 2.3.3 > Reporter: Martin Wiesner > Assignee: Martin Wiesner > Priority: Major > Fix For: 2.4.0 > > > The summarizer (sandbox) component lacks Tests for MetaSummarizer.java. > Currently, only an orphaned main method exists in the corresponding class. > Therefore, it is much desirable to have a fix set of tests that demonstrate > this class actually works (Note: it doesn't as is). > Aims: > * Provide a minimal set of tests for MetaSummarizer > * Improve the existing code so that summarization works (again) > * Improve the code quality > * Add JavaDoc where possible. -- This message was sent by Atlassian Jira (v8.20.10#820010)