SetPythonPath prints more information on error

tminka · tminka · commit fdd4f12b7f0e · 2021-01-03T18:38:31.000Z
diff --git a/src/embed_tests/TestPythonEngineProperties.cs b/src/embed_tests/TestPythonEngineProperties.cs
@@ -1,4 +1,7 @@
 using System;
+using System.IO;
+using System.Linq;
+
 using NUnit.Framework;
 using Python.Runtime;
 
@@ -201,7 +204,12 @@ public void SetPythonPath()
                 }
                 catch (PythonException ex)
                 {
-                    throw new Exception($"Py.Import(\"{moduleName}\") failed before setting PythonEngine.PythonPath.  sys.path={path}", ex);
+                    string[] messages = paths.Where(p => p.Contains("site-packages")).Select(folder =>
+                        (folder != null && Directory.Exists(folder)) ?
+                        $" {folder} contains {string.Join(Path.PathSeparator.ToString(), Directory.EnumerateFileSystemEntries(folder).Select(fullName => Path.GetFileName(fullName)).ToArray())}" :
+                        "").ToArray();
+                    string message = string.Join(" ", messages);
+                    throw new Exception($"Py.Import(\"{moduleName}\") failed before setting PythonEngine.PythonPath.  sys.path={path}{message}", ex);
                 }
             }
 

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,7 @@`
`1`	`1`	`using System;`
	`2`	`+using System.IO;`
	`3`	`+using System.Linq;`
	`4`	`+`
`2`	`5`	`using NUnit.Framework;`
`3`	`6`	`using Python.Runtime;`
`4`	`7`
`@@ -201,7 +204,12 @@ public void SetPythonPath()`
`201`	`204`	`}`
`202`	`205`	`catch (PythonException ex)`
`203`	`206`	`{`
`204`		`- throw new Exception($"Py.Import(\"{moduleName}\") failed before setting PythonEngine.PythonPath. sys.path={path}", ex);`
	`207`	`+ string[] messages = paths.Where(p => p.Contains("site-packages")).Select(folder =>`
	`208`	`+ (folder != null && Directory.Exists(folder)) ?`
	`209`	`+ $" {folder} contains {string.Join(Path.PathSeparator.ToString(), Directory.EnumerateFileSystemEntries(folder).Select(fullName => Path.GetFileName(fullName)).ToArray())}" :`
	`210`	`+ "").ToArray();`
	`211`	`+ string message = string.Join(" ", messages);`
	`212`	`+ throw new Exception($"Py.Import(\"{moduleName}\") failed before setting PythonEngine.PythonPath. sys.path={path}{message}", ex);`
`205`	`213`	`}`
`206`	`214`	`}`
`207`	`215`