Add a system of error report to not block the full parsing in case I …

…make a mistake at some point.

Fixes #18

src/Famix-Python-Importer/FamixPythonImporter.class.st

@@ -12,7 +12,8 @@ Class {
 	#name : 'FamixPythonImporter',
 	#superclass : 'Object',
 	#instVars : [
-		'visitor'
+		'visitor',
+		'errorReport'
 	],
 	#category : 'Famix-Python-Importer-Importer',
 	#package : 'Famix-Python-Importer',
@@ -26,6 +27,12 @@ FamixPythonImporter class >> import: aFileName [
 		import: aFileName
 ]
 
+{ #category : 'accessing' }
+FamixPythonImporter >> errorReport [
+
+	^ visitor errorReport
+]
+
 { #category : 'api' }
 FamixPythonImporter >> import: aFileName [
 
@@ -42,6 +49,9 @@ FamixPythonImporter >> import: aFileName [
 	self importFileReference: fileReference.
 
 	visitor solver resolveUnresolvedReferences.
+
+	self flag: #todo. "We should probably handle it another way in production."
+	self errorReport ifNotEmpty: [ self errorReport inspect ].
 
 	^ visitor model
 ]

src/Famix-Python-Importer/FamixPythonImporterVisitor.class.st

@@ -12,6 +12,19 @@ Class {
 	#tag : 'Visitors'
 }
 
+{ #category : 'visiting' }
+FamixPythonImporterVisitor >> acceptNode: aNode [
+
+	^ [ super acceptNode: aNode ]
+		  on: Exception
+		  do: [ :exception |
+			  (exception isKindOf: Warning)
+				  ifTrue: [
+					  self errorReport addWarning: exception.
+					  exception resume ]
+				  ifFalse: [ self errorReport addError: exception ] ]
+]
+
 { #category : 'visiting' }
 FamixPythonImporterVisitor >> acceptNodes: aCollection [
 	"override to return the result of the visits"