Skip to content

Commit 71e9fad

Browse files
authored
Fix number_of_bind_group_layouts_exceeds_the_maximum_value test (#4387)
The tests changed from using the default limit to using the adapter's maximum limit but this test was still assuming the default limit. Fixed to expect the adapter's maximum limit. * #4386
1 parent e25f9be commit 71e9fad

1 file changed

Lines changed: 2 additions & 3 deletions

File tree

src/webgpu/api/validation/createPipelineLayout.spec.ts

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ TODO: review existing tests, write descriptions, and make sure tests are complet
66

77
import { AllFeaturesMaxLimitsGPUTest } from '../.././gpu_test.js';
88
import { makeTestGroup } from '../../../common/framework/test_group.js';
9-
import { count } from '../../../common/util/util.js';
9+
import { count, range } from '../../../common/util/util.js';
1010
import {
1111
bufferBindingTypeInfo,
1212
getBindingLimitForBindingType,
@@ -104,8 +104,7 @@ g.test('number_of_bind_group_layouts_exceeds_the_maximum_value')
104104
entries: [],
105105
};
106106

107-
// 4 is the maximum number of bind group layouts.
108-
const maxBindGroupLayouts = [1, 2, 3, 4].map(() =>
107+
const maxBindGroupLayouts = range(t.device.limits.maxBindGroups, () =>
109108
t.device.createBindGroupLayout(bindGroupLayoutDescriptor)
110109
);
111110

0 commit comments

Comments
 (0)