nik/OMDSCloud
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge branch 'develop' into release-ccb

Browse Source
This commit is contained in:
SAITO-PC-3\saito.k 2024-05-20 13:57:26 +09:00
commit 7af5c50a27
7 changed files with 63 additions and 77 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
dictation_client/src/common/errors/code.ts
View File

@ -80,7 +80,6 @@ export const errorCodes = [
"E017001", // 親アカウント変更不可エラー（指定したアカウントが存在しない）
"E017002", // 親アカウント変更不可エラー（階層関係が不正）
"E017003", // 親アカウント変更不可エラー（リージョンが同一でない）
"E017004", // 親アカウント変更不可エラー（国が同一でない）
"E018001", // パートナーアカウント削除エラー（削除条件を満たしていない）
"E019001", // パートナーアカウント取得不可エラー（階層構造が不正）
"E020001", // パートナーアカウント変更エラー（変更条件を満たしていない）

6
dictation_client/src/features/license/partnerLicense/operations.ts
View File

@ -174,12 +174,6 @@ export const switchParentAsync = createAsyncThunk<
);
}
if (error.code === "E017004") {
errorMessage = getTranslationID(
"changeOwnerPopup.message.countryMismatchError"
);
}
thunkApi.dispatch(
openSnackbar({
level: "error",

1
dictation_server/src/common/error/code.ts
View File

@ -85,7 +85,6 @@ export const ErrorCodes = [
'E017001', // 親アカウント変更不可エラー（指定したアカウントが存在しない）
'E017002', // 親アカウント変更不可エラー（階層関係が不正）
'E017003', // 親アカウント変更不可エラー（リージョンが同一でない）
'E017004', // 親アカウント変更不可エラー（国が同一でない）
'E018001', // パートナーアカウント削除エラー（削除条件を満たしていない）
'E019001', // パートナーアカウント取得不可エラー（階層構造が不正）
'E020001', // パートナーアカウント変更エラー（変更条件を満たしていない）

1
dictation_server/src/common/error/message.ts
View File

@ -75,7 +75,6 @@ export const errors: Errors = {
E017001: 'Parent account switch failed Error: account not found',
E017002: 'Parent account switch failed Error: hierarchy mismatch',
E017003: 'Parent account switch failed Error: region mismatch',
E017004: 'Parent account switch failed Error: country mismatch',
E018001: 'Partner account delete failed Error: not satisfied conditions',
E019001: 'Partner account get failed Error: hierarchy mismatch',
E020001: 'Partner account change failed Error: not satisfied conditions',

103
dictation_server/src/features/accounts/accounts.service.spec.ts
View File

@ -8220,6 +8220,65 @@ describe('switchParent', () => {
expect(child2LicenseOrderStatuses).toBeTruthy();
});
it('第四<->第五の切り替えで、親子で国が異なる場合でも階層構造変更処理ができる', async () => {
if (!source) fail();
const module = await makeTestingModule(source);
if (!module) fail();
// 新規親アカウントのアカウントを作成する
const { account: newParent } = await makeTestAccount(source, {
tier: 4,
country: `AU`,
});
// 子アカウントを作成する
const { account: child1 } = await makeTestAccount(source, {
tier: 5,
country: `NZ`,
parent_account_id: undefined,
delegation_permission: true,
});
const { account: child2 } = await makeTestAccount(source, {
tier: 5,
country: `GB`,
parent_account_id: undefined,
delegation_permission: true,
});
// ライセンス注文作成
await createLicenseOrder(source, child1.id, 10, 1); // 注文先アカウントは何でもいいため適当
await createLicenseOrder(source, child1.id, 10, 1); // 親アカウントは何でもいいため適当
await createLicenseOrder(source, child2.id, 10, 1); // 親アカウントは何でもいいため適当
// テスト実行
const context = makeContext(`external_id`, 'requestId');
const service = module.get<AccountsService>(AccountsService);
await service.switchParent(context, newParent.id, [child1.id, child2.id]);
const child1Result = await getAccount(source, child1.id);
const child2Result = await getAccount(source, child2.id);
const child1LicenseOrderResult = await getLicenseOrders(source, child1.id);
const child2LicenseOrderResult = await getLicenseOrders(source, child2.id);
// アカウントテーブルの更新確認
expect(child1Result?.parent_account_id).toBe(newParent.id);
expect(child1Result?.delegation_permission).toBe(false);
expect(child2Result?.parent_account_id).toBe(newParent.id);
expect(child2Result?.delegation_permission).toBe(false);
// ライセンス注文が全てcancelされていることの確認
expect(child1LicenseOrderResult.length).toBe(2);
const child1LicenseOrderStatuses = child1LicenseOrderResult.every(
(x) => x.status === LICENSE_ISSUE_STATUS.CANCELED,
);
expect(child1LicenseOrderStatuses).toBeTruthy();
expect(child2LicenseOrderResult.length).toBe(1);
const child2LicenseOrderStatuses = child2LicenseOrderResult.every(
(x) => x.status === LICENSE_ISSUE_STATUS.CANCELED,
);
expect(child2LicenseOrderStatuses).toBeTruthy();
});
it('切り替え先親アカウントが存在しない場合は400エラー（親アカウント不在エラー）を返す', async () => {
if (!source) fail();
const module = await makeTestingModule(source);
@ -8403,50 +8462,6 @@ describe('switchParent', () => {
}
}
});
it('第四<->第五の切り替えで、親子で国が異なる場合は400エラー（国関係不一致エラー）を返す', async () => {
if (!source) fail();
const module = await makeTestingModule(source);
if (!module) fail();
const accountsRepositoryService = module.get<AccountsRepositoryService>(
AccountsRepositoryService,
);
const child1 = new Account();
child1.id = 1;
child1.tier = 5;
child1.country = 'AU';
const child2 = new Account();
child2.id = 2;
child2.tier = 5;
child2.country = 'NZ'; // このアカウントだけ国が異なるようにしておく
accountsRepositoryService.findAccountsById = jest
.fn()
.mockResolvedValue([child1, child2]);
const context = makeContext('external_id', 'requestId');
const service = module.get<AccountsService>(AccountsService);
const parent = new Account();
parent.id = 10;
parent.tier = 4;
parent.country = 'AU';
try {
accountsRepositoryService.findAccountById = jest
.fn()
.mockResolvedValue(parent);
await service.switchParent(context, parent.id, [child1.id, child2.id]);
} catch (e) {
if (e instanceof HttpException) {
expect(e.getStatus()).toEqual(HttpStatus.BAD_REQUEST);
expect(e.getResponse()).toEqual(makeErrorResponse('E017004'));
} else {
fail();
}
}
});
});
describe('deletePartnerAccount', () => {

19
dictation_server/src/features/accounts/accounts.service.ts
View File

@ -51,7 +51,6 @@ import { LicensesRepositoryService } from '../../repositories/licenses/licenses.
import {
AccountNotFoundError,
AdminUserNotFoundError,
CountryMismatchError,
DealerAccountNotFoundError,
HierarchyMismatchError,
RegionMismatchError,
@ -2769,11 +2768,6 @@ export class AccountsService {
makeErrorResponse('E017003'),
HttpStatus.BAD_REQUEST,
);
case CountryMismatchError:
throw new HttpException(
makeErrorResponse('E017004'),
HttpStatus.BAD_REQUEST,
);
}
}
@ -2825,9 +2819,10 @@ export class AccountsService {
children: Account[],
): {
success: boolean;
errorType: null | RegionMismatchError | CountryMismatchError;
errorType: null | RegionMismatchError;
} {
// 第三<->第四の切り替えはリージョンの一致を確認し、第四<->第五の切り替えは国の一致を確認する。
// 「プロダクト バックログ項目 4171: 階層構造切り替えのリージョンの制約を修正する」の対応で第五階層の親（第四階層）を切り替えるときはリージョン・国の制約をなくす。 2024年5月16日
// 第三<->第四の切り替えはリージョンの一致を確認する。
if (parent.tier === TIERS.TIER3) {
if (
!children.every(
@ -2843,13 +2838,7 @@ export class AccountsService {
return { success: true, errorType: null };
} else if (parent.tier === TIERS.TIER4) {
if (!children.every((child) => child.country === parent.country)) {
return {
success: false,
errorType: new CountryMismatchError('Invalid country relation'),
};
}
// 第四<->第五の切り替えは国・リージョンの一致を確認しない。
return { success: true, errorType: null };
} else {
// 親アカウントの階層が想定外の場合、本関数の使い方が間違っているので例外を投げる

9
dictation_server/src/repositories/accounts/errors/types.ts
View File

@ -52,12 +52,3 @@ export class RegionMismatchError extends Error {
this.name = 'RegionMismatchError';
}
}
/**
* 所属国不一致エラー
*/
export class CountryMismatchError extends Error {
constructor(message: string) {
super(message);
this.name = 'CountryMismatchError';
}
}