add test for UK correction

Author: jxmorris12

language_tool_python/server.py

@@ -13,7 +13,7 @@
 from .utils import *
 
 
-DEBUG_MODE = False
+DEBUG_MODE = True
 
 # Keep track of running server PIDs in a global list. This way,
 # we can ensure they're killed on exit.
@@ -95,7 +95,7 @@ def _spell_checking_categories(self):
     def check(self, text: str) -> [Match]:
         """Match text against enabled rules."""
         url = urllib.parse.urljoin(self._url, 'check')
-        response = self._get_root(url, self._encode(text))
+        response = self._query_server(url, self._encode(text))
         matches = response['matches']
         return [Match(match) for match in matches]
 
@@ -142,7 +142,7 @@ def _get_languages(self) -> set:
         self._start_server_if_needed()
         url = urllib.parse.urljoin(self._url, 'languages')
         languages = set()
-        for e in self._get_root(url, num_tries=1):
+        for e in self._query_server(url, num_tries=1):
             languages.add(e.get('code'))
             languages.add(e.get('longCode'))
         return languages
@@ -156,9 +156,9 @@ def _update_remote_server_config(self, url):
         self._url = url
         self._remote = True
 
-    def _get_root(self, url, data=None, num_tries=2):
+    def _query_server(self, url, data=None, num_tries=2):
         if DEBUG_MODE:
-            print('_get_root url:', url, 'data:', data)
+            print('_query_server url:', url, 'data:', data)
         for n in range(num_tries):
             try:
                 with urlopen(url, data, self._TIMEOUT) as f:

tests/test_major_functionality.py

@@ -3,7 +3,6 @@ def test_langtool_load():
 	import language_tool_python
 	lang_tool = language_tool_python.LanguageTool("en-US")
 	matches = lang_tool.check('ain\'t nothin but a thang')
-	open('test_langtool_load', 'w').write(str(matches))
 	assert str(matches) == """[Match({'ruleId': 'UPPERCASE_SENTENCE_START', 'message': 'This sentence does not start with an uppercase letter.', 'replacements': ['Ain'], 'offsetInContext': 0, 'context': "ain't nothin but a thang", 'offset': 0, 'errorLength': 3, 'category': 'CASING', 'ruleIssueType': 'typographical', 'sentence': "ain't nothin but a thang"}), Match({'ruleId': 'MORFOLOGIK_RULE_EN_US', 'message': 'Possible spelling mistake found.', 'replacements': ['nothing', 'no thin'], 'offsetInContext': 6, 'context': "ain't nothin but a thang", 'offset': 6, 'errorLength': 6, 'category': 'TYPOS', 'ruleIssueType': 'misspelling', 'sentence': "ain't nothin but a thang"}), Match({'ruleId': 'MORFOLOGIK_RULE_EN_US', 'message': 'Possible spelling mistake found.', 'replacements': ['than', 'thing', 'hang', 'thank', 'Chang', 'tang', 'thong', 'twang', 'Thant', 'thane', 'Thanh', 't hang', 'Shang', 'Zhang'], 'offsetInContext': 19, 'context': "ain't nothin but a thang", 'offset': 19, 'errorLength': 5, 'category': 'TYPOS', 'ruleIssueType': 'misspelling', 'sentence': "ain't nothin but a thang"})]"""
 
 def test_langtool_languages():
@@ -17,9 +16,19 @@ def test_match():
 	text = u'A sentence with a error in the Hitchhiker’s Guide tot he Galaxy'
 	matches = tool.check(text)
 	assert len(matches) == 2
-	open('test_match', 'w').write(str(matches[0]))
 	assert str(matches[0]) == 'Offset 16, length 1, Rule ID: EN_A_VS_AN\nMessage: Use “an” instead of ‘a’ if the following word starts with a vowel sound, e.g. ‘an article’, ‘an hour’\nSuggestion: an\nA sentence with a error in the Hitchhiker’s Guide tot he ...\n                ^'
 
+def test_uk_typo():
+	import language_tool_python
+	language = language_tool_python.LanguageTool("en-UK")
+
+	sentence1 = "If you think this sentence is fine then, your wrong."
+	results1 = language.check(sentence1)
+	assert len(results1) == 1
+	assert language_tool_python.utils.correct(sentence1, results1) == "If you think this sentence is fine then, you're wrong."
+
+	results2 = language.check("You're mum is called Emily, is that right?")
+	assert len(results2) == 0
 
 def test_remote_es():
 	import language_tool_python

tests/test_remote.bash

@@ -4,7 +4,7 @@ set -eux
 
 readonly port=8081
 readonly ltp_path=$(printf "import language_tool_python\nprint(language_tool_python.utils.get_language_tool_download_path())\n" | python)
-readonly jar="${ltp_path}LanguageTool-*/languagetool-server.jar"
+readonly jar="${ltp_path}/LanguageTool-*/languagetool-server.jar"
 
 java -cp $jar org.languagetool.server.HTTPServer --port "$port" &
 java_pid=$!