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

ブログページへのスマホ訪問者 E2Eテスト実装 #225

Merged

Conversation

uzulla
Copy link
Collaborator

@uzulla uzulla commented Feb 3, 2021

#137

#222 を先にマージしてください。

  • blog_crawl_spを作成
  • テスト用ヘルパーのリファクタリング及び機能追加
  • blog_crawl(PC用)のバグ修正およびリファクタリング
  • PC/SPでパラレルにコメント投稿を行うため、決め打ちとなっていた箇所を動的に変更

(作業時間 8h



- No longer the current blog_password to read.
- e2e test be fail caused by anchor tag that like`a src="#" onclick="form.submit()"`.
- Puppeteer's waitForNavigation() will be return null when with click() that element, Not return a Response object.
- If use timer for waiting, test will work. but not neat.
- My guessing. When click the a tag  (`src="#"`) by pupp, will be run waitForNavigation on not wanted timing.
- So, cancel a event in onclick event.
It is a runnable, but will be conflicts with blog_crawl.test.ts when run in parallel now(WIP).
Have to decoupling a entry(comment) id.
@fc2dev fc2dev merged commit c438bf8 into fc2blog:master Feb 8, 2021
@uzulla uzulla deleted the issue137/add-sp-e2e-test-for-guest-user branch July 31, 2021 03:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants