Skip to content

Commit 3742e99

Browse files
committed
refactor(governance-runtime): use decision category helpers
1 parent f4cead9 commit 3742e99

4 files changed

Lines changed: 20 additions & 20 deletions

File tree

‎src/Governance/Runtime/Approval/Execution/MutationRequestApprovalDecisionExecutor.cs‎

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -106,8 +106,8 @@ private static List<MutationRequestDecision> BuildDecisionHistory(
106106
string.Equals(requirement.ApprovalGroupId, resolvedRequirement.ApprovalGroupId, StringComparison.Ordinal) &&
107107
requirement.Status == MutationApprovalRequirementStatus.Satisfied))
108108
{
109-
decisions.Add(MutationRequestDecision.Create(
110-
MutationRequestDecisionType.Approval(MutationRequestApprovalDecisionType.QuorumSatisfied),
109+
decisions.Add(MutationRequestDecision.Approval(
110+
MutationRequestApprovalDecisionType.QuorumSatisfied,
111111
decisionContext,
112112
reason: $"Approval quorum satisfied for group '{resolvedRequirement.ApprovalGroupId}'.",
113113
metadata: new Dictionary<string, object>
@@ -122,15 +122,15 @@ private static List<MutationRequestDecision> BuildDecisionHistory(
122122

123123
if (finalizeApprovedRequest && isFullyApproved)
124124
{
125-
decisions.Add(MutationRequestDecision.Create(
126-
MutationRequestDecisionType.Lifecycle(MutationRequestLifecycleDecisionType.Approved),
125+
decisions.Add(MutationRequestDecision.Lifecycle(
126+
MutationRequestLifecycleDecisionType.Approved,
127127
decisionContext,
128128
reason: "All approval requirements were fulfilled."));
129129
}
130130
else if (!finalizeApprovedRequest)
131131
{
132-
decisions.Add(MutationRequestDecision.Create(
133-
MutationRequestDecisionType.Lifecycle(MutationRequestLifecycleDecisionType.Rejected),
132+
decisions.Add(MutationRequestDecision.Lifecycle(
133+
MutationRequestLifecycleDecisionType.Rejected,
134134
decisionContext,
135135
reason: reason ?? rejection?.Message ?? decisionContext.Reason ?? "Request was rejected during approval workflow."));
136136
}

‎src/Governance/Runtime/Approval/Execution/MutationRequestApprovalExpirationExecutor.cs‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -60,8 +60,8 @@ requirement.ExpiresAt is not null &&
6060
? "Approval requirement expired."
6161
: $"Approval requirement expired at '{requirement.ExpiresAt:O}'.")));
6262

63-
decisions.Add(MutationRequestDecision.Create(
64-
MutationRequestDecisionType.Lifecycle(MutationRequestLifecycleDecisionType.Rejected),
63+
decisions.Add(MutationRequestDecision.Lifecycle(
64+
MutationRequestLifecycleDecisionType.Rejected,
6565
decisionContext,
6666
reason: "Request was rejected because one or more approval requirements expired."));
6767

‎src/Governance/Runtime/Execution/Outcome/GovernedExecutionDecisionFactory.cs‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,8 @@ public static MutationRequestDecision CreateRejectedDecision(
1414
string reason,
1515
IReadOnlyDictionary<string, object> metadata)
1616
{
17-
return MutationRequestDecision.Create(
18-
MutationRequestDecisionType.Lifecycle(MutationRequestLifecycleDecisionType.Rejected),
17+
return MutationRequestDecision.Lifecycle(
18+
MutationRequestLifecycleDecisionType.Rejected,
1919
governanceContext,
2020
reason,
2121
metadata);
@@ -26,8 +26,8 @@ public static MutationRequestDecision CreateExecutedDecision<TState>(
2626
string resultingStateVersion,
2727
MutationResult<TState> mutationResult)
2828
{
29-
return MutationRequestDecision.Create(
30-
MutationRequestDecisionType.Lifecycle(MutationRequestLifecycleDecisionType.Executed),
29+
return MutationRequestDecision.Lifecycle(
30+
MutationRequestLifecycleDecisionType.Executed,
3131
governanceContext,
3232
"Governed request executed successfully.",
3333
new Dictionary<string, object>

‎src/Governance/Runtime/Resolution/Execution/MutationRequestVersionResolutionFactory.cs‎

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -20,8 +20,8 @@ public static MutationRequestVersionResolution BuildValidated(
2020
MutationRequestVersionEvaluation evaluation,
2121
MutationContext resolutionContext)
2222
{
23-
var validatedDecision = MutationRequestDecision.Create(
24-
MutationRequestDecisionType.VersionResolution(MutationRequestVersionResolutionDecisionType.Validated),
23+
var validatedDecision = MutationRequestDecision.VersionResolution(
24+
MutationRequestVersionResolutionDecisionType.Validated,
2525
resolutionContext,
2626
reason: MutationRequestVersionResolutionState.BuildValidatedReason(
2727
evaluation.ExpectedStateVersion,
@@ -48,8 +48,8 @@ public static MutationRequestVersionResolution BuildRejectedAsStale(
4848
MutationRequestVersionEvaluation evaluation,
4949
MutationContext resolutionContext)
5050
{
51-
var decision = MutationRequestDecision.Create(
52-
MutationRequestDecisionType.VersionResolution(MutationRequestVersionResolutionDecisionType.RejectedAsStale),
51+
var decision = MutationRequestDecision.VersionResolution(
52+
MutationRequestVersionResolutionDecisionType.RejectedAsStale,
5353
resolutionContext,
5454
reason: MutationRequestVersionResolutionState.BuildStaleReason(
5555
evaluation.ExpectedStateVersion!,
@@ -81,8 +81,8 @@ public static MutationRequestVersionResolution BuildRenewedApprovalRequired(
8181
MutationRequestVersionEvaluation evaluation,
8282
MutationContext resolutionContext)
8383
{
84-
var decision = MutationRequestDecision.Create(
85-
MutationRequestDecisionType.VersionResolution(MutationRequestVersionResolutionDecisionType.RenewedApprovalRequired),
84+
var decision = MutationRequestDecision.VersionResolution(
85+
MutationRequestVersionResolutionDecisionType.RenewedApprovalRequired,
8686
resolutionContext,
8787
reason: MutationRequestVersionResolutionState.BuildStaleReason(
8888
evaluation.ExpectedStateVersion!,
@@ -114,8 +114,8 @@ public static MutationRequestVersionResolution BuildRevalidationRequired(
114114
MutationRequestVersionEvaluation evaluation,
115115
MutationContext resolutionContext)
116116
{
117-
var decision = MutationRequestDecision.Create(
118-
MutationRequestDecisionType.VersionResolution(MutationRequestVersionResolutionDecisionType.RevalidationRequired),
117+
var decision = MutationRequestDecision.VersionResolution(
118+
MutationRequestVersionResolutionDecisionType.RevalidationRequired,
119119
resolutionContext,
120120
reason: MutationRequestVersionResolutionState.BuildStaleReason(
121121
evaluation.ExpectedStateVersion!,

0 commit comments

Comments
 (0)