From 0616ff2ff8f9b0e5be61153f764011a6ded10b14 Mon Sep 17 00:00:00 2001
From: Jeremy Friesen <jeremy.n.friesen@gmail.com>
Date: Thu, 21 Dec 2023 14:05:53 -0500
Subject: [PATCH] =?UTF-8?q?=F0=9F=A7=B9=20Ensure=20feature=20specs=20run?=
 =?UTF-8?q?=20clean?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Prior to this commit, we did not automatically clean the features.  The
below ripgrep (and output) shows that there were some features which did
not start from a clean state.

```
rg "(clean|clean_repo):" spec/features --files-without-match
``

```
spec/features/accounts_spec.rb
spec/features/proprietor_spec.rb
spec/features/featured_collections_spec.rb
spec/features/user_roles_spec.rb
spec/features/oai_pmh_spec.rb
```
---
 spec/rails_helper.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/spec/rails_helper.rb b/spec/rails_helper.rb
index 70800107a..5993aeb5d 100644
--- a/spec/rails_helper.rb
+++ b/spec/rails_helper.rb
@@ -151,7 +151,7 @@
     ActiveFedora::SolrService.reset!
     # Pass `:clean' (or hyrax's convention of :clean_repo) to destroy objects in fedora/solr and
     # start from scratch
-    if example.metadata[:clean] || example.metadata[:clean_repo]
+    if example.metadata[:clean] || example.metadata[:clean_repo] || example.metadata[:type] == :feature
       ## We don't need to do `Hyrax::SolrService.wipe!` so long as we're using `ActiveFedora.clean!`;
       ## but Valkyrie is coming so be prepared.
       # Hyrax::SolrService.wipe!