Merge pull request #5674 from fjordllc/feature/display-must-read-mark…

…-in-books 参考書籍一覧ページに必読マークを表示する
fjordllc · Nov 14, 2022 · fbcc104 · fbcc104
2 parents 36ee3a0 + 9588b9f
commit fbcc104
Show file tree

Hide file tree

Showing 4 changed files with 25 additions and 0 deletions.
diff --git a/app/decorators/book_decorator.rb b/app/decorators/book_decorator.rb
@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+module BookDecorator
+  def must_read_for_any_practices?
+    practices_books.any?(&:must_read)
+  end
+end
diff --git a/app/javascript/components/book.vue b/app/javascript/components/book.vue
@@ -16,6 +16,8 @@
           .card-books-item__rows
             .card-books-item__row
               h2.card-books-item__title
+                span.a-badge.is-danger.is-sm(v-if='book.mustRead')
+                  | 必読
                 a.card-books-item__title-link(
                   :href='book.pageUrl',
                   target='_blank',

diff --git a/app/views/api/books/_book.json.jbuilder b/app/views/api/books/_book.json.jbuilder
@@ -5,5 +5,6 @@ json.description book.description
 json.pageUrl book.page_url
 json.coverUrl book.cover_url
 json.editBookPath edit_book_path(book)
+json.mustRead book.must_read_for_any_practices?
 
 json.practices book.practices, partial: "api/books/practice", as: :practice
diff --git a/test/decorators/book_decorator_test.rb b/test/decorators/book_decorator_test.rb
@@ -0,0 +1,15 @@
+# frozen_string_literal: true
+
+require 'test_helper'
+
+class BookDecoratorTest < ActiveSupport::TestCase
+  setup do
+    @book1 = ActiveDecorator::Decorator.instance.decorate(books(:book1))
+    @book2 = ActiveDecorator::Decorator.instance.decorate(books(:book2))
+  end
+
+  test '#must_read_for_any_practices?' do
+    assert @book1.must_read_for_any_practices?
+    assert_not @book2.must_read_for_any_practices?
+  end
+end