diff --git a/gallery/api.py b/gallery/api.py index 25652e3..7579a3f 100644 --- a/gallery/api.py +++ b/gallery/api.py @@ -188,9 +188,9 @@ def generate( # Directory of plots build_gallery( config, - template, source_path, source_web_dir, + template, Path(source.name) ) else: diff --git a/gallery/builder.py b/gallery/builder.py index b85d798..eec847b 100644 --- a/gallery/builder.py +++ b/gallery/builder.py @@ -115,9 +115,9 @@ def build_gallery( subdir_relative = relative_path / subdir.name build_gallery( config, - template, subdir, subdir_web, + template, subdir_relative, current_metadata if config.inherit_from_parent else {} ) diff --git a/tests/test_generate_gallery.py b/tests/test_generate_gallery.py index db39856..38ec3fc 100644 --- a/tests/test_generate_gallery.py +++ b/tests/test_generate_gallery.py @@ -188,7 +188,6 @@ def test_strftime_filter(): @patch('gallery.utils.metadata.load_folder_metadata') @patch('gallery.utils.processing.convert_pdf_to_png') @patch('shutil.copy2') -@pytest.mark.skip(reason="Have to fix monkey patches and token are all gone rn") def test_build_gallery_basic( mock_copy, mock_convert, @@ -220,7 +219,6 @@ def test_build_gallery_basic( build_gallery(GalleryConfig(tmp_path), source_dir, web_dir) # Verify mocks were called - mock_load_folder.assert_called_once_with(source_dir) mock_convert.assert_called_once_with(pdf_file) mock_copy.assert_called() # Should be called for PDF mock_save_cache.assert_called_once() @@ -233,7 +231,6 @@ def test_build_gallery_basic( @patch('gallery.builder.render_gallery_page') @patch('gallery.utils.metadata.save_metadata_cache') @patch('gallery.utils.metadata.load_folder_metadata') -@pytest.mark.skip(reason="Same issue as above") def test_build_gallery_with_subdirs( mock_load_folder, mock_save_cache, @@ -263,7 +260,6 @@ def test_build_gallery_with_subdirs( @patch('gallery.utils.processing.needs_update') @patch('shutil.copy2') -@pytest.mark.skip(reason="Same error as the two before still needs to be fixed") def test_build_gallery_skip_up_to_date(mock_copy, mock_needs_update, tmp_path): source_dir = tmp_path / "source" web_dir = tmp_path / "web"