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

Members に対してキャッシュを作成 #91

Merged
merged 9 commits into from
Nov 21, 2022
Merged

Conversation

kokoichi206
Copy link
Member

@kokoichi206 kokoichi206 commented Nov 16, 2022

Issue 番号

#90

対応内容・対応背景

やったこと

  • Members API をキャッシュするよう変更
    • キャッシュするための Room DBを作成
    • ローカル DB に値がなければ API を呼ぶ
    • onRefresh による手動更新時には、強制的に API を呼ぶ

やってないこと

  • Members 以外の API に対するキャッシュ対応
  • Network が取れない時の対応
    • refresh した時、ネットワークが悪ければ強制 refresh させないようにしたい
    • いやどうだろ、後で整理しないと

UI before / after

テスト観点

以下を確認

  • ローカルに値がある時、API が呼び出されないこと
  • API を連続で読んでも、重複してデータが保存されないこと
  • リモートの値が更新された時、ローカルの値がそちらに合わせられること
    • リモートが正
    • このために全てのデータを消すのは最低限の作戦。。。

補足

@kokoichi206 kokoichi206 temporarily deployed to production November 16, 2022 15:48 Inactive
@kokoichi206 kokoichi206 temporarily deployed to production November 16, 2022 16:26 Inactive
@kokoichi206 kokoichi206 temporarily deployed to production November 16, 2022 18:19 Inactive
@kokoichi206 kokoichi206 temporarily deployed to production November 17, 2022 00:42 Inactive
@kokoichi206 kokoichi206 temporarily deployed to production November 17, 2022 07:11 Inactive
@kokoichi206
Copy link
Member Author

Local ではテスト成功する

  • Android Studio 上で "Run tests" を 実行
  • Local で ./gradlew :app:connectedCheck --stacktrace を叩く

なぜか CI 上で ./gradlew :app:connectedCheck --stacktrace をするとテストがこける・・・

@kokoichi206
Copy link
Member Author

CI 環境で失敗するのは話が大きくなってしまいそうなので、別件で対応する。
DI を別モジュールでやるようにした影響とは思うが。。。

#92

@kokoichi206 kokoichi206 merged commit 1e83f12 into leading Nov 21, 2022
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