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

4/22の本番環境デプロイのために確認未実施の修正をコメントアウト

Browse Source
This commit is contained in:
SAITO-PC-3\saito.k 2024-04-21 22:41:44 +09:00
parent 0b01da936d
commit 7eecb001c6
2 changed files with 57 additions and 54 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -2645,34 +2645,34 @@ describe('getDealers', () => {
], ],
}); });
}); });
// TODO 本番環境デプロイのためにこのテストはスキップする
// it('非表示指定されたDealer以外のDealerを取得できる', async () => {
// if (!source) fail();
// const module = await makeTestingModule(source);
// if (!module) fail();
// // 100件のDealerを作成し、country,id,company_nameを取得する
// const dealers: { country: string; id: number; name: string }[] = [];
// for (let i = 0; i < 100; i++) {
// const { id, company_name, country } = (
// await makeTestAccount(source, {
// parent_account_id: i,
// tier: TIERS.TIER4,
// country: 'JP',
// company_name: `DEALER_${i}`,
// })
// ).account;
// dealers.push({ id, name: company_name, country });
// }
// const service = module.get<AccountsService>(AccountsService);
// const context = makeContext(`uuidv4`, 'requestId');
// const result = await service.getDealers(context);
// // idが50と99のDealerを非表示にする
it('非表示指定されたDealer以外のDealerを取得できる', async () => { // expect(result.dealers.length).toBe(98);
if (!source) fail(); // expect(result).toEqual({
const module = await makeTestingModule(source); // dealers: dealers.filter((dealer) => dealer.id !== 50 && dealer.id !== 99),
if (!module) fail(); // });
// 100件のDealerを作成し、country,id,company_nameを取得する // });
const dealers: { country: string; id: number; name: string }[] = [];
for (let i = 0; i < 100; i++) {
const { id, company_name, country } = (
await makeTestAccount(source, {
parent_account_id: i,
tier: TIERS.TIER4,
country: 'JP',
company_name: `DEALER_${i}`,
})
).account;
dealers.push({ id, name: company_name, country });
}
const service = module.get<AccountsService>(AccountsService);
const context = makeContext(`uuidv4`, 'requestId');
const result = await service.getDealers(context);
// idが50と99のDealerを非表示にする
expect(result.dealers.length).toBe(98);
expect(result).toEqual({
dealers: dealers.filter((dealer) => dealer.id !== 50 && dealer.id !== 99),
});
});
it('0件でもDealerを取得できる', async () => { it('0件でもDealerを取得できる', async () => {
if (!source) fail(); if (!source) fail();

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

@ -76,8 +76,9 @@ import { ConfigService } from '@nestjs/config';
@Injectable() @Injectable()
export class AccountsService { export class AccountsService {
// プロダクト バックログ項目 4077: [保守]本番環境動作確認用のDealerアカウントを表示しないようにする の対応 // TODO 本番環境デプロイのためにこの変数はコメントアウトする　2024年4月22日
private readonly dealerAccountIdHiddenList: number[] = []; //プロダクト バックログ項目 4077: [保守]本番環境動作確認用のDealerアカウントを表示しないようにする の対応
// private readonly dealerAccountIdHiddenList: number[] = [];
private readonly logger = new Logger(AccountsService.name); private readonly logger = new Logger(AccountsService.name);
constructor( constructor(
private readonly accountRepository: AccountsRepositoryService, private readonly accountRepository: AccountsRepositoryService,
@ -93,20 +94,21 @@ export class AccountsService {
const dealerAccountIdList = this.configService.get<string>( const dealerAccountIdList = this.configService.get<string>(
'DEALER_ACCOUNT_ID_HIDDEN_LIST', 'DEALER_ACCOUNT_ID_HIDDEN_LIST',
); );
// TODO 本番環境デプロイのためにこの変数はコメントアウトする　2024年4月22日
// ディーラーアカウントIDリストを数値配列に変換する // ディーラーアカウントIDリストを数値配列に変換する
// 変換できない場合はエラーをスローする // 変換できない場合はエラーをスローする
// 存在しない場合や空文字列の場合は空の配列を返す // 存在しない場合や空文字列の場合は空の配列を返す
if (dealerAccountIdList) { // if (dealerAccountIdList) {
this.dealerAccountIdHiddenList = dealerAccountIdList // this.dealerAccountIdHiddenList = dealerAccountIdList
.split(',') // .split(',')
.map((x) => { // .map((x) => {
const id = parseInt(x, 10); // const id = parseInt(x, 10);
if (isNaN(id)) { // if (isNaN(id)) {
throw new Error('DEALER_ACCOUNT_ID_HIDDEN_LIST is invalid'); // throw new Error('DEALER_ACCOUNT_ID_HIDDEN_LIST is invalid');
} // }
return id; // return id;
}); // });
} // }
} }
/** /**
@ -1197,26 +1199,27 @@ export class AccountsService {
const dealerAccounts = await this.accountRepository.findDealerAccounts( const dealerAccounts = await this.accountRepository.findDealerAccounts(
context, context,
); );
// TODO 本番環境デプロイのためにこの変数はコメントアウトする　2024年4月22日
// プロダクト バックログ項目 4077: [保守]本番環境動作確認用のDealerアカウントを表示しないようにする の対応 // プロダクト バックログ項目 4077: [保守]本番環境動作確認用のDealerアカウントを表示しないようにする の対応
// this.dealerAccountIdHiddenListに含まれるアカウント(動作確認用のアカウント)を除外する。 // this.dealerAccountIdHiddenListに含まれるアカウント(動作確認用のアカウント)を除外する。
// 除外したアカウントをlogに出力する // 除外したアカウントをlogに出力する
const filteredDealerAccounts = dealerAccounts.filter((dealerAccount) => { // const filteredDealerAccounts = dealerAccounts.filter((dealerAccount) => {
const isHidden = this.dealerAccountIdHiddenList.includes( // const isHidden = this.dealerAccountIdHiddenList.includes(
dealerAccount.id, // dealerAccount.id,
); // );
if (isHidden) { // if (isHidden) {
this.logger.log( // this.logger.log(
`[${context.getTrackingId()}] hidden dealer account: ${ // `[${context.getTrackingId()}] hidden dealer account: ${
dealerAccount.id // dealerAccount.id
}`, // }`,
); // );
} // }
return !isHidden; // return !isHidden;
}); // });
// レスポンス用の型に変換 // レスポンス用の型に変換
const dealers: GetDealersResponse = { const dealers: GetDealersResponse = {
dealers: filteredDealerAccounts.map((dealerAccount): Dealer => { dealers: dealerAccounts.map((dealerAccount): Dealer => {
return { return {
id: dealerAccount.id, id: dealerAccount.id,
name: dealerAccount.company_name, name: dealerAccount.company_name,