search

baserproject / basercms

baserCMS : Based Website Development Project
http://basercms.net
Other
165 stars 124 forks source link

add unitTest_BcBaserHelper_getKeywords #3451

Closed thangnnmd closed 1 month ago

thangnnmd commented 1 month ago

add unitTest_BcBaserHelper_getKeywords

HungDV2022 commented 1 month ago

@ryuring レビューをお願いします。

ryuring commented 1 month ago

@thangnnmd @HungDV2022 getKeywords 内の $currentSite = $this->getView()->getRequest()->getAttribute('currentSite'); を考慮したテストになっていません。

サイトには、キーワードを設定できるようになっています。 実行時に setKeywords() を実行しない場合は、サイトに設定されたキーワードを出す仕様です。

HungDV2022 commented 1 month ago

@ryuring https://github.com/baserproject/basercms/pull/3494　こちらPRでレビューをお願いします