fix error messages

Author: glaukiol1

main

@@ -64,22 +64,22 @@ func getCwd(self py.Object, args py.Tuple) (py.Object, error) {
 
 // change current working directory
 func chdir(self py.Object, args py.Tuple) (py.Object, error) {
-	if len(args) == 0 || len(args) > 1 {
-		return nil, py.ExceptionNewf(py.TypeError, "One argument required")
+	if len(args) == 0 {
+		return nil, py.ExceptionNewf(py.TypeError, "Missing required argument 'path' (pos 1)")
 	}
 	if objectIsString(args[0]) {
 		dir, err := py.ReprAsString(args[0])
 		if err != nil {
-			return nil, py.ExceptionNewf(py.TypeError, "Failed to parse string")
+			return nil, py.ExceptionNewf(py.TypeError, "Failed to parse string") // never going to run
 		}
 		dir = strings.ReplaceAll(dir, "'", "")
 		err = os.Chdir(dir)
 		if err != nil {
-			return nil, py.ExceptionNewf(py.OSError, "Couldn't change cwd; "+err.Error())
+			return nil, py.ExceptionNewf(py.NotADirectoryError, "Couldn't change cwd; "+err.Error())
 		}
 		return py.None, nil
 	}
-	return nil, py.ExceptionNewf(py.TypeError, "Expected string argument at position 0")
+	return nil, py.ExceptionNewf(py.TypeError, "str expected, not "+args[0].Type().Name)
 }
 
 // get a enviroment variable by key
@@ -89,25 +89,27 @@ func getenv(self py.Object, args py.Tuple) (py.Object, error) {
 	var default_ py.Object // return when not found
 	var err error          // error obj
 
-	if len(args) > 2 {
-		return nil, py.ExceptionNewf(py.KeyError, "1 argument required, \"key\"")
-	}
 	if len(args) == 1 {
 		if objectIsString(args[0]) {
 			key = args[0]
 		} else {
-			return nil, py.ExceptionNewf(py.TypeError, "Expected argument of type string")
+			return nil, py.ExceptionNewf(py.TypeError, "str expected (pos 1), not "+args[0].Type().Name)
 		}
 		default_ = py.None
 	} else if len(args) == 2 {
+		if objectIsString(args[0]) {
+			key = args[0]
+		} else {
+			return nil, py.ExceptionNewf(py.TypeError, "str expected (pos 1), not "+args[0].Type().Name)
+		}
 		if objectIsString(args[1]) {
 			key = args[0]
 			default_ = args[1]
 		} else {
-			return nil, py.ExceptionNewf(py.TypeError, "Expected argument of type string")
+			return nil, py.ExceptionNewf(py.TypeError, "str expected (pos 2), not"+args[1].Type().Name)
 		}
 	} else {
-		return nil, py.ExceptionNewf(py.TypeError, "Expected argument of type string")
+		return nil, py.ExceptionNewf(py.TypeError, "missing one required argument: 'name:str'")
 	}
 	var res py.Object // hold the result value
 	res, err = getEnvVariables().M__getitem__(key)
@@ -142,7 +144,7 @@ func putenv(self py.Object, args py.Tuple) (py.Object, error) {
 		}
 		return py.None, nil
 	}
-	return nil, py.ExceptionNewf(py.TypeError, "Expected 2 arguments of type string")
+	return nil, py.ExceptionNewf(py.TypeError, "missing required arguments: 'key:str' and 'value:str'")
 }
 
 // Unset (delete) the environment variable named key.
@@ -161,7 +163,10 @@ func unsetenv(self py.Object, args py.Tuple) (py.Object, error) {
 			return py.None, nil
 		}
 	}
-	return nil, py.ExceptionNewf(py.TypeError, "Expected 1 argument of type string")
+	if len(args) == 0 {
+		return nil, py.ExceptionNewf(py.TypeError, "missing one required argument: 'key:str'")
+	}
+	return nil, py.ExceptionNewf(py.TypeError, "str expected, not "+args[0].Type().Name)
 }
 
 // os._exit() immediate program termination; unlike sys.exit(), which raises a SystemExit, this function will termninate the program immediately.
@@ -187,7 +192,7 @@ func _exit(self py.Object, args py.Tuple) (py.Object, error) { // can never retu
 // os.system(command string) this function runs a shell command and directs the output to standard output.
 func system(self py.Object, args py.Tuple) (py.Object, error) {
 	if len(args) == 0 && !(objectIsString(args[0])) {
-		return nil, py.ExceptionNewf(py.TypeError, "Expected 1 or more arguments all of type string")
+		return nil, py.ExceptionNewf(py.TypeError, "missing one required argument: 'command:str'")
 	}
 	var cargs []string
 	_carg, err := py.ReprAsString(args[0])

os/os.module.go

@@ -12,8 +12,10 @@ class bcolors:
     BOLD = '\033[1m'
     UNDERLINE = '\033[4m'
 
+print("--------- Manual Check ---------")
+print("Check if the data below is correct:")
 print("os.error:", os.error) # prints <class 'OSError'>
-print("$HOME:", os.environ.get("HOME")) # prints $HOME variable
+print("os.environ.get('HOME'):", os.environ.get("HOME")) # prints $HOME variable
 print("cwd():", os.getcwd()) # prints current working directory
 print("chdir(os.environ.get(\"HOME\"))):", os.chdir(os.environ.get("HOME"))) # changes the current working directory to $HOME
 print("cwd() after chdir():", os.getcwd()) # should print $HOME dir
@@ -27,6 +29,7 @@ class bcolors:
 print()
 print("--------------")
 print()
+print("Tests:")
 
 # tests
 passed_tests = 0
@@ -114,7 +117,7 @@ def _fail(test_name):
 except TypeError:
     _pass('os.unsetenv("1","2") (more arguments than needed)')    
     passed_tests+=1
-
+    
 if os.getcwd() == os.environ.get("HOME"):
     _pass('os.getcwd() == os.environ.get("HOME")')
     passed_tests+=1
@@ -140,7 +143,7 @@ def _fail(test_name):
     _pass('os.getenv("TEST_VAR") == "TEST_VALUE"')
     passed_tests+=1
 else:
-    _fail('os.getenv("TEST_VAR") == "TEST_VALUE"')
+    _fail('os.getenv("TEST_VAR") == "TEST_VALUE" actual %s' % os.getenv("TEST_VAR"))
     failed+=1
 
 if os.unsetenv("TEST_VAR") == None: