Merge pull request #432 from guardian/remove-books-and-arts-pillars

Remove references to Books and Art pillars
guardian · Sep 11, 2024 · d285ddd · d285ddd
2 parents ee6dba8 + 8144ebb
commit d285ddd
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 7 deletions.
diff --git a/client/src/main/scala/com.gu.contentapi.client/utils/CapiModelEnrichment.scala b/client/src/main/scala/com.gu.contentapi.client/utils/CapiModelEnrichment.scala
@@ -186,7 +186,7 @@ object CapiModelEnrichment {
         (tagExistsWithId("tone/comment")(content) && isPillar("News")(content)) ||
           (tagExistsWithId("tone/letters")(content) && isPillar("News")(content)) ||
           isPillar("Opinion")(content)
-      val isCulture: ContentFilter = content => isPillar("Arts")(content) || isPillar("Books")(content)
+      val isCulture: ContentFilter = content => isPillar("Arts")(content)
 
       val predicates: List[(ContentFilter, Theme)] = List(
         isSpecialReport -> SpecialReportTheme,

diff --git a/client/src/test/scala/com.gu.contentapi.client/model/utils/CapiModelEnrichmentTest.scala b/client/src/test/scala/com.gu.contentapi.client/model/utils/CapiModelEnrichmentTest.scala
@@ -520,12 +520,6 @@ class CapiModelEnrichmentFormatTest extends AnyFlatSpec with MockitoSugar with M
     f.content.theme shouldEqual CulturePillar
   }
 
-  it should "return a theme of 'CulturePillar' when has a pillarName of 'Books'" in {
-    val f = fixture
-    when(f.content.pillarName) thenReturn Some("Books")
-
-    f.content.theme shouldEqual CulturePillar
-  }
 
   it should "return a theme of 'LifestylePillar' when has a pillarName of 'Lifestyle'" in {
     val f = fixture