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

最新バージョンではないMisskeyのフォークのテストが失敗するようになっている #6

Closed
u1-liquid opened this issue Dec 27, 2023 · 15 comments · Fixed by #7

Comments

@u1-liquid
Copy link
Sponsor

テストで参照していたファイルが d32911a で移動されてしまい旧バージョンの全てのMisskeyのテストが失敗しています。

@kakkokari-gtyih
Copy link
Contributor

kakkokari-gtyih commented Dec 31, 2023

7ad169c (一つ前)を明示的に参照して対応できませんか?

@syuilo
Copy link
Member

syuilo commented Dec 31, 2023

バージョン更新してもらえば良さそう

@u1-liquid
Copy link
Sponsor Author

更新するとか修正すれば直るのは直りますが…
全ての過去のソースコードのテストが使い物にならなくなっているということを重く見てほしいですね

@syuilo
Copy link
Member

syuilo commented Dec 31, 2023

過去のテストを現在になって実行することは想定していないわね

@u1-liquid
Copy link
Sponsor Author

そんなに過去のテストでもないし1.2kの全てのmisskeyのフォークが常にdevelopを追従できるわけではないということをわかってほしい

@syuilo
Copy link
Member

syuilo commented Dec 31, 2023

  • Misskeyはフォークの面倒までは見れない(そこまでの余裕がない)
  • misskey-dev/misskeyをフォークするならmisskey-dev/assetsも同様にフォークしてもらった方が良さそう

@syuilo syuilo changed the title 最新バージョンではない全てのMisskeyのテストに失敗するようになっている 最新バージョンではないMisskeyのフォークのテストが失敗するようになっている Dec 31, 2023
@syuilo
Copy link
Member

syuilo commented Dec 31, 2023

Misskeyはフォークの面倒までは見れない

READMEとかに書いとくべきかしら

@u1-liquid
Copy link
Sponsor Author

「fork向けの変更点とか発行してくれ」など「面倒」をみてほしいということではない
こういうテストに使われているリソースの変更に気を付けてほしい(すくなくとも時間をおいて消すなどをしてほしい)ということです

@anatawa12
Copy link
Member

そもそもなのですがこれは「勝手に壊れた」という主張でしょうか?

git submoduleを正しく運用していればsubmoduleでcheckoutされるコミットは固定されるので勝手に壊れることはなく、更新したら壊れたということたまと思うのですが間違いないですか?

@riku6460
Copy link

タイムラインのテストで main を参照しているため submodule 関係なく勝手に壊れますね (main ではなくコミットを指定していれば防げました)

例: https://github.com/misskey-dev/misskey/blob/379079ee42355ae1b1982bc092e06e863a901d09/packages/backend/test/e2e/timelines.ts#L368-L369

@kakkokari-gtyih
Copy link
Contributor

とりあえずtestで参照されてるファイルだけ直下に置きなおすか

@tamaina
Copy link

tamaina commented Dec 31, 2023

タイムラインのテストで main を参照しているため submodule 関係なく勝手に壊れますね (main ではなくコミットを指定していれば防げました)

それならテストをコミットハッシュに書き換えれば済む話かなと…

@tamaina
Copy link

tamaina commented Dec 31, 2023

raw.githubusercontent.comを指定するなりする時はブランチではなくコミットハッシュやタグを参照すべきではあるとは思った

(それはそうと過去のソースコードのテストが動かないと言われても対応しきれないわけですが……)

@riku6460
Copy link

タイムラインのテストで main を参照しているため submodule 関係なく勝手に壊れますね (main ではなくコミットを指定していれば防げました)

それならテストをコミットハッシュに書き換えれば済む話かなと…

個人的には、もう壊れてしまったものは仕方がないので、旧バージョンについてはそれで良い気はします。
現行のバージョンについては、次の破壊に備えてコミットを指定すると良いのではと思っていますが、どうでしょう。

@kakkokari-gtyih
Copy link
Contributor

次の破壊に備えてコミットを指定すると良いのでは
そうします

(旧バージョンはとりあえず #7 をマージすれば何とかなるはず)

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 a pull request may close this issue.

6 participants