Skip to content

Commit e678553

Browse files
committed
test(flagsmith): asserting boolean default parameter outside of constructor is also taken into account
1 parent 30fefd8 commit e678553

File tree

1 file changed

+7
-1
lines changed

1 file changed

+7
-1
lines changed

libs/providers/flagsmith-client/src/lib/flagsmith-client-provider.spec.ts

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -270,12 +270,18 @@ describe('FlagsmithProvider', () => {
270270
expect(details.value).toEqual(0);
271271
expect(details.reason).toEqual('ERROR');
272272
});
273-
it('should use defaults for flags that do not exist', async () => {
273+
it('should use defaults for number flags that do not exist', async () => {
274274
await OpenFeature.setProviderAndWait(provider);
275275
const details = client.getNumberDetails('dont exist', 0);
276276
expect(details.value).toEqual(0);
277277
expect(details.reason).toEqual('DEFAULT');
278278
});
279+
it('should use defaults for boolean flags that do not exist', async () => {
280+
await OpenFeature.setProviderAndWait(provider);
281+
const details = client.getBooleanDetails('dont exist', false);
282+
expect(details.value).toEqual(false);
283+
expect(details.reason).toEqual('DEFAULT');
284+
});
279285
});
280286
describe('events', () => {
281287
it('should call the ready handler when initialized', async () => {

0 commit comments

Comments
 (0)