fix(init): only add readme key if README file exists

Author: ammargit93

src/poetry/layouts/layout.py

@@ -161,11 +161,13 @@ def generate_project_content(self) -> TOMLDocument:
         else:
             project_content.remove("license")
 
-        if os.path.exists(self.basedir/f"README.{self._readme_format}"):
+        readme_path = self.basedir / f"README.{self._readme_format}"
+        if readme_path.exists():
             project_content["readme"] = f"README.{self._readme_format}"
         else:
             project_content.pop("readme", None)
 
+
         if self._python:
             project_content["requires-python"] = self._python
         else:

tests/layouts/test_layout.py

@@ -1,22 +1,18 @@
 from pathlib import Path
-
 import pytest
 
 from poetry.layouts.layout import Layout
 
-from pathlib import Path
-
-from poetry.layouts.layout import Layout
-
 
 class DummyLayout(Layout):
-    def __init__(self, *args, basedir: Path, **kwargs):
-        super().__init__(*args, **kwargs)
-        self._basedir = basedir
+    """Override basedir for testing purposes."""
+    def __init__(self, basedir: Path, **kwargs):
+        super().__init__(**kwargs)
+        self._test_basedir = basedir
 
     @property
     def basedir(self) -> Path:
-        return self._basedir
+        return self._test_basedir
 
 
 def test_readme_key_removed_if_readme_missing(tmp_path: Path) -> None:
@@ -27,13 +23,13 @@ def test_readme_key_removed_if_readme_missing(tmp_path: Path) -> None:
     )
 
     content = layout.generate_project_content()
-    project = content["project"]
-
-    assert "readme" not in project
+    # README.md does not exist, key should be removed
+    assert "readme" not in content["project"]
 
 
 def test_readme_key_present_if_readme_exists(tmp_path: Path) -> None:
-    (tmp_path / "README.md").touch()
+    readme = tmp_path / "README.md"
+    readme.touch()
 
     layout = DummyLayout(
         project="demo_project",
@@ -42,6 +38,5 @@ def test_readme_key_present_if_readme_exists(tmp_path: Path) -> None:
     )
 
     content = layout.generate_project_content()
-    project = content["project"]
-
-    assert project["readme"] == "README.md"
+    # README.md exists, key should be present
+    assert content["project"]["readme"] == "README.md"