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

refactor: forkでデフォルトのノート文字数を変更した場合、E2Eテストが落ちる問題を修正する #11366

Merged
merged 2 commits into from
Jul 27, 2023

Conversation

deflis
Copy link
Contributor

@deflis deflis commented Jul 23, 2023

What

デフォルトのノート文字数を変更した場合、文字数のチェックのテストが通らない箇所を修正します

Why

packages/backend/src/const.ts に定義されているデフォルトの投稿可能文字数から文字数を変更した場合、CIが落ちるようになっています。
forkなどでデフォルト文字数が変更されていた場合に文字数のチェックがうまくいかない状態になります。
forkの事情ではありますが、CIが落ちている状態をよしとすることはできないと思うので、変更可能な定数の箇所であることを踏まえ、テストにべた書きをしないようにしました。

Additional info (optional)

E2Eテストを動かす方法についてコントリビューションガイドのドキュメントになかったのですが、github actionsで動かして確認しています。

Checklist

  • Read the contribution guide
  • Test working in a local environment
  • (If needed) Add story of storybook
  • (If needed) Update CHANGELOG.md
  • (If possible) Add tests

@codecov
Copy link

codecov bot commented Jul 23, 2023

Codecov Report

Merging #11366 (f6e2210) into develop (7097d55) will increase coverage by 0.11%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop   #11366      +/-   ##
===========================================
+ Coverage    77.68%   77.79%   +0.11%     
===========================================
  Files          919      919              
  Lines        92635    92635              
  Branches      7749     7774      +25     
===========================================
+ Hits         71967    72069     +102     
+ Misses       20668    20566     -102     

see 17 files with indirect coverage changes

@syuilo syuilo merged commit 5083458 into misskey-dev:develop Jul 27, 2023
@syuilo
Copy link
Member

syuilo commented Jul 27, 2023

👍🏻

@deflis deflis deleted the fix-max-note-text-length branch July 29, 2023 20:52
@tai-cha tai-cha mentioned this pull request Jul 31, 2023
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packages/backend:test packages/backend Server side specific issue/PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants