According to README it download_all should ignore filtered albums.

I missed it in preparing pull requests

src/sync_photos.py

@@ -147,6 +147,8 @@ def sync_photos(config, photos):
     download_all = config_parser.get_photos_all_albums(config=config)
     if download_all:
         for album in photos.albums.keys():
+            if filters["albums"] and album in iter(filters["albums"]):
+                continue
             sync_album(
                 album=photos.albums[album],
                 destination_path=os.path.join(destination_path, album),

tests/test_sync_photos.py

@@ -31,6 +31,8 @@ def setUp(self) -> None:
     def tearDown(self) -> None:
         """Remove temp directory."""
         shutil.rmtree(tests.TEMP_DIR)
+        if os.path.exists(tests.PHOTOS_DIR):
+            shutil.rmtree(tests.PHOTOS_DIR)
 
     @patch(target="keyring.get_password", return_value=data.VALID_PASSWORD)
     @patch(
@@ -65,7 +67,7 @@ def test_sync_photos_original(
     )
     @patch("icloudpy.ICloudPyService")
     @patch("src.read_config")
-    def test_sync_photos_all_albums(
+    def test_sync_photos_all_albums_filtered(
         self, mock_read_config, mock_service, mock_get_username, mock_get_password
     ):
         """Test for successful original photo size download."""
@@ -84,6 +86,35 @@ def test_sync_photos_all_albums(
         album_1_path = os.path.join(
             self.destination_path, config["photos"]["filters"]["albums"][1]
         )
+        self.assertFalse(os.path.isdir(album_0_path))
+        self.assertFalse(os.path.isdir(album_1_path))
+
+    @patch(target="keyring.get_password", return_value=data.VALID_PASSWORD)
+    @patch(
+        target="src.config_parser.get_username", return_value=data.AUTHENTICATED_USER
+    )
+    @patch("icloudpy.ICloudPyService")
+    @patch("src.read_config")
+    def test_sync_photos_all_albums_filtered(
+        self, mock_read_config, mock_service, mock_get_username, mock_get_password
+    ):
+        """Test for successful original photo size download."""
+        mock_service = self.service
+        config = self.config.copy()
+        config["photos"]["destination"] = self.destination_path
+        config["photos"]["all_albums"] = True
+        mock_read_config.return_value = config
+        album_0_path = os.path.join(
+            self.destination_path, config["photos"]["filters"]["albums"][0]
+        )
+        album_1_path = os.path.join(
+            self.destination_path, config["photos"]["filters"]["albums"][1]
+        )
+        config["photos"]["filters"]["albums"] = None
+        # Sync original photos
+        self.assertIsNone(
+            sync_photos.sync_photos(config=config, photos=mock_service.photos)
+        )
         self.assertTrue(os.path.isdir(album_0_path))
         self.assertTrue(os.path.isdir(album_1_path))
         self.assertTrue(len(os.listdir(album_0_path)) > 1)