上限・下限のどちらも設定していない項目は自動判定できません。
diff --git a/docs/quality.md b/docs/quality.md
index 285c3c4..061799e 100644
--- a/docs/quality.md
+++ b/docs/quality.md
@@ -36,6 +36,21 @@ next_title: 品質保証
発行すると検査指示番号が採番され（`IN20260501-0001`）、**対象ロットの在庫ステータスが「検査待ち」になります**。
+
+
次のロットには検査指示を発行できません（サンプル検査を除く。サンプル検査はロットを拘束しません）。
+
+- 保留・廃棄予定のロット。保留の解除は不適合の処置で行います
+- 不良のロット。ただし再検査は発行できます
+- 判定前の検査指示が既にあるロット。その指示を判定するか取り消してから発行します
+
+
工程内検査は、取り消した作業指示には発行できません。対象ロットを指定する場合は、その作業指示の品目のロットに限ります。
+
+
+
+
再検査は、そのロットの直近の判定済みの検査をやり直すものです。検査項目は元の検査から引き継がれ、規格値はいまの基準の版が使われます。
+判定済みの検査が無いロットには発行できません。
+
+
検査項目は自動で選ばれます。 マスタ管理 → 検査項目・基準 に登録された項目のうち、
検査種別・対象品目・対象工程が合致するものが検査指示に組み込まれます。
@@ -70,7 +85,8 @@ next_title: 品質保証
- 測定値を入力します。規格値の範囲と照合して、合否が自動判定されます。
- 数値でない検査は定性記録に文字で記入します。
-- 自動判定できない項目（規格値が未設定の項目）は、判定（手動）で合格／不合格を選びます。
+- 自動判定できない項目（規格値が未設定の項目）は、判定（手動）で合格／不合格を選びます。
+測定値と規格値で合否が決まる項目は、手動の判定で変えることはできません（規格外の品を使う場合は、不適合の特採で処置します）。
- 検査実績を登録を押します。
@@ -84,7 +100,7 @@ next_title: 品質保証
### 総合判定する
-**全検査項目の実績を登録してから** 判定します。
+**全検査項目の実績を登録してから** 判定します。サンプリング数が決まっている項目は、その数だけサンプル番号の違う実績がそろっている必要があります。
- 必要ならグレードを入力します（検査結果で出荷先や等級が変わる製品向け。任意）。
@@ -115,7 +131,8 @@ next_title: 品質保証
誤って発行した検査指示は **検査指示を取消** で取り消せます（承認済み・取消済みは不可）。
-
取消すと、その指示のために検査待ちで拘束していたロットが解放され、正常に戻ります。
+
取消すと、その指示のために検査待ちで拘束していたロットが、発行する前のステータスに戻ります。
+通常は正常に戻りますが、不良ロットの再検査を取り消した場合は不良に戻ります。
判定が済んでいる指示を取り消しても、ロットの現在のステータスはそのままです。
-
承認済みの検査は訂正できません。 また、判定済みの検査を訂正した場合は、
+
承認済み・取消済みの検査は訂正できません。 また、判定済みの検査を訂正した場合は、
もう一度総合判定をやり直す必要があります。
+ /// - 測定値があり規格値（下限・上限のどちらか）があれば、規格値で自動判定する（下限≦測定値≦上限）。
+ /// 手で指定した合否が自動判定と食い違えば拒否する。規格外の品を使うなら不適合の特採で処置する（C-30）
+ /// - 自動判定できない項目（規格値が無い・測定値が無い）は、手で指定した合否を使う（指定が無ければ拒否）
+ ///
+ ///
+ public static (InspectionJudgment? Judgment, string? Error) Resolve(
+ InspectionOrderItem item, decimal? measuredValue, InspectionJudgment? explicitJudgment)
+ {
+ if (measuredValue is { } measured && (item.LowerLimit is not null || item.UpperLimit is not null))
+ {
+ var pass = (item.LowerLimit is null || measured >= item.LowerLimit)
+ && (item.UpperLimit is null || measured <= item.UpperLimit);
+ var auto = pass ? InspectionJudgment.Pass : InspectionJudgment.Fail;
+ if (explicitJudgment is { } manual && manual != auto)
+ {
+ return (null, ApiText.T(
+ "検査項目 '{0}' の測定値 {1} は規格値による判定が「{2}」です。判定を手で変えることはできません（規格外の品を使う場合は不適合の特採で処置してください）。",
+ item.ItemCode, measured, EnumLabels.Of(auto)));
+ }
+ return (auto, null);
+ }
+ return explicitJudgment is null
+ ? (null, ApiText.T("検査項目 '{0}' は規格値による自動判定ができません。judgmentを指定してください。", item.ItemCode))
+ : (explicitJudgment, null);
+ }
+
+ ///
+ /// 総合判定の前提。全項目に実績があり、サンプリング数を持つ項目はその数以上のサンプル（サンプル番号の種類）が
+ /// そろっていること。満たさなければ日本語の理由を返す。サンプリング数は下限として扱う（多く測るのは妨げない）
+ ///
+ public static string? CheckReadyToJudge(
+ IReadOnlyCollection items, IReadOnlyCollection results)
+ {
+ var withoutResult = items.Count(i => results.All(r => r.InspectionItemId != i.InspectionItemId));
+ if (withoutResult > 0)
+ {
+ return ApiText.T("実績未登録の検査項目が {0} 件あります。全項目の実績登録後に判定してください。", withoutResult);
+ }
+ foreach (var item in items.Where(i => i.SamplingCount is > 0).OrderBy(i => i.ItemCode, StringComparer.Ordinal))
+ {
+ var samples = results.Where(r => r.InspectionItemId == item.InspectionItemId)
+ .Select(r => r.SampleNo).Distinct().Count();
+ if (samples < item.SamplingCount)
+ {
+ return ApiText.T("検査項目 '{0}' はサンプリング数 {1} に対して実績が {2} サンプルです。サンプリング数以上の実績を登録してから判定してください。",
+ item.ItemCode, item.SamplingCount!.Value, samples);
+ }
+ }
+ return null;
+ }
+}
diff --git a/src/MesApp.Api/Policies/InspectionLotPolicy.cs b/src/MesApp.Api/Policies/InspectionLotPolicy.cs
new file mode 100644
index 0000000..e8268e5
--- /dev/null
+++ b/src/MesApp.Api/Policies/InspectionLotPolicy.cs
@@ -0,0 +1,37 @@
+using MesApp.Api.Localization;
+using MesApp.Core.Entities;
+using MesApp.Core.Localization;
+
+namespace MesApp.Api.Policies;
+
+///
+/// 検査指示が対象ロットを検査待ちで拘束してよいかの判定（Spec.md 5.7 検査とロットの在庫ステータス。C-20）。
+///
+/// 検査指示の発行はロットを検査待ちにし、判定で正常/不良へ、取消で発行前のステータスへ戻す。
+/// 保留・廃棄予定のロットへ発行できると、発行→取消や発行→合格で保留が外れてしまう（保留の解除は不適合の処置で行う）。
+/// 判定前の指示が2件あると、先に合格した方でロットが正常になり、もう一方の判定を待たずに使えてしまう。
+///
+/// 単票API（InspectionOrdersController）と実績CSV取込の両方が InspectionService 経由でここを通る。
+///
+public static class InspectionLotPolicy
+{
+ ///
+ /// ロットを拘束する検査指示（サンプル検査以外）を発行できるか。できなければ日本語の理由を返す。
+ /// 正常・検査待ちのロットに発行できる。再検査（C-20-50-01）に限り、不合格で不良になったロットにも発行できる
+ ///
+ /// そのロットを拘束している判定前の検査指示の番号（無ければ null）
+ public static string? CheckIssuable(Lot lot, InspectionOrderType type, string? pendingOrderNo)
+ {
+ if (pendingOrderNo is not null)
+ {
+ return ApiText.T("ロット '{0}' には判定前の検査指示 {1} があります。判定するか取り消してから発行してください。",
+ lot.LotNumber, pendingOrderNo);
+ }
+ var issuable = lot.StockStatus is LotStockStatus.Normal or LotStockStatus.AwaitingInspection
+ || (type == InspectionOrderType.Reinspection && lot.StockStatus == LotStockStatus.Defective);
+ return issuable
+ ? null
+ : ApiText.T("在庫ステータスが「{0}」のロット '{1}' には検査指示を発行できません（不良ロットは再検査のみ。保留は不適合の処置で解除してください）。",
+ EnumLabels.Of(lot.StockStatus), lot.LotNumber);
+ }
+}
diff --git a/src/MesApp.Api/Services/InspectionService.cs b/src/MesApp.Api/Services/InspectionService.cs
index daace3a..0f52afb 100644
--- a/src/MesApp.Api/Services/InspectionService.cs
+++ b/src/MesApp.Api/Services/InspectionService.cs
@@ -24,7 +24,8 @@ public sealed class InspectionService(
{
///
/// 検査指示（依頼）の発行（C-20-10-02 ほか）。検査項目未指定時は種別・対象品目/工程に
- /// 合致する有効な検査基準を自動選択する。対象ロットは検査待ちになる（サンプル検査を除く）。
+ /// 合致する有効な検査基準を自動選択する（）。
+ /// 検査項目を指定した場合も同じ条件に合わない基準は拒否する。対象ロットは検査待ちになる（サンプル検査を除く）。
///
public async Task> CreateAsync(
InspectionOrderCreateRequest request, string? userId, CancellationToken ct)
@@ -43,6 +44,10 @@ public async Task> CreateAsync(
{
return Outcome.Invalid(ApiText.T("存在しない作業指示IDです。"));
}
+ if (workOrder.Status == WorkOrderStatus.Canceled)
+ {
+ return Outcome.Conflict(ApiText.T("取消済みの作業指示 {0} には工程内検査を発行できません。", workOrder.WorkOrderNo));
+ }
}
else if (request.TargetLotId is null)
{
@@ -56,9 +61,55 @@ public async Task> CreateAsync(
{
return Outcome.Invalid(ApiText.T("存在しないロットIDです。"));
}
+ // 工程内検査で指定したロットは、その作業指示の品目のロットであること（別品目のロットを拘束・判定させない）
+ if (workOrder is not null && lot.ProductId != workOrder.ProductId)
+ {
+ return Outcome.Invalid(
+ ApiText.T("ロット '{0}' は作業指示 {1} の品目のロットではありません。", lot.LotNumber, workOrder.WorkOrderNo));
+ }
+ // ロットを検査待ちで拘束する検査は、拘束してよい状態のときだけ発行する（サンプル検査は拘束しない）
+ if (request.Type != InspectionOrderType.Sample)
+ {
+ var pending = await db.InspectionOrders.AsNoTracking()
+ .Where(o => o.TargetLotId == lot.Id && o.Type != InspectionOrderType.Sample
+ && (o.Status == InspectionOrderStatus.Instructed || o.Status == InspectionOrderStatus.InProgress))
+ .Select(o => o.OrderNo)
+ .FirstOrDefaultAsync(ct);
+ if (InspectionLotPolicy.CheckIssuable(lot, request.Type, pending) is { } blocked)
+ {
+ return Outcome.Conflict(blocked);
+ }
+ }
}
- // 検査項目セットの決定
+ // 検査項目セットの決定：種別＋対象品目/工程に「かつ」で合う有効な基準（C-10-10）
+ var itemType = InspectionItemPolicy.ItemTypeOf(request.Type);
+ var processId = workOrder?.ProcessId;
+ List? inheritedItemIds = null;
+ if (request.Type == InspectionOrderType.Reinspection)
+ {
+ // 再検査（C-20-50-01）は、そのロットの直近の判定済みの検査をやり直すもの。基準はその検査の項目を引き継ぎ、
+ // 版はいまの基準を使う（受入ロットの再検査に完成品の基準を当てない）
+ var source = await db.InspectionOrders.AsNoTracking()
+ .Include(o => o.Items)
+ .Include(o => o.TargetWorkOrder)
+ .Where(o => o.TargetLotId == lot!.Id && o.Type != InspectionOrderType.Sample
+ && (o.Status == InspectionOrderStatus.Judged || o.Status == InspectionOrderStatus.Approved))
+ .OrderByDescending(o => o.Id)
+ .FirstOrDefaultAsync(ct);
+ if (source is null)
+ {
+ return Outcome.Invalid(
+ ApiText.T("ロット '{0}' には判定済みの検査が無いため再検査できません。", lot!.LotNumber));
+ }
+ inheritedItemIds = [.. source.Items.Select(i => i.InspectionItemId)];
+ itemType = await db.InspectionItems.AsNoTracking()
+ .Where(i => inheritedItemIds.Contains(i.Id))
+ .Select(i => i.Type)
+ .FirstAsync(ct);
+ processId = source.TargetWorkOrder?.ProcessId;
+ }
+ var applicable = InspectionItemPolicy.Applicable(itemType, lot?.ProductId ?? workOrder!.ProductId, processId);
List items;
if (request.ItemIds is { Count: > 0 })
{
@@ -70,24 +121,22 @@ public async Task> CreateAsync(
{
return Outcome.Invalid(ApiText.T("存在しない検査項目IDが含まれています。"));
}
+ // 指定した基準も自動選択と同じ条件で確かめる（無効な基準や他品目の基準で判定させない）
+ var isApplicable = applicable.Compile();
+ if (items.FirstOrDefault(i => !isApplicable(i)) is { } unusable)
+ {
+ return Outcome.Invalid(ApiText.T(
+ "検査項目 '{0}' はこの検査に使えません（無効、検査種別が違う、または対象の品目・工程が合いません）。", unusable.Code));
+ }
}
else
{
- // 自動選択：種別（再検査は完成品基準）＋対象品目/工程に合致する有効な基準（C-10-10）
- var itemType = request.Type switch
+ var candidates = db.InspectionItems.AsNoTracking().Where(applicable);
+ if (inheritedItemIds is not null)
{
- InspectionOrderType.Receiving => InspectionType.Receiving,
- InspectionOrderType.InProcess => InspectionType.InProcess,
- InspectionOrderType.Sample => InspectionType.Sample,
- _ => InspectionType.FinalProduct,
- };
- var productId = lot?.ProductId ?? workOrder!.ProductId;
- var processId = workOrder?.ProcessId;
- items = await db.InspectionItems.AsNoTracking()
- .Where(i => i.IsActive && i.Type == itemType &&
- (i.TargetProductId == productId ||
- (processId != null && i.TargetProcessId == processId)))
- .ToListAsync(ct);
+ candidates = candidates.Where(i => inheritedItemIds.Contains(i.Id));
+ }
+ items = await candidates.ToListAsync(ct);
if (items.Count == 0)
{
return Outcome.Invalid(
@@ -179,12 +228,11 @@ public async Task> AddResultsAsync(
{
return Outcome.Invalid(ApiText.T("検査項目ID {0} はこの検査指示の対象ではありません。", request.InspectionItemId));
}
- // 判定は指示発行時点の規格値（スナップショット）で行う
- var judgment = Judge(item, request.MeasuredValue, request.Judgment);
- if (judgment is null)
+ // 判定は指示発行時点の規格値（スナップショット）で行う。規格値で決まる合否は手で変えられない
+ var (judgment, invalid) = InspectionJudgmentPolicy.Resolve(item, request.MeasuredValue, request.Judgment);
+ if (invalid is not null)
{
- return Outcome.Invalid(
- ApiText.T("検査項目 '{0}' は規格値による自動判定ができません。judgmentを指定してください。", item.ItemCode));
+ return Outcome.Invalid(invalid);
}
db.InspectionResults.Add(new InspectionResult
{
@@ -193,7 +241,7 @@ public async Task> AddResultsAsync(
SampleNo = request.SampleNo ?? 1,
MeasuredValue = request.MeasuredValue,
TextValue = request.TextValue,
- Judgment = judgment.Value,
+ Judgment = judgment!.Value,
InspectedByUserId = userId,
InspectionDeviceId = request.InspectionDeviceId,
});
@@ -226,13 +274,10 @@ public async Task> JudgeAsync(
return Outcome.Conflict(ApiText.T("状態 '{0}' の検査指示は判定できません。", EnumLabels.Of(order.Status)));
}
- var itemsWithoutResult = order.Items
- .Where(i => order.Results.All(r => r.InspectionItemId != i.InspectionItemId))
- .ToList();
- if (itemsWithoutResult.Count > 0)
+ // 全項目の実績と、サンプリング数（下限）だけのサンプルがそろってから判定する
+ if (InspectionJudgmentPolicy.CheckReadyToJudge(order.Items, order.Results) is { } notReady)
{
- return Outcome.Invalid(
- ApiText.T("実績未登録の検査項目が {0} 件あります。全項目の実績登録後に判定してください。", itemsWithoutResult.Count));
+ return Outcome.Invalid(notReady);
}
var pass = order.Results.All(r => r.Judgment == InspectionJudgment.Pass);
@@ -281,7 +326,7 @@ await auditLogger.LogAsync("Quality", "InspectionJudge", nameof(InspectionOrder)
public async Task> CorrectResultAsync(
int orderId, int resultId, InspectionResultCorrectionRequest request, string? userId, CancellationToken ct)
{
- var order = await db.InspectionOrders.Include(o => o.TargetLot)
+ var order = await db.InspectionOrders.Include(o => o.TargetLot).Include(o => o.Items)
.FirstOrDefaultAsync(o => o.Id == orderId, ct);
if (order is null)
{
@@ -291,12 +336,23 @@ public async Task> CorrectResultAsync(
{
return Outcome.Conflict(ApiText.T("承認済みの検査は訂正できません。"));
}
+ if (order.Status == InspectionOrderStatus.Canceled)
+ {
+ return Outcome.Conflict(ApiText.T("取消済みの検査は訂正できません。"));
+ }
var result = await db.InspectionResults
.FirstOrDefaultAsync(r => r.Id == resultId && r.InspectionOrderId == orderId, ct);
if (result is null)
{
return Outcome.NotFound(ApiText.T("検査実績が存在しません。"));
}
+ // 訂正後の合否も登録と同じ条件で決める（訂正が規格外品を合格にする抜け道にならないように）
+ var item = order.Items.Single(i => i.InspectionItemId == result.InspectionItemId);
+ var (judgment, invalid) = InspectionJudgmentPolicy.Resolve(item, request.MeasuredValue, request.Judgment);
+ if (invalid is not null)
+ {
+ return Outcome.Invalid(invalid);
+ }
var before = new { value = result.MeasuredValue, text = result.TextValue, judgment = result.Judgment };
@@ -311,14 +367,14 @@ public async Task> CorrectResultAsync(
BeforeJudgment = result.Judgment,
AfterMeasuredValue = request.MeasuredValue,
AfterTextValue = request.TextValue,
- AfterJudgment = request.Judgment,
+ AfterJudgment = judgment!.Value,
Reason = request.Reason,
CorrectedByUserId = userId,
});
result.MeasuredValue = request.MeasuredValue;
result.TextValue = request.TextValue;
- result.Judgment = request.Judgment;
+ result.Judgment = judgment.Value;
result.CorrectionNote = string.IsNullOrEmpty(result.CorrectionNote)
? request.Reason
: $"{result.CorrectionNote}\n{request.Reason}";
@@ -347,7 +403,7 @@ await auditLogger.LogAsync("Quality", "InspectionCorrect", nameof(InspectionResu
{
value = request.MeasuredValue,
text = request.TextValue,
- judgment = request.Judgment,
+ judgment = judgment.Value,
},
reason = request.Reason,
}, ct: ct);
@@ -375,7 +431,10 @@ await auditLogger.LogAsync("Quality", "InspectionApprove", nameof(InspectionOrde
return Outcome.Ok(order);
}
- /// 検査指示の取消。承認済み・取消済み以外を取り消し、検査待ちで拘束していたロットを解放する
+ ///
+ /// 検査指示の取消。承認済み・取消済み以外を取り消し、検査待ちで拘束していたロットを**発行前のステータスへ戻す**。
+ /// 一律に正常へ戻すと、不良ロットの再検査を取り消しただけで不良が解除されてしまう
+ ///
public async Task> CancelAsync(int orderId, string? userId, CancellationToken ct)
{
var order = await db.InspectionOrders.Include(o => o.TargetLot)
@@ -389,34 +448,25 @@ public async Task> CancelAsync(int orderId, string? use
return Outcome.Conflict(ApiText.T("状態 '{0}' の検査指示は取消できません。", EnumLabels.Of(order.Status)));
}
order.Status = InspectionOrderStatus.Canceled;
- // 検査待ちで拘束していたロットを解放する
- if (order.TargetLot is { StockStatus: LotStockStatus.AwaitingInspection })
+ // 検査待ちで拘束していたロットを、この指示が拘束する前のステータスへ戻す（状態履歴の最初の拘束から引く）
+ if (order.TargetLot is { StockStatus: LotStockStatus.AwaitingInspection } lot)
{
- lotStatus.ChangeStatus(order.TargetLot, LotStockStatus.Normal, LotStatusChangeSource.Inspection,
- $"検査指示 {order.OrderNo} の取消による拘束解除", userId, inspectionOrderId: order.Id);
+ var before = await db.LotStatusHistories.AsNoTracking()
+ .Where(h => h.LotId == lot.Id && h.InspectionOrderId == order.Id
+ && h.ToStatus == LotStockStatus.AwaitingInspection)
+ .OrderBy(h => h.Id)
+ .Select(h => (LotStockStatus?)h.FromStatus)
+ .FirstOrDefaultAsync(ct);
+ // 履歴が無いのは発行前から検査待ちだった場合。そのままにする
+ if (before is not null)
+ {
+ lotStatus.ChangeStatus(lot, before.Value, LotStatusChangeSource.Inspection,
+ $"検査指示 {order.OrderNo} の取消による拘束解除", userId, inspectionOrderId: order.Id);
+ }
}
await db.SaveChangesAsync(ct);
await auditLogger.LogAsync("Quality", "InspectionCancel", nameof(InspectionOrder), orderId.ToString(), ct: ct);
return Outcome.Ok(order);
}
- ///
- /// 規格値との照合による自動判定（下限≦測定値≦上限）。判定不能ならnull。
- /// 基準はマスタの現在値ではなく、指示発行時点のスナップショットを使う
- ///
- private static InspectionJudgment? Judge(
- InspectionOrderItem item, decimal? measuredValue, InspectionJudgment? explicitJudgment)
- {
- if (explicitJudgment is not null)
- {
- return explicitJudgment;
- }
- if (measuredValue is null || (item.LowerLimit is null && item.UpperLimit is null))
- {
- return null;
- }
- var pass = (item.LowerLimit is null || measuredValue >= item.LowerLimit)
- && (item.UpperLimit is null || measuredValue <= item.UpperLimit);
- return pass ? InspectionJudgment.Pass : InspectionJudgment.Fail;
- }
}
diff --git a/src/MesApp.Api/Services/MasterCsvService.Import.cs b/src/MesApp.Api/Services/MasterCsvService.Import.cs
index 76e7386..01ce040 100644
--- a/src/MesApp.Api/Services/MasterCsvService.Import.cs
+++ b/src/MesApp.Api/Services/MasterCsvService.Import.cs
@@ -680,9 +680,11 @@ private async Task ImportInspectionItemsAsync(
var method = reader.Text("Method", item.Method, 200);
var sampling = reader.IntOrNull("SamplingCount", item.SamplingCount, 0);
var isActive = reader.Bool("IsActive", item.IsActive);
- if (lower is not null && upper is not null && lower > upper)
+ // 判定条件は単票APIと共通（片方だけ通る状態を作らない。Spec.md 7.4）
+ if (!reader.Failed
+ && InspectionItemPolicy.CheckDefinition(type, productId, processId, lower, upper, standard) is { } invalid)
{
- reader.Fail(ApiText.T("規格値の下限が上限を超えています。"));
+ reader.Fail(invalid);
}
if (reader.Failed)
{
diff --git a/tests/MesApp.Api.Tests/MasterCsvTests.cs b/tests/MesApp.Api.Tests/MasterCsvTests.cs
index 9e1ce58..fe90ee9 100644
--- a/tests/MesApp.Api.Tests/MasterCsvTests.cs
+++ b/tests/MesApp.Api.Tests/MasterCsvTests.cs
@@ -333,9 +333,13 @@ public class MasterCsvTests
using var factory = new ApiFactory();
using var client = await TestAuth.CreateAdminClientAsync(factory);
+ Assert.True((await ImportAsync(client, "processes", """
+ Code,Name,Category
+ PR-01,加工,InHouse
+ """)).Succeeded);
Assert.True((await ImportAsync(client, "inspection-items", """
- Code,Name,Type,LowerLimit,UpperLimit,StandardValue,Method,SamplingCount
- INS-01,外径測定,InProcess,9.5,10.5,10,ノギス,5
+ Code,Name,TargetProcessCode,Type,LowerLimit,UpperLimit,StandardValue,Method,SamplingCount
+ INS-01,外径測定,PR-01,InProcess,9.5,10.5,10,ノギス,5
""")).Succeeded);
// 名称だけの変更では版数は上がらない
@@ -354,13 +358,19 @@ public class MasterCsvTests
items = await client.GetFromJsonAsync>("/api/inspection-items");
Assert.Equal(2, items!.Single().Version);
- // 下限が上限を超える行はエラー
+ // 単票APIと同じ条件で弾く：下限＞上限・基準値が規格外・工程内検査以外で工程を指定・対象なし
var invalid = await ImportAsync(client, "inspection-items", """
- Code,Name,LowerLimit,UpperLimit
- INS-02,逆転,10,1
+ Code,Name,TargetProcessCode,Type,LowerLimit,UpperLimit,StandardValue
+ INS-02,逆転,PR-01,InProcess,10,1,
+ INS-03,基準値外,PR-01,InProcess,1,2,3
+ INS-04,受入に工程,PR-01,Receiving,,,
+ INS-05,対象なし,,InProcess,,,
""");
Assert.False(invalid.Succeeded);
- Assert.Contains(invalid.Errors, e => e.Message.Contains("下限"));
+ Assert.Contains(invalid.Errors, e => e.Line == 2 && e.Message.Contains("下限"));
+ Assert.Contains(invalid.Errors, e => e.Line == 3 && e.Message.Contains("基準値"));
+ Assert.Contains(invalid.Errors, e => e.Line == 4 && e.Message.Contains("工程内検査の基準だけ"));
+ Assert.Contains(invalid.Errors, e => e.Line == 5 && e.Message.Contains("対象品目か対象工程"));
}
[Fact]
diff --git a/tests/MesApp.Api.Tests/MasterTests.cs b/tests/MesApp.Api.Tests/MasterTests.cs
index 449d40f..e3bcfa9 100644
--- a/tests/MesApp.Api.Tests/MasterTests.cs
+++ b/tests/MesApp.Api.Tests/MasterTests.cs
@@ -310,8 +310,9 @@ public class MasterTests
new LocationRequest("LOC-01", LocationAreaType.MaterialWarehouse, "A-1-1"));
Assert.Equal(HttpStatusCode.Created, location.StatusCode);
+ var process = await CreateProcessAsync(client, "PR-01", "加工");
var inspection = await client.PostAsJsonAsync("/api/inspection-items",
- new InspectionItemRequest("INS-01", "外径測定", null, null, InspectionType.InProcess,
+ new InspectionItemRequest("INS-01", "外径測定", null, process.Id, InspectionType.InProcess,
9.5m, 10.5m, 10m, "ノギス", 5));
Assert.Equal(HttpStatusCode.Created, inspection.StatusCode);
@@ -325,7 +326,7 @@ public class MasterTests
// 検査項目の更新で版数が上がる（C-10-10-03）
var inspectionBody = await inspection.Content.ReadFromJsonAsync();
var updated = await client.PutAsJsonAsync($"/api/inspection-items/{inspectionBody!.Id}",
- new InspectionItemRequest("INS-01", "外径測定", null, null, InspectionType.InProcess,
+ new InspectionItemRequest("INS-01", "外径測定", null, process.Id, InspectionType.InProcess,
9.0m, 11.0m, 10m, "マイクロメータ", 10));
var updatedBody = await updated.Content.ReadFromJsonAsync();
Assert.Equal(2, updatedBody!.Version);
diff --git a/tests/MesApp.Api.Tests/QualityTests.cs b/tests/MesApp.Api.Tests/QualityTests.cs
index 7ed40a7..edce2dd 100644
--- a/tests/MesApp.Api.Tests/QualityTests.cs
+++ b/tests/MesApp.Api.Tests/QualityTests.cs
@@ -96,6 +96,169 @@ private static async Task CreateFinalInspectionItemAsync
return (await response.Content.ReadFromJsonAsync())!;
}
+ [Fact]
+ public async Task 工程内検査の基準は対象品目と対象工程の両方が合うものだけが選ばれる()
+ {
+ using var factory = new ApiFactory();
+ using var admin = await TestAuth.CreateAdminClientAsync(factory);
+ var ctx = await Phase3TestData.SetupAsync(admin);
+ var other = await MasterTests.CreateProductAsync(admin, "FG-99", "別の完成品", ProductType.Product);
+ var otherProcess = await MasterTests.CreateProcessAsync(admin, "PR-99", "別工程");
+ async Task CreateItemAsync(string code, int? productId, int? processId)
+ {
+ var response = await admin.PostAsJsonAsync("/api/inspection-items",
+ new InspectionItemRequest(code, code, productId, processId, InspectionType.InProcess,
+ null, null, null, null, null));
+ response.EnsureSuccessStatusCode();
+ return (await response.Content.ReadFromJsonAsync())!.Id;
+ }
+ await CreateItemAsync("INS-A", ctx.ProductId, ctx.ProcessId); // この品目のこの工程
+ await CreateItemAsync("INS-B", ctx.ProductId, null); // この品目のどの工程でも
+ await CreateItemAsync("INS-C", null, ctx.ProcessId); // この工程ならどの品目でも
+ var otherProductItem = await CreateItemAsync("INS-D", other.Id, ctx.ProcessId); // 別品目の同じ工程
+ await CreateItemAsync("INS-E", ctx.ProductId, otherProcess.Id); // 同じ品目の別工程
+
+ var order = await Phase3TestData.CreateReleasedOrderAsync(admin, ctx.ProductId, 10m);
+ var workOrderId = order.WorkOrders[0].Id;
+ var created = await admin.PostAsJsonAsync("/api/inspection-orders",
+ new InspectionOrderCreateRequest(InspectionOrderType.InProcess, null, workOrderId, null, null));
+ Assert.Equal(HttpStatusCode.Created, created.StatusCode);
+ var inspection = await created.Content.ReadFromJsonAsync();
+ Assert.Equal(["INS-A", "INS-B", "INS-C"], inspection!.Items.Select(i => i.Code).Order());
+
+ // 指定した基準も同じ条件で確かめる（別品目の基準では検査させない）
+ var manual = await admin.PostAsJsonAsync("/api/inspection-orders",
+ new InspectionOrderCreateRequest(InspectionOrderType.InProcess, null, workOrderId, [otherProductItem], null));
+ Assert.Equal(HttpStatusCode.BadRequest, manual.StatusCode);
+ Assert.Contains("INS-D", await manual.Content.ReadAsStringAsync());
+ }
+
+ [Fact]
+ public async Task 保留ロットや判定前の指示があるロットには発行できず再検査の取消で不良に戻る()
+ {
+ using var factory = new ApiFactory();
+ using var admin = await TestAuth.CreateAdminClientAsync(factory);
+ var ctx = await Phase3TestData.SetupAsync(admin);
+ var item = await CreateFinalInspectionItemAsync(admin, ctx.ProductId);
+ var lot = await Phase3TestData.ReceiveAsync(admin, ctx.ProductId, 100m, ctx.ProductLocationId);
+ async Task IssueAsync(InspectionOrderType type) =>
+ await admin.PostAsJsonAsync("/api/inspection-orders",
+ new InspectionOrderCreateRequest(type, lot.Id, null, null, null));
+ async Task LotStatusAsync() =>
+ (await admin.GetFromJsonAsync($"/api/inventory/lots/{lot.Id}"))!.StockStatus;
+
+ // 保留中のロットには発行できない（発行→取消で保留が外れてしまう）
+ (await admin.PostAsJsonAsync("/api/inventory/status",
+ new Core.Contracts.Inventory.LotStatusRequest(lot.Id, LotStockStatus.OnHold, "調査中"))).EnsureSuccessStatusCode();
+ Assert.Equal(HttpStatusCode.Conflict, (await IssueAsync(InspectionOrderType.FinalProduct)).StatusCode);
+ Assert.Equal(LotStockStatus.OnHold, await LotStatusAsync());
+ (await admin.PostAsJsonAsync("/api/inventory/status",
+ new Core.Contracts.Inventory.LotStatusRequest(lot.Id, LotStockStatus.Normal, "調査完了"))).EnsureSuccessStatusCode();
+
+ // 判定前の指示があるロットには重ねて発行できない（先に合格した方でロットが正常になってしまう）
+ var first = await (await IssueAsync(InspectionOrderType.FinalProduct)).Content.ReadFromJsonAsync();
+ var duplicated = await IssueAsync(InspectionOrderType.FinalProduct);
+ Assert.Equal(HttpStatusCode.Conflict, duplicated.StatusCode);
+ Assert.Contains(first!.OrderNo, await duplicated.Content.ReadAsStringAsync());
+ // サンプル検査はロットを拘束しないので重ねてよい
+ (await admin.PostAsJsonAsync("/api/inspection-items",
+ new InspectionItemRequest("INS-S", "保管サンプル外観", ctx.ProductId, null, InspectionType.Sample,
+ null, null, null, "目視", 1))).EnsureSuccessStatusCode();
+ Assert.Equal(HttpStatusCode.Created, (await IssueAsync(InspectionOrderType.Sample)).StatusCode);
+
+ // 不合格で不良になったロットは再検査だけ発行でき、再検査を取り消すと不良に戻る（正常にしない）
+ await admin.PostAsJsonAsync($"/api/inspection-orders/{first.Id}/results",
+ new List { new(item.Id, 1, 12.0m, null, null) });
+ (await admin.PostAsJsonAsync($"/api/inspection-orders/{first.Id}/judge", new InspectionJudgeRequest(null)))
+ .EnsureSuccessStatusCode();
+ Assert.Equal(LotStockStatus.Defective, await LotStatusAsync());
+ Assert.Equal(HttpStatusCode.Conflict, (await IssueAsync(InspectionOrderType.FinalProduct)).StatusCode);
+ var reinspection = await (await IssueAsync(InspectionOrderType.Reinspection)).Content.ReadFromJsonAsync();
+ Assert.Equal(LotStockStatus.AwaitingInspection, await LotStatusAsync());
+ (await admin.PostAsync($"/api/inspection-orders/{reinspection!.Id}/cancel", null)).EnsureSuccessStatusCode();
+ Assert.Equal(LotStockStatus.Defective, await LotStatusAsync());
+ }
+
+ [Fact]
+ public async Task 規格値で決まる合否は手で変えられずサンプリング数がそろうまで判定できない()
+ {
+ using var factory = new ApiFactory();
+ using var admin = await TestAuth.CreateAdminClientAsync(factory);
+ var ctx = await Phase3TestData.SetupAsync(admin);
+ var response = await admin.PostAsJsonAsync("/api/inspection-items",
+ new InspectionItemRequest("INS-01", "外径測定", ctx.ProductId, null, InspectionType.FinalProduct,
+ 9.5m, 10.5m, 10m, "ノギス", 2));
+ var item = (await response.Content.ReadFromJsonAsync())!;
+ var lot = await Phase3TestData.ReceiveAsync(admin, ctx.ProductId, 100m, ctx.ProductLocationId);
+ var order = (await (await admin.PostAsJsonAsync("/api/inspection-orders",
+ new InspectionOrderCreateRequest(InspectionOrderType.FinalProduct, lot.Id, null, null, null)))
+ .Content.ReadFromJsonAsync())!;
+
+ // 規格外の測定値を「合格」と指定しても通さない（規格外品を使うなら不適合の特採で処置する）
+ var overridden = await admin.PostAsJsonAsync($"/api/inspection-orders/{order.Id}/results",
+ new List { new(item.Id, 1, 12.0m, null, InspectionJudgment.Pass) });
+ Assert.Equal(HttpStatusCode.BadRequest, overridden.StatusCode);
+ // 自動判定と同じ合否なら指定してもよい
+ var recorded = await (await admin.PostAsJsonAsync($"/api/inspection-orders/{order.Id}/results",
+ new List { new(item.Id, 1, 10.0m, null, InspectionJudgment.Pass) }))
+ .Content.ReadFromJsonAsync();
+
+ // サンプリング数2に対して1サンプルでは判定できない
+ var early = await admin.PostAsJsonAsync($"/api/inspection-orders/{order.Id}/judge", new InspectionJudgeRequest(null));
+ Assert.Equal(HttpStatusCode.BadRequest, early.StatusCode);
+ Assert.Contains("サンプリング数", await early.Content.ReadAsStringAsync());
+
+ // 訂正も同じ条件：規格外の値へ直しながら合格にはできない
+ var resultId = recorded!.Results.Single().Id;
+ var badCorrection = await admin.PutAsJsonAsync($"/api/inspection-orders/{order.Id}/results/{resultId}",
+ new InspectionResultCorrectionRequest(11.0m, null, InspectionJudgment.Pass, "転記ミス"));
+ Assert.Equal(HttpStatusCode.BadRequest, badCorrection.StatusCode);
+
+ (await admin.PostAsJsonAsync($"/api/inspection-orders/{order.Id}/results",
+ new List { new(item.Id, 2, 10.1m, null, null) })).EnsureSuccessStatusCode();
+ var judged = await admin.PostAsJsonAsync($"/api/inspection-orders/{order.Id}/judge", new InspectionJudgeRequest(null));
+ Assert.Equal(HttpStatusCode.OK, judged.StatusCode);
+ }
+
+ [Fact]
+ public async Task 再検査は直近の判定済みの検査の基準を引き継ぎ取消済みの検査は訂正できない()
+ {
+ using var factory = new ApiFactory();
+ using var admin = await TestAuth.CreateAdminClientAsync(factory);
+ var ctx = await Phase3TestData.SetupAsync(admin);
+ var receivingItem = (await (await admin.PostAsJsonAsync("/api/inspection-items",
+ new InspectionItemRequest("INS-R", "受入寸法", ctx.MaterialId, null, InspectionType.Receiving,
+ 9.5m, 10.5m, 10m, "ノギス", 1))).Content.ReadFromJsonAsync())!;
+ async Task IssueAsync(InspectionOrderType type, int lotId) =>
+ await admin.PostAsJsonAsync("/api/inspection-orders", new InspectionOrderCreateRequest(type, lotId, null, null, null));
+
+ // 判定済みの検査が無いロットは再検査できない
+ var fresh = await Phase3TestData.ReceiveAsync(admin, ctx.MaterialId, 10m, ctx.MaterialLocationId);
+ Assert.Equal(HttpStatusCode.BadRequest, (await IssueAsync(InspectionOrderType.Reinspection, fresh.Id)).StatusCode);
+
+ // 受入検査で不合格になった部材ロットの再検査は、受入の基準を引き継ぐ（完成品の基準を探さない）
+ var lot = await Phase3TestData.ReceiveAsync(admin, ctx.MaterialId, 100m, ctx.MaterialLocationId);
+ var first = (await (await IssueAsync(InspectionOrderType.Receiving, lot.Id)).Content.ReadFromJsonAsync())!;
+ await admin.PostAsJsonAsync($"/api/inspection-orders/{first.Id}/results",
+ new List { new(receivingItem.Id, 1, 12.0m, null, null) });
+ (await admin.PostAsJsonAsync($"/api/inspection-orders/{first.Id}/judge", new InspectionJudgeRequest(null)))
+ .EnsureSuccessStatusCode();
+ var reinspection = await IssueAsync(InspectionOrderType.Reinspection, lot.Id);
+ Assert.Equal(HttpStatusCode.Created, reinspection.StatusCode);
+ var reinspectionOrder = (await reinspection.Content.ReadFromJsonAsync())!;
+ Assert.Equal(["INS-R"], reinspectionOrder.Items.Select(i => i.Code));
+
+ // 取り消した検査の実績は訂正できない
+ var recorded = (await (await admin.PostAsJsonAsync($"/api/inspection-orders/{reinspectionOrder.Id}/results",
+ new List { new(receivingItem.Id, 1, 10.0m, null, null) }))
+ .Content.ReadFromJsonAsync())!;
+ (await admin.PostAsync($"/api/inspection-orders/{reinspectionOrder.Id}/cancel", null)).EnsureSuccessStatusCode();
+ var correction = await admin.PutAsJsonAsync(
+ $"/api/inspection-orders/{reinspectionOrder.Id}/results/{recorded.Results.Single().Id}",
+ new InspectionResultCorrectionRequest(10.1m, null, InspectionJudgment.Pass, "転記ミス"));
+ Assert.Equal(HttpStatusCode.Conflict, correction.StatusCode);
+ }
+
[Fact]
public async Task 検査指示を取消すと検査待ちのロットが解放され承認済みは取消せない()
{