Fix/ wrong if in self.directory (#680)

* Fixed a wrong if comparison.

Added setter and getter to ``self.directory``.

Added also a wrapper to `CWD` so it will show a warning when the directory we are changing to does not exist.

* Added test case for mapdl.cwd and mapdl.directory

* Apply suggestions from code review

Co-authored-by: Alex Kaszynski <[email protected]>

* Added example to docstring.

* Grammar fix.

Co-authored-by: Alex Kaszynski <[email protected]>

Author: germa89

ansys/mapdl/core/mapdl.py

@@ -12,6 +12,8 @@
 import weakref
 import warnings
 import pathlib
+from warnings import warn
+from functools import wraps
 
 import numpy as np
 
@@ -2248,6 +2250,14 @@ def directory(self):
         >>> mapdl.directory
         'C:/temp_directory/'
 
+        Setting the directory
+
+        >>> mapdl.directory = 'C:/temp_directory/'
+        None
+
+        In case the directory does not exist or it is not
+        accessible, ``cwd`` (:func:`_MapdlCore.cwd`) will raise
+        a warning.
         """
         # always attempt to cache the path
         try:
@@ -2256,12 +2266,18 @@ def directory(self):
             pass
 
         # os independent path format
-        if self._path is not None:
+        if self._path: # self.inquire might return ''.
             self._path = self._path.replace("\\", "/")
             # new line to fix path issue, see #416
             self._path = repr(self._path)[1:-1]
         return self._path
 
+    @directory.setter
+    @supress_logging
+    def directory(self, path):
+        """Change the directory using ``Mapdl.cwd``"""
+        self.cwd(path)  # this has been wrapped in Mapdl to show a warning if the file does not exist.
+
     @property
     def _lockfile(self):
         """lockfile path"""
@@ -2473,3 +2489,13 @@ def list(self, filename, ext=""):
         path = str(path) + ext
         with open(path) as fid:
             return fid.read()
+
+    @wraps(Commands.cwd)
+    def cwd(self, *args, **kwargs):
+        """Wraps cwd"""
+        returns_ = super().cwd( *args, **kwargs)
+
+        if '*** WARNING ***' in self._response:
+            warn('\n' + self._response)
+
+        return returns_

tests/test_mapdl.py

@@ -873,3 +873,16 @@ def test_path_with_single_quote(mapdl, path_tests):
     with pytest.raises(RuntimeError):
         resp = mapdl.cwd(path_tests.path_with_single_quote)
         assert 'WARNING' not in resp
+
+
+@skip_in_cloud
+def test_cwd_directory(mapdl, tmpdir):
+
+    mapdl.directory = str(tmpdir)
+    assert mapdl.directory == str(tmpdir).replace('\\', '/')
+
+    wrong_path = 'wrong_path'
+    with pytest.warns(Warning) as record:
+        mapdl.directory = wrong_path
+        assert 'The working directory specified' in record.list[-1].message.args[0]
+        assert 'is not a directory on' in record.list[-1].message.args[0]