rvesse commented on PR #2121: URL: https://github.com/apache/jena/pull/2121#issuecomment-1840457816
> I'm not familiar with writing java tests, so not sure why the jena-text doesn't pick up the example. However when running a single test from intellij, it passes. The test could also be better named. Could I get some guiding on this? Jena uses JUnit suite classes so I think you need to add an entry for this new test class into https://github.com/apache/jena/blob/main/jena-text/src/test/java/org/apache/jena/query/text/TS_Text.java Jena is historically fairly lazy about test naming so the current name is ok. If the test comes out of a specific issue we generally leave a comment to that effect in the source code e.g. `// GH-2094` or for the test so future developers can understand the context of the test -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
