You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@opennlp.apache.org by "Sujithra Rajan (Jira)" <ji...@apache.org> on 2023/11/21 19:05:00 UTC
[jira] [Updated] (OPENNLP-1519) Modify Tests in DictionaryAsSetCaseInsensitiveTest + DictionaryAsSetCaseSensitiveTest to handle any Iteration order
[ https://issues.apache.org/jira/browse/OPENNLP-1519?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Sujithra Rajan updated OPENNLP-1519:
------------------------------------
Summary: Modify Tests in DictionaryAsSetCaseInsensitiveTest + DictionaryAsSetCaseSensitiveTest to handle any Iteration order (was: Use LinkedHashSet for deterministic iteration order)
> Modify Tests in DictionaryAsSetCaseInsensitiveTest + DictionaryAsSetCaseSensitiveTest to handle any Iteration order
> -------------------------------------------------------------------------------------------------------------------
>
> Key: OPENNLP-1519
> URL: https://issues.apache.org/jira/browse/OPENNLP-1519
> Project: OpenNLP
> Issue Type: Improvement
> Reporter: Sujithra Rajan
> Priority: Minor
>
> Two tests
> * `opennlp.tools.dictionary.DictionaryAsSetCaseInsensitiveTest.testEquals`
> * `opennlp.tools.dictionary.DictionaryAsSetCaseInsensitiveTest.testEqualsDifferentCase`
> uses HashSet for entrySet while initializing the 'Dictionary' and thus the order is not constant all the time.
> This was found by using the [NonDex][https://github.com/TestingResearchIllinois/NonDex] tool.
> Encountered the following error messages:
> {quote}org.opentest4j.AssertionFailedError: expected: <[1a, 1b]> but was: <[1b, 1a]>
> at opennlp.tools.dictionary.DictionaryAsSetCaseInsensitiveTest.testEquals(DictionaryAsSetCaseInsensitiveTest.java:121)
> {quote}
> {quote}org.opentest4j.AssertionFailedError: expected: <[1a, 1b]> but was: <[1B, 1A]>
> at opennlp.tools.dictionary.DictionaryAsSetCaseInsensitiveTest.testEqualsDifferentCase(DictionaryAsSetCaseInsensitiveTest.java:142)
> {quote}
> {quote}org.opentest4j.AssertionFailedError: expected: <[[Berlin], [Stockholm], [New,York], [London], [Copenhagen], [Paris]]> but was: <[[Copenhagen], [London], [New,York], [Stockholm], [Paris], [Berlin]]>
> at opennlp.uima.dictionary.DictionaryResourceTest.testDictionaryWasLoaded(DictionaryResourceTest.java:76)
> {quote}
>
> The fix is to change HashSet to LinkedHashSet so that the iteration order remains stable all the time.
> Assertion statement of 'testDictionaryWasLoaded' was modified to match the exact ordering of dictionary.dic.
>
> {*}REPRODUCE{*}:
> ```
> mvn edu.illinois:nondex-maven-plugin:2.1.7-SNAPSHOT:nondex -Dtest=opennlp.tools.dictionary.DictionaryAsSetCaseInsensitiveTest#testEquals
> ```
> Can I proceed and create PR ?
--
This message was sent by Atlassian Jira
(v8.20.10#820010)