fix: package-type fixer to avoid duplicated empty section (#2578)

* fix: packagetype fixer avoid dupl. empty section

close: #2577

* test: add test for the fixer fix

* doc: add news

* Update src/pdm/cli/commands/fix/fixers.py

* Update tests/cli/test_install.py

* fix: test project missing requires-python

* address ming's review comment

* Update tests/cli/test_install.py

---------

Co-authored-by: Frost Ming <[email protected]>

news/2578.bugfix.md

@@ -0,0 +1 @@
+fix the package-type fixer won't update toml properly for "Nested Section Ordering Issue in TOML".

src/pdm/cli/commands/fix/fixers.py

@@ -83,7 +83,17 @@ def check(self) -> bool:
         return "package-type" in self.project.pyproject.settings
 
     def fix(self) -> None:
-        package_type = self.project.pyproject.settings.pop("package-type")
-        dist = bool(package_type == "library")
-        self.project.pyproject.settings["distribution"] = dist
+        # Copy the project settings
+        settings = self.project.pyproject.settings.copy()
+
+        # Pop the package type and convert it to a distribution type
+        package_type = settings.pop("package-type")
+        dist = package_type == "library"
+        settings["distribution"] = dist
+
+        # Update the project settings with the new distribution type
+        self.project.pyproject._data["tool"].pop("pdm")
+        self.project.pyproject.settings.update(settings)
+
+        # Write the updated settings back to the project
         self.project.pyproject.write(False)

tests/cli/test_install.py

@@ -305,3 +305,10 @@ def test_install_requirement_with_extras(project, pdm, working_set):
     pdm(["lock", "-Gsocks"], obj=project, strict=True)
     pdm(["sync", "-Gsocks"], obj=project, strict=True)
     assert "pysocks" in working_set
+
+
+def test_fix_package_type_and_update(fixture_project, pdm, working_set):
+    project = fixture_project("test-package-type-fixer")
+    pdm(["fix", "package-type"], obj=project, strict=True)
+    pdm(["update"], obj=project, strict=True)
+    assert "test-package-type-fixer" not in working_set

tests/fixtures/projects/test-package-type-fixer/pyproject.toml

@@ -0,0 +1,23 @@
+[build-system]
+requires = ["pdm-backend"]
+build-backend = "pdm.backend"
+
+[project]
+
+version = "0.0.1"
+dependencies = []
+name = "test-package-type-fixer"
+requires-python = ">=3.8"
+
+[tool.pdm.version]
+source = "file"
+path = "src/test_package_type_fixer/__init__.py"
+
+[tool.pdm]
+package-type = "application"
+
+[tool.pdm.dev-dependencies]
+
+dev = [
+    "requests==2.19.1"
+]

tests/fixtures/projects/test-package-type-fixer/src/test_package_type_fixer/__init__.py

@@ -0,0 +1 @@
+__verson__ = '0.1.0'