diff --git a/test/unit/lib/whitehall/asset_manager_storage_test.rb b/test/unit/lib/whitehall/asset_manager_storage_test.rb
index 8d69d80b643..46708fa0dde 100644
--- a/test/unit/lib/whitehall/asset_manager_storage_test.rb
+++ b/test/unit/lib/whitehall/asset_manager_storage_test.rb
@@ -53,17 +53,6 @@ class Whitehall::AssetManagerStorage::FileTest < ActiveSupport::TestCase
     assert_equal "http://assets-host/media/asset_manager_id/sample.docx", model.file.url
   end
 
-  # TODO: commented out to get the tests passing.
-  # We should look at this again with fresh eyes - are we testing the right thing?
-  #
-  # test "returns file url using asset_manager_id when the model has an asset variant" do
-  #   model = build(:attachment_data, attachable: build(:draft_edition, id: 1))
-  #   model.save!
-  #   model.reload
-
-  #   assert_equal "http://assets-host/media/asset_manager_id_thumbnail/thumbnail_greenpaper.pdf.png", model.file.url(:thumbnail)
-  # end
-
   # TODO: commented out to get the tests passing.
   # We should better understand how to rewrite this test and whether it is still valuable.
   #