fix: Include prerequisiteKey in migration op event. (#481)

kinyoklion · web-flow · commit 824f59d256e6 · 2024-06-14T14:08:25.000-07:00
diff --git a/packages/shared/sdk-server/src/MigrationOpEventConversion.ts b/packages/shared/sdk-server/src/MigrationOpEventConversion.ts
@@ -177,7 +177,7 @@ function validateEvaluation(evaluation: LDMigrationEvaluation): LDMigrationEvalu
   }
 
   if (TypeValidators.String.is(inReason.prerequisiteKey)) {
-    outReason.ruleId = inReason.ruleId;
+    outReason.prerequisiteKey = inReason.prerequisiteKey;
   }
 
   if (TypeValidators.Boolean.is(inReason.inExperiment)) {
diff --git a/packages/store/node-server-sdk-dynamodb/__tests__/DynamoDBBigSegmentStore.test.ts b/packages/store/node-server-sdk-dynamodb/__tests__/DynamoDBBigSegmentStore.test.ts
@@ -81,7 +81,7 @@ async function setSegments(
   client.destroy();
 }
 
-describe.each([undefined, 'app1'])('given a redis big segment store', (prefixParam) => {
+describe.each([undefined, 'app1'])('given a dynamodb big segment store', (prefixParam) => {
   let store: DynamoDBBigSegmentStore;
 
   beforeEach(async () => {

Original file line number	Diff line number	Diff line change
`@@ -177,7 +177,7 @@ function validateEvaluation(evaluation: LDMigrationEvaluation): LDMigrationEvalu`
`177`	`177`	`}`
`178`	`178`
`179`	`179`	`if (TypeValidators.String.is(inReason.prerequisiteKey)) {`
`180`		`- outReason.ruleId = inReason.ruleId;`
	`180`	`+ outReason.prerequisiteKey = inReason.prerequisiteKey;`
`181`	`181`	`}`
`182`	`182`
`183`	`183`	`if (TypeValidators.Boolean.is(inReason.inExperiment)) {`
Original file line number	Diff line number	Diff line change
`@@ -81,7 +81,7 @@ async function setSegments(`
`81`	`81`	`client.destroy();`
`82`	`82`	`}`
`83`	`83`
`84`		`-describe.each([undefined, 'app1'])('given a redis big segment store', (prefixParam) => {`
	`84`	`+describe.each([undefined, 'app1'])('given a dynamodb big segment store', (prefixParam) => {`
`85`	`85`	`let store: DynamoDBBigSegmentStore;`
`86`	`86`
`87`	`87`	`beforeEach(async () => {`