Merge pull request #160 from n8downs/n8downs/fix-type-error-in-project-manager-report

Fix a type error in ProjectManager's report() function.

Author: cmutel

README.rst

@@ -14,3 +14,13 @@ It is part of the `Brightway LCA framework <https://brightway.dev>`_.
 `Online documentation <https://docs.brightway.dev/>`_ is available, and the source code is hosted on `GitHub brightway-lca organization <https://github.com/brightway-lca/brightway2-data>`_.
 
 Note that version 4.0 and higher are only compatible with Brightway 2.5, as described in the `changelog <https://github.com/brightway-lca/brightway2-data/blob/main/CHANGES.md>`__.
+
+
+## Development Setup
+
+```bash
+brightway2-data $ python3 -m venv venv
+brightway2-data $ source ./venv/bin/activate
+brightway2-data $ python3 -m pip install -r requirements-test.txt
+brightway2-data $ pytest
+```

bw2data/project.py

@@ -363,7 +363,7 @@ def get_dir_size(dirpath):
 
             Does not follow symbolic links"""
             return sum(
-                sum(os.path.getsize(root / name) for name in files)
+                sum(os.path.getsize(Path(root) / name) for name in files)
                 for root, dirs, files in os.walk(dirpath)
             )
 

tests/projects.py

@@ -130,7 +130,9 @@ def test_funny_project_names():
 
 @bw2test
 def test_report():
-    assert projects.report
+    projects.set_current("foo")
+    report = projects.report()
+    assert "foo" in [name for (name, _, _) in report]
 
 
 @bw2test