fixed lint issues as well as get_mkdocs_yaml_path

Author: silentfatez

readthedocs/projects/models.py

@@ -972,7 +972,6 @@ def conf_file(self, version=LATEST):
             )
 
         raise ProjectConfigurationError(ProjectConfigurationError.NOT_FOUND)
-    
 
     def yaml_file(self, version=LATEST):
         """Find a mkdocs ``mkdocs.yml`` file in the project checkout."""
@@ -981,8 +980,7 @@ def yaml_file(self, version=LATEST):
             files = self.full_find("mkdocs.yml", version)
         if len(files) == 1:
             return files[0]
-        ## TODO Handle for multiple Files
-
+        # TODO: Handle for multiple Files
         raise MkDocsYAMLParseError(MkDocsYAMLParseError.NOT_FOUND)
 
     def conf_dir(self, version=LATEST):
@@ -991,11 +989,9 @@ def conf_dir(self, version=LATEST):
             return os.path.dirname(conf_file)
 
     def yaml_dir(self, version=LATEST):
-        conf_file = self.conf_file(version)
-        if conf_file:
-            return os.path.dirname(conf_file)
-
-
+        yaml_file = self.yaml_file(version)
+        if yaml_file:
+            return os.path.dirname(yaml_file)
 
     @property
     def has_good_build(self):

readthedocs/rtd_tests/tests/test_doc_builder.py

@@ -150,11 +150,11 @@ def test_project_without_mkdocs_yaml(
         _,
         docs_dir,
     ):
-        """
+        """ 
         Test for a project without ``mkdocs.yaml`` file.
 
         When this happen, the ``get_mkdocs_yaml_path`` raises a
-        ``ProjectConfigurationError`` which is captured by our own code.
+        ``MkDocsYAMLParseError`` which is captured by our own code.
         """
         tmp_dir = tempfile.mkdtemp()
         checkout_path.return_value = tmp_dir