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

[参考] テストの例 #59

Closed
wants to merge 6 commits into from

Conversation

kosukesaigusa
Copy link
Owner

@kosukesaigusa kosukesaigusa commented Jul 20, 2023

強制アップデート関係のテストを書くための対応の例を示しました(※ 参考に指定ただければいいので、この PR を feature/force-update にマージする必要はありません)。

いまのところ mottai_flutter_app は cloud_firestore パッケージに依存しない予定(いまの main ブランチはそうなっているので)flutterfire_gen で DocumentReference のフィールドを生成しないようにする対応をします(ほかにも公開する通常のコンストラクタも生成します)。

その変更を main ブランチに入れたら伝えますので、なるんさんの feature/force-update に main に取り込んでタスク(この PR の TODO コメントも含む)を再開していただけたらと思います!

@kosukesaigusa kosukesaigusa self-assigned this Jul 20, 2023
@kosukesaigusa kosukesaigusa changed the title [WIP] テストの例 [参考] テストの例 Jul 20, 2023
required this.forceUpdateConfigId,
required this.forceUpdateConfigReference,
// required this.forceUpdateConfigReference,
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

暫定対応としてコメントアウトした

@@ -5,9 +5,9 @@
part of 'force_update_config.dart';

class ReadForceUpdateConfig {
const ReadForceUpdateConfig._({
const ReadForceUpdateConfig({
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

モックで差し替えたいので公開する通常のコンストラクタに変えた(flutterfire_gen でそのように生成されるように変える予定)

Base automatically changed from feature/force-update to main July 26, 2023 15:54
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 this pull request may close these issues.

2 participants