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
SA1019: firestore.WriteBatch is deprecatedの対応 #158
Conversation
This comment has been minimized.
This comment has been minimized.
Visit the 👀 Preview 👀 for this PR (updated for commit 4ac9f96): |
This comment has been minimized.
This comment has been minimized.
@@ -19,7 +20,8 @@ import ( | |||
const verifyCustomTokenURL = "https://www.googleapis.com/identitytoolkit/v3/relyingparty/verifyCustomToken?key=%s" | |||
|
|||
type User struct { | |||
UserToken string `yaml:"userToken"` | |||
UserToken1 string `yaml:"userToken1"` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
以下のissueの対応の土台でe2eでユーザーを2つ作成できるように修正
@@ -45,10 +49,26 @@ func newGraph() graph.Graph { | |||
AuthUseCase: &moq_usecase_auth.UseCaseMock{}, | |||
} | |||
|
|||
credJSON, err := ioutil.ReadFile("../serviceAccount.json") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
testでFirestoreClientを使用するので認証のコードを追加
g.App.InviteRepository.Create(ctx, g.FirestoreClient, batch, i) | ||
|
||
if err := g.App.CommonRepository.Commit(ctx, batch); err != nil { | ||
batch := g.FirestoreClient.BulkWriter(ctx) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Batch→BulkWriterに変更
@@ -7,7 +7,7 @@ import ( | |||
"os" | |||
|
|||
cloudtasks "cloud.google.com/go/cloudtasks/apiv2" | |||
taskspb "google.golang.org/genproto/googleapis/cloud/tasks/v2" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
deprecatedになっていたので修正
Code Metrics Report
Details | | 150 (4f9f956) | 150 (4ac9f96) | +/- |
|---------------------|---------------|---------------|------|
| Coverage | 40.4% | 40.4% | 0.0% |
| Files | 9 | 9 | 0 |
| Lines | 559 | 559 | 0 |
| Covered | 226 | 226 | 0 |
| Code to Test Ratio | 1:0.1 | 1:0.1 | 0.0 |
| Code | 14700 | 14700 | 0 |
| Test | 1390 | 1390 | 0 |
- | Test Execution Time | 30s | 33s | +3s | Code coverage of files in pull request scope (69.3% → 69.3%)
Reported by octocov |
関連 issue
対応内容
開発用メモ
無し