test(backend): APIテストの復活 #10163

saschanaz · 2023-03-01T18:55:43Z

なぜか一部にモデュールがabsolute pathでimportされる場合があるので、サポートできるようにmoduleNameMapperを編集
request()とapi()が混ざっていたのでapi()で統一
一部動作変更に対応
- reactions/createはもう修復を拒否しない
- /notes/showはもうリアクションのユーザーIDをreturnしない
- 名前を空白にできる（できていいのか）
- api-docとapi.jsonは廃止された (openapi)
- /notes/createはファイル関連エラーを無視しないようになった
- ICOファイルのmedia typeがimage/x-iconからimage/vnd.microsoft.iconに変更された

status確認しているので要らない

saschanaz · 2023-03-01T20:36:56Z

Windows case insensitivity廃止を要求する

codecov · 2023-03-01T20:37:41Z

Codecov Report

Merging #10163 (3cdf1c2) into develop (187a698) will increase coverage by 43.99%.
The diff coverage is 94.39%.

@@             Coverage Diff              @@
##           develop   #10163       +/-   ##
============================================
+ Coverage    25.17%   69.16%   +43.99%     
============================================
  Files          705      705               
  Lines        65273    65325       +52     
  Branches      2329     5184     +2855     
============================================
+ Hits         16431    45181    +28750     
+ Misses       48842    20144    -28698

Impacted Files	Coverage Δ
...s/backend/src/core/chart/ChartManagementService.ts	`90.14% <20.00%> (+1.73%)`	⬆️
packages/backend/src/core/NoteReadService.ts	`83.40% <92.59%> (+58.87%)`	⬆️
packages/backend/src/GlobalModule.ts	`100.00% <100.00%> (ø)`
packages/backend/src/boot/common.ts	`78.37% <100.00%> (+78.37%)`	⬆️
...ages/backend/src/core/CreateNotificationService.ts	`92.00% <100.00%> (+54.71%)`	⬆️
packages/backend/src/core/NoteCreateService.ts	`85.69% <100.00%> (+37.07%)`	⬆️
...es/backend/src/core/chart/charts/per-user-notes.ts	`83.05% <100.00%> (ø)`
packages/backend/src/server/ServerService.ts	`45.11% <100.00%> (+45.11%)`	⬆️
packages/backend/src/server/api/ApiCallService.ts	`62.42% <100.00%> (+62.42%)`	⬆️
...ackages/backend/src/server/api/ApiServerService.ts	`80.00% <100.00%> (+80.00%)`	⬆️
... and 475 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

saschanaz · 2023-03-01T20:38:27Z

coverageが上がらない？？

saschanaz · 2023-03-01T20:42:45Z

jestjs/jest#5274

Edit: jestjs/jest#3190 (comment) jestjs/jest#3190 (comment) nyc使うとなんとかできるといわれる

syuilo · 2023-03-01T21:03:25Z

サーバー用に子プロセスを起動すると、カバレッジの問題もありますしなりより遅いので、テストと同じプロセス内でNestインスタンスを作成してテストするようにしたいですね(他のServiceのテストと同様の方法で出来るはず)

saschanaz · 2023-03-02T10:19:26Z

サーバー用に子プロセスを起動すると、カバレッジの問題もありますしなりより遅いので、テストと同じプロセス内でNestインスタンスを作成してテストするようにしたいですね(他のServiceのテストと同様の方法で出来るはず)

これやってみていますが、あちこちでpostgresやioredisにawaitなしにアクセスしているのでテスト終わったあとconnection destroyしたらまだアクセスしたいのになんてことするんだっておこられます😭

saschanaz · 2023-03-02T11:12:28Z

とconnection destroyしたらまだアクセスしたいのになんてことするんだっておこられます

一旦destroyしないようにしました

絶対いいとは思いませんが他にいい方法がなかったので…

saschanaz · 2023-03-02T11:14:04Z

packages/backend/src/server/api/endpoints.ts

-		meta: ep.meta ?? {},
-		params: ep.paramDef,
+		get meta() { return ep.meta ?? {}; },
+		get params() { return ep.paramDef; },


インポートするだけで"Cannot access ep.meta before initialization"とかエラーが出るのでgetterにして回避しました

saschanaz · 2023-03-02T11:41:58Z

destroyしないとout of memory...

syuilo · 2023-03-02T11:53:35Z

あちこちでpostgresやioredisにawaitなしにアクセスしているのでテスト終わったあとconnection destroyしたらまだアクセスしたいのになんてことするんだっておこられます

テストが終わった後少し待ってからdestroyすれば回避できるかも？

saschanaz · 2023-03-02T12:57:41Z

あれ、治らなかったです…

saschanaz · 2023-03-02T15:51:20Z

ついに！！！（Jestやだ）

saschanaz · 2023-03-02T15:53:42Z

workerIdleMemoryLimitは発動しませんでしたね。偶然OOM出なかっただけ？

saschanaz · 2023-03-02T17:09:27Z

#10163 (comment)

Coverage 25.17% 68.72% +43.54%

偉業（自画自賛）
レターパックで現金もらえますか？ｗ

syuilo · 2023-03-03T02:13:15Z

🙏🙏🙏🙏

test(backend): APIテストの復活 (misskey-dev#10163) * Revert 1c5291f * APIテストの復活 * apiテストの移行 * moduleNameMapper修正 * simpleGetでthrowしないように status確認しているので要らない * longer timeout * ローカルでは問題ないのになんで * case sensitive * Run Nest instance within the current process * Skip some setIntervals * wait for 5 seconds * kill them all!! * logHeapUsage: true * detectOpenHandlesがじゃましているらしい * maxWorkers=1? * restore drive api tests * workerIdleMemoryLimit: 500MB * 1024MiB * Wait what

saschanaz added 2 commits March 1, 2023 12:30

Revert 1c5291f

26128df

APIテストの復活

bdf5a9f

github-actions bot added packages/backend Server side specific issue/PR 🧪Test labels Mar 1, 2023

github-actions bot requested review from acid-chicken and rinsuki March 1, 2023 18:56

saschanaz added 6 commits March 1, 2023 20:11

apiテストの移行

7d54f9c

moduleNameMapper修正

6a08f4b

simpleGetでthrowしないように

c33b5a6

status確認しているので要らない

longer timeout

7fed4e4

ローカルでは問題ないのになんで

170c288

case sensitive

ce2665a

Merge branch 'develop' into api-e2e

0e1aae0

Run Nest instance within the current process

3209a0c

saschanaz commented Mar 2, 2023

View reviewed changes

Skip some setIntervals

d9125dd

wait for 5 seconds

b081ca5

saschanaz added 3 commits March 2, 2023 15:10

kill them all!!

f0c6f09

logHeapUsage: true

30704a9

detectOpenHandlesがじゃましているらしい

7cf6c11

maxWorkers=1?

9e56b9a

saschanaz added 5 commits March 2, 2023 19:01

restore drive api tests

ed04376

Merge branch 'develop' into api-e2e

dae757e

workerIdleMemoryLimit: 500MB

8581732

1024MiB

cce9ba5

Wait what

3cdf1c2

saschanaz mentioned this pull request Mar 2, 2023

Windowsでjestが動かない #10151

Open

syuilo merged commit 61215e5 into misskey-dev:develop Mar 3, 2023

ycanardeau mentioned this pull request Mar 9, 2023

Use IMemoryDb from pg-mem for unit testing #9988

Closed

10 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

test(backend): APIテストの復活 #10163

test(backend): APIテストの復活 #10163

saschanaz commented Mar 1, 2023 •

edited

Loading

saschanaz commented Mar 1, 2023

codecov bot commented Mar 1, 2023 •

edited

Loading

saschanaz commented Mar 1, 2023

saschanaz commented Mar 1, 2023 •

edited

Loading

syuilo commented Mar 1, 2023 •

edited

Loading

saschanaz commented Mar 2, 2023 •

edited

Loading

saschanaz commented Mar 2, 2023

saschanaz Mar 2, 2023

saschanaz commented Mar 2, 2023

syuilo commented Mar 2, 2023

saschanaz commented Mar 2, 2023

saschanaz commented Mar 2, 2023

saschanaz commented Mar 2, 2023 •

edited

Loading

saschanaz commented Mar 2, 2023

syuilo commented Mar 3, 2023

test(backend): APIテストの復活 #10163

test(backend): APIテストの復活 #10163

Conversation

saschanaz commented Mar 1, 2023 • edited Loading

saschanaz commented Mar 1, 2023

codecov bot commented Mar 1, 2023 • edited Loading

Codecov Report

saschanaz commented Mar 1, 2023

saschanaz commented Mar 1, 2023 • edited Loading

syuilo commented Mar 1, 2023 • edited Loading

saschanaz commented Mar 2, 2023 • edited Loading

saschanaz commented Mar 2, 2023

saschanaz Mar 2, 2023

Choose a reason for hiding this comment

saschanaz commented Mar 2, 2023

syuilo commented Mar 2, 2023

saschanaz commented Mar 2, 2023

saschanaz commented Mar 2, 2023

saschanaz commented Mar 2, 2023 • edited Loading

saschanaz commented Mar 2, 2023

syuilo commented Mar 3, 2023

saschanaz commented Mar 1, 2023 •

edited

Loading

codecov bot commented Mar 1, 2023 •

edited

Loading

saschanaz commented Mar 1, 2023 •

edited

Loading

syuilo commented Mar 1, 2023 •

edited

Loading

saschanaz commented Mar 2, 2023 •

edited

Loading

saschanaz commented Mar 2, 2023 •

edited

Loading