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

日報のコメントの投稿日時をクリックするとコメントURLがコピーされる機能のテストが落ちる #4201

Closed
kaisumi opened this issue Feb 10, 2022 · 3 comments
Assignees
Labels

Comments

@kaisumi
Copy link
Contributor

kaisumi commented Feb 10, 2022

概要

テストで不合格が出るが、実際の開発環境ではクリップボードにコピーされた内容の末尾に空白は追加されていない。

image

Failure:
CommentsTest#test_comment_url_is_copied_when_click_its_updated_time [/home/runner/work/bootcamp/bootcamp/test/system/comments_test.rb:245]:
--- expected
+++ actual
@@ -1 +1 @@
-"http://127.0.0.1:44295/reports/701335546?_login_name=komagata#comment_122467993"
+"http://127.0.0.1:44295/reports/701335546?_login_name=komagata#comment_122467993"
2022-02-10.20.11.52.mov

該当のテストコード

test 'comment url is copied when click its updated_time' do
visit_with_auth "/reports/#{reports(:report1).id}", 'komagata'
find('#comments.loaded', wait: 10)
first(:css, '.thread-comment__created-at').click
# クリップボードを直接読み取る方法がないので、未入力のテキストエリアを経由してクリップボードの値を読み取っている
# また、Ctrl-Vではペーストできなかったので、かわりにShift-Insertをショートカットキーとして使っている
# 参考 https://stackoverflow.com/a/57955123/1058763
find('#js-new-comment').send_keys %i[shift insert]
clip_text = find('#js-new-comment').value
assert_equal current_url + "#comment_#{comments(:comment1).id}", clip_text
end

@kaisumi kaisumi added the バグ label Feb 10, 2022
@kaisumi kaisumi self-assigned this Feb 10, 2022
@komagata
Copy link
Member

@kaisumi 空白ではなくて、改行コードなどの何かのコードが入っている気がします。

@kaisumi
Copy link
Contributor Author

kaisumi commented Feb 15, 2022

対策したPRが埋もれてしまったので以下に記載します:

@kaisumi
Copy link
Contributor Author

kaisumi commented Feb 28, 2022

develop環境のみが関与するテストの修正です。マージ後テストOKによりクローズします。

@kaisumi kaisumi closed this as completed Feb 28, 2022
@komagata komagata moved this to 完成 in bootcamp Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

No branches or pull requests

2 participants