HOME画面に、レビュー件数（総数）を表示する

[takmita]

close #222

[takuya-okamoto-esm]

テストが落ちていますね。
坂部さんが追加したテストモジュールに関するエラーっぽいので、rebase/mergeしたときに
何かを消してしまったのかもしれません。
元のmainブランチと比較するとすぐにわかるかなと思います。

この手の問題があるので、PRを出す前には一通りのチェック（lint/type-check/test）は
手元で掛けるようにした方が良いですね。
今回は特にActionsが動かなかったので余計に気付かなかった。。。

root@28a303426a3d:/workspaces/boat-bee# make test 
pipenv run pytest
============================================================================================================================== test session starts ==============================================================================================================================
platform linux -- Python 3.9.11, pytest-7.1.1, pluggy-1.0.0
rootdir: /workspaces/boat-bee
collected 53 items                                                                                                                                                                                                                                                              

bee_slack_app/repository/test_google_books_repository.py .......                                                                                                                                                                                                          [ 13%]
bee_slack_app/repository/test_review_repository.py ............                                                                                                                                                                                                           [ 35%]
bee_slack_app/repository/test_user_repository.py .........                                                                                                                                                                                                                [ 52%]
bee_slack_app/service/test_book_search.py ....                                                                                                                                                                                                                            [ 60%]
bee_slack_app/service/test_recommend.py ....                                                                                                                                                                                                                              [ 67%]
bee_slack_app/service/test_review.py ........EE.                                                                                                                                                                                                                          [ 88%]
bee_slack_app/service/test_user.py ......                                                                                                                                                                                                                                 [100%]

==================================================================================================================================== ERRORS =====================================================================================================================================
________________________________________________________________________________________________________________ ERROR at setup of test_post_reviewでレビューを投稿できること ________________________________________________________________________________________________________________
file /workspaces/boat-bee/bee_slack_app/service/test_review.py, line 361
  def test_post_reviewでレビューを投稿できること(
E       fixture 'mocker' not found
>       available fixtures: cache, capfd, capfdbinary, caplog, capsys, capsysbinary, doctest_namespace, monkeypatch, pytestconfig, record_property, record_testsuite_property, record_xml_attribute, recwarn, tmp_path, tmp_path_factory, tmpdir, tmpdir_factory
>       use 'pytest --fixtures [testpath]' for help on them.

/workspaces/boat-bee/bee_slack_app/service/test_review.py:361
__________________________________________________________________________________________________ ERROR at setup of test_post_reviewでreview_repositoryの処理でエラーが発生した場合Noneを返すこと __________________________________________________________________________________________________
file /workspaces/boat-bee/bee_slack_app/service/test_review.py, line 397
  def test_post_reviewでreview_repositoryの処理でエラーが発生した場合Noneを返すこと(
E       fixture 'mocker' not found
>       available fixtures: cache, capfd, capfdbinary, caplog, capsys, capsysbinary, doctest_namespace, monkeypatch, pytestconfig, record_property, record_testsuite_property, record_xml_attribute, recwarn, tmp_path, tmp_path_factory, tmpdir, tmpdir_factory
>       use 'pytest --fixtures [testpath]' for help on them.

/workspaces/boat-bee/bee_slack_app/service/test_review.py:397
============================================================================================================================ short test summary info ============================================================================================================================
ERROR bee_slack_app/service/test_review.py::test_post_reviewでレビューを投稿できること
ERROR bee_slack_app/service/test_review.py::test_post_reviewでreview_repositoryの処理でエラーが発生した場合Noneを返すこと
========================================================================================================================= 51 passed, 2 errors in 9.37s ==========================================================================================================================
make: *** [Makefile:28: test] Error 1
root@28a303426a3d:/workspaces/boat-bee# 

[takuya-okamoto-esm]

limit を空（None）でも受けられるようにしたい場合、空で呼ぶ時には指定しなくても良いように
デフォルト引数にしてしまうのが良いと思います。

次のkeysも同様で、指定なしで呼ぶ時には、空（Noneや空のlist）を渡すよりは、引数をつけない
という形にした方が良いです。

多分こんな感じで書けるはず。

    limit: Optional[int] = None,
    keys: list[ReviewItemKey] = [],

参考：

• Pythonの関数でのデフォルト引数の使い方と注意点
• 引数にデフォルト値を設定する

こうすることで、今回は呼び出す側（homeのViewControllde）からはlimitやkeysのことを意識せずに
呼べるのでコードがスッキリします。
例えば、homeのコードだけを見た人は、「なんでlimitとかkeysとか渡してるの？」となるので。

多分呼ぶ側はこうなるかな。

        reviews = get_reviews(logger=getLogger())

homeからすると、limitもkeysも知らなくていことなので、わざわざ空の値を用意して入れるのではなく
最初から引数として渡さない方が良いかなと思います。

[takmita]

@takuya-okamoto-esm 対応しました。

[takuya-okamoto-esm]

命名規則は半分趣味の問題ですが。

一般的に、変数名は名詞にするのが良い感じになりやすいですね。
その場合「すべての」という修飾は単語の前に持ってきた方が英語っぽいので
例えば岡本だったら　total_review_count とかにするかなあ、と思います。

[takmita]

変数名を見直しました。

[takmita]

変数名を見直しました。

[takuya-okamoto-esm]

テストが落ちていますね。

失礼、make init 忘れてました。（テストPASSしてます）