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.

Sign up for GitHub

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

Jump to bottom

fix tests #5544

Merged
merged 53 commits into from
Nov 10, 2019
Merged

fix tests #5544

merged 53 commits into from
Nov 10, 2019

Conversation

rinsuki
Copy link
Contributor

@rinsuki rinsuki commented Oct 24, 2019

Summary

GitHub ActionでCIを回そうとしていたら (#5522) 既存のテストが壊れていたのでテストを直すPR

rinsuki added 26 commits October 25, 2019 05:58
@rinsuki
Copy link
Contributor Author

rinsuki commented Nov 1, 2019

cc: @acid-chicken @syuilo

@acid-chicken
Copy link
Member

ちょっと面倒そうなことになってますし、また別に分けて良いと思います。

@syuilo
Copy link
Member

syuilo commented Nov 9, 2019

'TS_NODE_FILES' は、内部コマンドまたは外部コマンド、
操作可能なプログラムまたはバッチ ファイルとして認識されていません。

って出ますね

@acid-chicken
Copy link
Member

cross-env 使えば良いのでは

@rinsuki
Copy link
Contributor Author

rinsuki commented Nov 9, 2019

'TS_NODE_FILES' は、内部コマンドまたは外部コマンド、
操作可能なプログラムまたはバッチ ファイルとして認識されていません。

って出ますね

syuilo/Misskey.old@3bbf2b6 でcross-env を使って対応した

@rinsuki
Copy link
Contributor Author

rinsuki commented Nov 9, 2019

cc:@syuilo

@syuilo
Copy link
Member

syuilo commented Nov 10, 2019

API関係のテストが軒並み失敗するけどおま環？

@rinsuki
Copy link
Contributor Author

rinsuki commented Nov 10, 2019

コケるだけ言われてもログを貼らないと何もわからない (CIでは通っているので)

@rinsuki
Copy link
Contributor Author

rinsuki commented Nov 10, 2019

あー

@syuilo
Copy link
Member

syuilo commented Nov 10, 2019

私以外の人で失敗しないなら私の環境の問題だけど

@rinsuki
Copy link
Contributor Author

rinsuki commented Nov 10, 2019

.config/test.yml のlisten portを8080にしないといけないかもしれない

@syuilo
Copy link
Member

syuilo commented Nov 10, 2019 

(node:23772) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'token' of null
    at C:\Users\ai\projects\misskey\test\utils.ts:82:68
    at new Promise (<anonymous>)
    at Object.connectStream (C:\Users\ai\projects\misskey\test\utils.ts:81:9)
    at C:\Users\ai\projects\misskey\test\mute.ts:77:20
    at processTicksAndRejections (internal/process/task_queues.js:85:5)

とか出ます

@rinsuki
Copy link
Contributor Author

rinsuki commented Nov 10, 2019

ふぅ〜む

@syuilo
Copy link
Member

syuilo commented Nov 10, 2019

.config/test.yml のlisten portを8080にしないといけないかもしれない

ありがとうございます

@syuilo
Copy link
Member

syuilo commented Nov 10, 2019

んー、やっぱり同じエラー出ますね

@syuilo
Copy link
Member

syuilo commented Nov 10, 2019

他の人が成功してるなら私の問題なのでマージしたい

@rinsuki
Copy link
Contributor Author

rinsuki commented Nov 10, 2019 •
edited
Loading

それより上に別のエラー出てたりしませんか

@syuilo
Copy link
Member

syuilo commented Nov 10, 2019

あー、
QueryFailedError: relation "meta" does not exist
が出てました

@rinsuki
Copy link
Contributor Author

rinsuki commented Nov 10, 2019

test用のDBってどうしてます?

@syuilo
Copy link
Member

syuilo commented Nov 10, 2019

test用のDBってどうしてます?

開発用のDBとは分けて create database misskeytest; みたいにして作ったものです
いったんdropしてみます

@syuilo
Copy link
Member

syuilo commented Nov 10, 2019

(テストするたびに勝手にデータリセットされると思ってた)

@syuilo
Copy link
Member

syuilo commented Nov 10, 2019

できました、前のデータがテストDBに残ってたみたいです。
お騒がせしました。

@syuilo syuilo merged commit 8e5691a into add/github-actions Nov 10, 2019
@syuilo syuilo deleted the fix/tests branch November 10, 2019 05:49
@syuilo
Copy link
Member

syuilo commented Nov 10, 2019

ありがとうございます🙏🙏🙏

@rinsuki
Copy link
Contributor Author

rinsuki commented Nov 10, 2019

それはそれとしてテストDBがテスト前に初期化されないのはバグっぽい

@rinsuki
Copy link
Contributor Author

rinsuki commented Nov 10, 2019

あとで見ます

syuilo added a commit that referenced this pull request Jan 9, 2020
@rinsuki @syuilo
Add GitHub Actions (#5522)
3ecb0ab 
* add .github/workflows/nodejs.yml

* fix

* GitHub Actions: Node.js 8.xはサポートしない

* add .github/workflows/docker.yml

* Dockerビルドをキャッシュするように

* Run test in github actions

* 同リポジトリ内からのプルリクだと無駄に二回走るのを抑制

* 6925c00のdocker.ymlへの適応忘れ

* fix .circleci/misskey/test.yml

* test実行時にDBとかredisとか動かすように

* fix

* fix tests (#5544)

* fix test

* fix compile errors

* PATH引き継ぎでchild_process.spawn時のENOENTを修正

*  サーバー起動処理を共通化

* fix coding style

* fd=4をipcに使うように

* fix port

* fix

* fix ws port

* #4033 にテストケースを追従

* fix?

* fix??

* fix

* fix

* fix

* maybe fix

* fix

* node 10.xサポートしてなかった

* 11.10じゃないとだめだった

* fix

* remove chart test

* fix

* chart test復活

* fix

* 一回一回コネクションを閉じる

* Revert "一回一回コネクションを閉じる"

This reverts commit 56e35cf.

* 一回一回sync→dropしてるのをやめてみる

* fix

* fix

* …

* キャッシュを切ってみる

* add ts to require target

* omg fix

* Revert "キャッシュを切ってみる"

This reverts commit 88161c5.

* done呼び忘れ

* 実際の文字数リミットと違ってたので対応

* テストケースがバグってたので修正

* Revert "一回一回sync→dropしてるのをやめてみる"

This reverts commit a9e543b.

* fix

* fix

* fix

* fix?

* fix

* chartのconnectionを分離する

* fix

* fix

* fix tsconfig?

* Revert "fix tsconfig?"

This reverts commit ba9269e.

* fix

* TS_NODE_FILES を scripts の方で指定

* Windowsェ

* Circle CIの実行条件をmasterへのpushのみに

Co-authored-by: syuilo <Syuilotan@yahoo.co.jp>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@acid-chicken acid-chicken acid-chicken approved these changes

@syuilo syuilo Awaiting requested review from syuilo

@mei23 mei23 Awaiting requested review from mei23

@AyaMorisawa AyaMorisawa Awaiting requested review from AyaMorisawa

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@rinsuki @syuilo @acid-chicken @mei23