-
Notifications
You must be signed in to change notification settings - Fork 1
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
Adminについてバックエンド・フロントエンド間の結合テストを行う #1574
Comments
api-docs配下のフォルダ階層が深くなっているので、
|
API定義の差分一旦全量比較する PostCatalogItemRequest/PutCatalogItemRequestに差分maia側はPostにrowVersionがあり、PutにrowVersionがない(更新処理につくべきなので逆) CatalogItemResponseに差分maia側はCatalogItemResponesにrowVersionが含まれていないので、 getById の戻り値の型maia: PagedListOfCatalogItemResponse既知の事象だが、妥当なほうに合わせておきたい。
this.TotalPages = (int)Math.Ceiling(totalCount / (double)pageSize);
/// <summary>
/// 前のページが存在するかどうか示す値を取得します。
/// </summary>
public bool HasPrevious => this.Page > 1;
/// <summary>
/// 次のページが存在するかどうか示す値を取得します。
/// </summary>
public bool HasNext => this.Page < this.TotalPages; |
PagedListOfCatalogItemResponseについてはMaris側に合わせる。 |
# 開発環境において Web API をプロキシ経由で呼び出す際のエンドポイントです。
VITE_PROXY_ENDPOINT_ORIGIN=http://localhost:8081
conf.addExposedHeader("Location"); |
Consumerも同様の仕様だが、500エラーのときはエラー画面(/error)に遷移したほうがよい? |
認証を透過している関係でPermissionDeniedExceptionを打鍵で出すのが難しい。 あとは未認証ユーザーでのAPIコールも確かめられない。 |
|
念のためnpm run generate-clientをして想定外の差分が発生していないかを確認しておく。 |
更新と削除が競合した場合に、アイテム削除の HTTP DELETE のコールで 409 が検出されて適切にハンドリングされることを確認する必要がある。 |
概要
Adminアプリについて、
別々に開発していたバックエンド-フロントエンド間の結合テストを行う。
PRの差分とレビュー観点をわかりやすくするために、
developブランチに統合してから、mainブランチにマージする。
タスク
完了条件
The text was updated successfully, but these errors were encountered: