Minor update to msvc exceptions and rework new unit tests.

2 files changed, 13 insertions, 7 deletions

diff --git a/SCons/Tool/MSCommon/MSVC/Exceptions.py b/SCons/Tool/MSCommon/MSVC/Exceptions.py
index a12b3c6..7b24a2b 100644
--- a/SCons/Tool/MSCommon/MSVC/Exceptions.py
+++ b/SCons/Tool/MSCommon/MSVC/Exceptions.py
@@ -25,27 +25,32 @@
 Exceptions for Microsoft Visual C/C++.
 """
 
+# reminder: add exceptions to MSCommon if necessary
+
 class VisualCException(Exception):
     pass
 
 class MSVCInternalError(VisualCException):
     pass
 
+class MSVCUserError(VisualCException):
+    pass
+
 class MSVCScriptExecutionError(VisualCException):
     pass
 
-class MSVCVersionNotFound(VisualCException):
+class MSVCVersionNotFound(MSVCUserError):
     pass
 
-class MSVCSDKVersionNotFound(VisualCException):
+class MSVCSDKVersionNotFound(MSVCUserError):
     pass
 
-class MSVCToolsetVersionNotFound(VisualCException):
+class MSVCToolsetVersionNotFound(MSVCUserError):
     pass
 
-class MSVCSpectreLibsNotFound(VisualCException):
+class MSVCSpectreLibsNotFound(MSVCUserError):
     pass
 
-class MSVCArgumentError(VisualCException):
+class MSVCArgumentError(MSVCUserError):
     pass
 
diff --git a/SCons/Tool/MSCommon/MSVC/Policy.py b/SCons/Tool/MSCommon/MSVC/Policy.py
index 9b7025e..fe8da31 100644
--- a/SCons/Tool/MSCommon/MSVC/Policy.py
+++ b/SCons/Tool/MSCommon/MSVC/Policy.py
@@ -45,6 +45,7 @@ from ..common import (
 )
 
 from .Exceptions import (
+    MSVCArgumentError,
     MSVCVersionNotFound,
     MSVCScriptExecutionError,
 )
@@ -141,7 +142,7 @@ def _msvc_notfound_policy_lookup(symbol):
                      repr(symbol),
                      ', '.join([repr(s) for s in MSVC_NOTFOUND_POLICY_EXTERNAL.keys()])
                   )
-        raise ValueError(err_msg)
+        raise MSVCArgumentError(err_msg)
 
     return notfound_policy_def
 
@@ -225,7 +226,7 @@ def _msvc_scripterror_policy_lookup(symbol):
                      repr(symbol),
                      ', '.join([repr(s) for s in MSVC_SCRIPTERROR_POLICY_EXTERNAL.keys()])
                   )
-        raise ValueError(err_msg)
+        raise MSVCArgumentError(err_msg)
 
     return scripterror_policy_def