Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

卒業生の声ページに、「卒業生の声」というタグがある記事をpublished_atが新しい順に取得し表示したい。 #8175

Open
machida opened this issue Nov 5, 2024 · 2 comments

Comments

@machida
Copy link
Member

machida commented Nov 5, 2024

/alumni_voices 卒業生の声ページ。

ここにインタビュー記事が並んでいるが、これはHTMLをベタ書きしている。

Image

これを動的に取得したい。
「卒業生の声」というタグがある記事をpublished_atが新しい順に取得し表示したい。

@mousu-a
Copy link
Contributor

mousu-a commented Nov 25, 2024

@komagata
お疲れ様です。下記についてご相談させていただきたいです!(すみません、内容は簡単ですが説明もあるのでちょっと長いです)

前提の説明

welcome_test.rbでは、welcome_controller.rbの各アクション(ルーティング)に対して、基本的に1アクションにつき1テストの形で実装しています。

# welcome_controller.rb
  def index
    @mentors = current_user ? User.mentors_sorted_by_created_at : User.visible_sorted_mentors
  end

  def alumni_voices; end

...

  def pricing; end

...

  def practices; end

...

# welcome_test.rb

 test 'GET /welcome' do
    visit '/welcome'
    assert_equal 'プログラミングスクール FJORD BOOT CAMP(フィヨルドブートキャンプ)', title
    assert_selector "meta[property='og:title'][content='FJORD BOOT CAMP(フィヨルドブートキャンプ)']", visible: false
    assert_selector "meta[name='twitter:title'][content='FJORD BOOT CAMP(フィヨルドブートキャンプ)']", visible: false
  end

  test 'GET /practices' do
    visit '/practices'
    assert_equal '学習内容 | FJORD BOOT CAMP(フィヨルドブートキャンプ)', title
    assert_selector "meta[property='og:title'][content='学習内容']", visible: false
    assert_selector "meta[name='twitter:title'][content='学習内容']", visible: false
  end

  test 'GET /pricing' do
    visit '/pricing'
    assert_equal '料金 | FJORD BOOT CAMP(フィヨルドブートキャンプ)', title
    assert_selector "meta[property='og:title'][content='料金']", visible: false
    assert_selector "meta[name='twitter:title'][content='料金']", visible: false
  end
...

やりたいこと

その文脈を踏襲する形で、本issueで変更を加える/alumni_voicesに対しても↓のようなシステムテストを実装する予定です。

# welcome_test.rb
...
 test 'GET /alumni_voices' do
    visit '/alumni_voices'
    assert_equal '卒業生の声| FJORD BOOT CAMP(フィヨルドブートキャンプ)', title
    assert_selector "meta[property='og:title'][content='卒業生の声']", visible: false
    assert_selector "meta[name='twitter:title'][content='卒業生の声']", visible: false
  end

(補足:本issueで/alumni_voicesは必要なデータを動的に取得、表示するように変更するので、これ以外にも/alumni_voicesのシステムテストを追加予定です)

問題

こちらのissueにある通り(#8216)卒業生の声のページのmeta[title], meta[description]は現在FAQのものを流用しています。このままでは前述した実装予定のテストが通らないため、先にそちらのissue(#8216)をマージしてからこちらのissue(のテストの実装)を進めたいです。

お願いしたいこと

こちらのissueをmouseにアサインお願い出来ないでしょうか。
(もしくは簡単なissueですのでチーム開発に入りたての方に振っていただくのも良いかもしれません)

ご検討のほどよろしくお願い致します🙇‍♂️


ただ前提からして、そもそもこの提示した実装予定のテストが必要なさそうなどあれば仰ってください🙏

@komagata
Copy link
Member

@mousu-a

こちらのissueをmouseにアサインお願い出来ないでしょうか。

アサインさせていただきました〜

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: 作業中
Development

No branches or pull requests

3 participants