Merged PR 873: dev動作確認不具合修正

## 概要
[Task4131: dev動作確認不具合修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/4131)

- メール送信対象のアカウント取得条件を修正
  - 第五階層はDPAの規約に同意することがないため、accepted_dpa_versionは常にNULLとなる
    - 取得条件にaccepted_dpa_versionがNOTNULLを追加するとカラムに値が入る契機がないのでアカウントを取得できなくなってしまっていた。

## レビューポイント
- 特になし

## 動作確認状況
- ローカルで確認
- 行った修正がデグレを発生させていないことを確認できるか
  - ほかのテストケースで使用しているユーザーデータをaccepted_dpa_versionはNULLの状態で作成するようにし、テストがすべて通ることを確認

## 補足
- 相談、参考資料などがあれば
This commit is contained in:
saito.k 2024-04-17 01:01:31 +00:00
parent f975ecf551
commit 566da623bf
2 changed files with 14 additions and 6 deletions

View File

@ -168,12 +168,13 @@ async function getAlertMailTargetAccount(
// 第五のアカウントを取得
const accountRepository = datasource.getRepository(Account);
// 管理者ユーザーが規約に同意しているアカウントのみを取得
// 第五階層が同意する規約はeulaとprivacy_noticeのみであるため、それらがnullでないことを条件に追加
// 管理者が規約同意していない = 一度もログインしていないアカウントであるため、そのアカウントにはアラートメールを送信しない
// プロダクト バックログ項目 4073: [保守]システムに一度もログインしていないユーザーにはライセンスアラートメールを送信しないようにしたい の対応
const accounts = await accountRepository.find({
where: {
tier: TIERS.TIER5,
primaryAdminUser: {
accepted_dpa_version: Not(IsNull()),
accepted_eula_version: Not(IsNull()),
accepted_privacy_notice_version: Not(IsNull()),
},

View File

@ -54,7 +54,7 @@ describe("licenseAlert", () => {
const { account, admin } = await makeTestAccount(
source,
{ tier: 5 },
{ external_id: "external_id1" }
{ external_id: "external_id1", accepted_dpa_version: null }
);
await createLicense(
source,
@ -97,7 +97,11 @@ describe("licenseAlert", () => {
const { account, admin } = await makeTestAccount(
source,
{ tier: 5 },
{ external_id: "external_id2", auto_renew: false }
{
external_id: "external_id2",
auto_renew: false,
accepted_dpa_version: null,
}
);
await createLicense(
source,
@ -141,7 +145,7 @@ describe("licenseAlert", () => {
const { account, admin } = await makeTestAccount(
source,
{ tier: 5 },
{ external_id: "external_id3" }
{ external_id: "external_id3", accepted_dpa_version: null }
);
await createLicense(
source,
@ -196,7 +200,11 @@ describe("licenseAlert", () => {
const { account, admin } = await makeTestAccount(
source,
{ tier: 5 },
{ external_id: "external_id4", auto_renew: true }
{
external_id: "external_id4",
auto_renew: true,
accepted_dpa_version: null,
}
);
await createLicense(
source,
@ -242,7 +250,6 @@ describe("licenseAlert", () => {
{
external_id: "external_id5",
auto_renew: false,
accepted_dpa_version: null,
accepted_eula_version: null,
accepted_privacy_notice_version: null,
}