Skip to content

Commit 0db79b2

Browse files
committed
test: add getFeatureDefinitions mock
1 parent 155d030 commit 0db79b2

1 file changed

Lines changed: 16 additions & 12 deletions

File tree

packages/openfeature-node-provider/src/index.test.ts

Lines changed: 16 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -16,8 +16,8 @@ vi.mock("@bucketco/node-sdk", () => {
1616
});
1717

1818
const bucketClientMock = {
19-
getFeatures: vi.fn(),
2019
getFeature: vi.fn(),
20+
getFeatureDefinitions: vi.fn().mockReturnValue([]),
2121
initialize: vi.fn().mockResolvedValue({}),
2222
flush: vi.fn(),
2323
track: vi.fn(),
@@ -54,6 +54,7 @@ describe("BucketNodeProvider", () => {
5454
enabled: boolean,
5555
configKey?: string | null,
5656
configPayload?: any,
57+
flagKey = testFlagKey,
5758
) {
5859
const config = {
5960
key: configKey,
@@ -65,15 +66,15 @@ describe("BucketNodeProvider", () => {
6566
config,
6667
});
6768

68-
bucketClientMock.getFeatures = vi.fn().mockReturnValue({
69-
[testFlagKey]: {
70-
isEnabled: enabled,
71-
config: {
72-
key: "key",
73-
payload: configPayload,
74-
},
69+
// Mock getFeatureDefinitions to return feature definitions that include the specified flag
70+
bucketClientMock.getFeatureDefinitions = vi.fn().mockReturnValue([
71+
{
72+
key: flagKey,
73+
description: "Test feature flag",
74+
flag: {},
75+
config: {},
7576
},
76-
});
77+
]);
7778
}
7879

7980
beforeEach(async () => {
@@ -181,8 +182,11 @@ describe("BucketNodeProvider", () => {
181182
expect(mockTranslatorFn).toHaveBeenCalledTimes(1);
182183
expect(mockTranslatorFn).toHaveBeenCalledWith(context);
183184

184-
expect(bucketClientMock.getFeatures).toHaveBeenCalledTimes(1);
185-
expect(bucketClientMock.getFeatures).toHaveBeenCalledWith(bucketContext);
185+
expect(bucketClientMock.getFeatureDefinitions).toHaveBeenCalledTimes(1);
186+
expect(bucketClientMock.getFeature).toHaveBeenCalledWith(
187+
bucketContext,
188+
testFlagKey,
189+
);
186190
});
187191
});
188192

@@ -239,7 +243,7 @@ describe("BucketNodeProvider", () => {
239243
value: true,
240244
});
241245

242-
expect(bucketClientMock.getFeatures).toHaveBeenCalled();
246+
expect(bucketClientMock.getFeatureDefinitions).toHaveBeenCalled();
243247
expect(bucketClientMock.getFeature).toHaveBeenCalledWith(
244248
bucketContext,
245249
testFlagKey,

0 commit comments

Comments
 (0)