Fixed a bug that only allowed harvesting if not using poetry.

Author: notactuallyfinn

src/hermes_toml/harvest.py

@@ -23,7 +23,7 @@ class TomlHarvestPlugin(HermesHarvestPlugin):
             ("runtimePlatform", "requires-python"), ("author", "authors"),
             ("maintainer", "maintainers"), ("keywords", "keywords")
         ],
-        "tool.poetry": [
+        "poetry": [
             ("name", "name"), ("version", "version"), ("description", "description"),
             ("author", "authors"), ("maintainer", "maintainers"), ("url", "homepage"),
             ("codeRepository", "repository"), ("keywords", "keywords")
@@ -64,8 +64,18 @@ def read_from_toml(cls, file):
         #if more than one table existis raise an error as
         #the information could be overlapping and there should only be one table
         for table, mapping in cls.table_with_mapping.items():
-            table = data.get(table)
+            #choose correct dictionary representing the table
+            if table == "project":
+                table = data.get(table)
+            else:
+                temp = data.get("tool")
+                if temp is None:
+                    continue
+                table = temp.get(table)
+
+            #check if the table exists
             if not table is None:
+                #if the table exists
                 if len(ret_data.keys()) != 0:
                     raise ValueError("Both project and tool.poetry table exist.")
                 #read the data from the table

test/hermes_toml_test/test_harvest.py

@@ -34,7 +34,7 @@ def toml_file(tmp_path_factory):
 
 @pytest.mark.parametrize("in_data, out_data", [
     ({}, {}), ({"project": {"name":"a"}}, {"name": "a"}),
-    ({"tool.poetry": {"name":"a"}}, {"name": "a"}),
+    ({"tool": {"poetry": {"name":"a"}}}, {"name": "a"}),
     ({"project":{"name":"a"}, "a":{"b":"c"}}, {"name":"a"}),
     ({"project":{"name":"a", "requires-python":">3.7"}}, {"name":"a", "runtimePlatform":"Python >3.7"}),
     ({"project":{"authors":{"givenName":"a"}}}, {"author":{"givenName":"a", "@type":"Person"}}),
@@ -49,9 +49,9 @@ def test_read_from_toml(in_data, out_data, toml_file):
     assert TomlHarvestPlugin.read_from_toml(str(toml_file)) == out_data
 
 @pytest.mark.parametrize("in_data", [
-    ({"project": {"authors":"a"}}), ({"tool.poetry": {"authors":"a"}}),
-    ({"project": {"authors":["a"]}}), ({"tool.poetry": {"authors":["a"]}}),
-    ({"project": {"name":"a"}, "tool.poetry": {"name":"a"}})
+    ({"project": {"authors":"a"}}), ({"tool": {"poetry": {"authors":"a"}}}),
+    ({"project": {"authors":["a"]}}), ({"tool": {"poetry": {"authors":["a"]}}}),
+    ({"project": {"name":"a"}, "tool": {"poetry": {"name":"a"}}})
 ])
 def test_read_from_toml_with_error(in_data, toml_file):
     toml.dump(in_data, open(toml_file, "w", encoding="utf8"))