Changed error message

vaniacer · Aug 20, 2018 · bd80332 · bd80332
1 parent e85ae16
commit bd80332
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/tests/test_app.py b/tests/test_app.py
@@ -64,11 +64,11 @@ def test_app_with_invalid_form(self):
         self.assertEqual(response.code, 200)
         body = 'hostname=&port=&username=&password'
         response = self.fetch('/', method="POST", body=body)
-        self.assertIn(b'"status": "Empty hostname"', response.body)
+        self.assertIn(b'"status": "The hostname field is required"', response.body) # noqa
 
         body = 'hostname=127.0.0.1&port=&username=&password'
         response = self.fetch('/', method="POST", body=body)
-        self.assertIn(b'"status": "Empty port"', response.body)
+        self.assertIn(b'"status": "The port field is required"', response.body)
 
         body = 'hostname=127.0.0.1&port=port&username=&password'
         response = self.fetch('/', method="POST", body=body)
@@ -80,7 +80,7 @@ def test_app_with_invalid_form(self):
 
         body = 'hostname=127.0.0.1&port=7000&username=&password'
         response = self.fetch('/', method="POST", body=body)
-        self.assertIn(b'"status": "Empty username"', response.body)
+        self.assertIn(b'"status": "The username field is required"', response.body) # noqa
 
     def test_app_with_wrong_credentials(self):
         response = self.fetch('/')

diff --git a/webssh/handler.py b/webssh/handler.py
@@ -113,7 +113,7 @@ def get_port(self):
     def get_value(self, name):
         value = self.get_argument(name)
         if not value:
-            raise ValueError('Empty {}'.format(name))
+            raise ValueError('The {} field is required'.format(name))
         return value
 
     def get_args(self):