From 37f5f9e55cfad202ce4c1b6e1944d8d56c8e164c Mon Sep 17 00:00:00 2001 From: ayu-0505 Date: Wed, 15 Jan 2025 16:42:20 +0900 Subject: [PATCH] =?UTF-8?q?=E3=83=96=E3=83=AD=E3=82=B0=E8=A8=98=E4=BA=8B?= =?UTF-8?q?=E4=BD=9C=E6=88=90=E6=99=82=E3=81=AB=E9=80=9A=E7=9F=A5=E3=81=8C?= =?UTF-8?q?=E9=A3=9B=E3=81=B0=E3=81=AA=E3=81=84=E3=82=88=E3=81=86=E3=80=81?= =?UTF-8?q?=E8=A9=B2=E5=BD=93=E9=83=A8=E5=88=86=E3=81=AE=E3=82=B3=E3=83=BC?= =?UTF-8?q?=E3=83=89=E3=82=92=E3=82=B3=E3=83=A1=E3=83=B3=E3=83=88=E3=82=A2?= =?UTF-8?q?=E3=82=A6=E3=83=88?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- app/controllers/articles_controller.rb | 5 ++- test/system/notification/articles_test.rb | 44 ++++++++++++----------- 2 files changed, 27 insertions(+), 22 deletions(-) diff --git a/app/controllers/articles_controller.rb b/app/controllers/articles_controller.rb index 8e42d14cf90..14c0093732e 100644 --- a/app/controllers/articles_controller.rb +++ b/app/controllers/articles_controller.rb @@ -38,7 +38,10 @@ def create @article.user = current_user if @article.user.nil? set_wip if @article.save - Newspaper.publish(:create_article, { article: @article }) + # Newspaper.publish(:create_article, { article: @article }) + # 上のコードのコメントアウトは、以下のissueのための一時的なものなので、mergeされ次第コメントアウトを外すこと。 + # https://github.com/fjordllc/bootcamp/issues/8244 + redirect_to redirect_url(@article), notice: notice_message(@article) else render :new diff --git a/test/system/notification/articles_test.rb b/test/system/notification/articles_test.rb index 5efd26e284f..4b3b0967c7c 100644 --- a/test/system/notification/articles_test.rb +++ b/test/system/notification/articles_test.rb @@ -13,27 +13,29 @@ class ArticlesTest < ApplicationSystemTestCase AbstractNotifier.delivery_mode = @delivery_mode end - test 'the notification is sent only when the article is first published' do - visit_with_auth new_article_path, 'komagata' - fill_in('article_title', with: '通知テスト1回目') - fill_in('article_body', with: 'test') - click_on '公開する' - assert_text '記事を作成しました' - - visit_with_auth notifications_path, 'hajime' - within first('.card-list-item.is-unread') do - assert_text 'komagataさんがブログに「通知テスト1回目」を投稿しました。' - end - click_link '全て既読にする' - - visit_with_auth edit_article_path(@article), 'komagata' - fill_in('article_title', with: '通知テスト2回目') - click_on '更新する' - - visit_with_auth notifications_path, 'hajime' - assert_no_selector '.card-list-item.is-unread' - assert_no_text 'komagataさんがブログに「通知テスト2回目」を投稿しました。' - end + # test 'the notification is sent only when the article is first published' do + # visit_with_auth new_article_path, 'komagata' + # fill_in('article_title', with: '通知テスト1回目') + # fill_in('article_body', with: 'test') + # click_on '公開する' + # assert_text '記事を作成しました' + + # visit_with_auth notifications_path, 'hajime' + # within first('.card-list-item.is-unread') do + # assert_text 'komagataさんがブログに「通知テスト1回目」を投稿しました。' + # end + # click_link '全て既読にする' + + # visit_with_auth edit_article_path(@article), 'komagata' + # fill_in('article_title', with: '通知テスト2回目') + # click_on '更新する' + + # visit_with_auth notifications_path, 'hajime' + # assert_no_selector '.card-list-item.is-unread' + # assert_no_text 'komagataさんがブログに「通知テスト2回目」を投稿しました。' + # end + # 上のコードのコメントアウトは、以下のissueのための一時的なものなので、mergeされ次第コメントアウトを外すこと。 + # https://github.com/fjordllc/bootcamp/issues/8244 test 'the notification is not sent when the article with WIP is saved' do visit_with_auth new_article_path, 'komagata'