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

fix(backend): テストが死んでいるのを修正 #12738

Merged
merged 5 commits into from
Dec 22, 2023

Conversation

samunohito
Copy link
Member

What

develop本線でバックエンドのテストが死んでいるのを修正しました。
アップロード・ダウンロードのテストで使用するURLに間違いがあったこと、
テスト用のモジュール内で循環参照の対応ができていなかったことが原因でした。

Why

CIの妨げになっているため

Additional info (optional)

ローカルでpassすること、自分のforkのActionsでpassすることを確認

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

@github-actions github-actions bot added packages/backend Server side specific issue/PR packages/backend:test labels Dec 21, 2023
Copy link

codecov bot commented Dec 21, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (79ca93c) 81.91% compared to head (eeddfed) 79.76%.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #12738      +/-   ##
===========================================
- Coverage    81.91%   79.76%   -2.15%     
===========================================
  Files          180      956     +776     
  Lines        27475   108745   +81270     
  Branches       509     8366    +7857     
===========================================
+ Hits         22505    86743   +64238     
- Misses        4970    22002   +17032     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@samunohito samunohito mentioned this pull request Dec 21, 2023
@syuilo syuilo merged commit 433d46e into misskey-dev:develop Dec 22, 2023
16 of 18 checks passed
@syuilo
Copy link
Member

syuilo commented Dec 22, 2023

🙏🙏🙏🙏🙏

camilla-ett pushed a commit to kaseiski/misskey that referenced this pull request Jan 2, 2024
* fix test

* fix test

* fix test

* fix test

* fix test
@samunohito samunohito deleted the fix/test-failed-20231221 branch January 11, 2024 00:10
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