Skip to content

Commit 1c9dc56

Browse files
Merge pull request #1052 from IntelPython/fix-sycl-queue-argument-validation
Fix sycl queue argument validation
2 parents db8fe2f + 8dd757b commit 1c9dc56

File tree

3 files changed

+14
-1
lines changed

3 files changed

+14
-1
lines changed

.pre-commit-config.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ repos:
1818
- id: black
1919
exclude: "versioneer.py|dpctl/_version.py"
2020
- repo: https://github.com/pycqa/isort
21-
rev: 5.10.1
21+
rev: 5.12.0
2222
hooks:
2323
- id: isort
2424
name: isort (python)

dpctl/_sycl_queue.pyx

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -290,6 +290,11 @@ cdef class SyclQueue(_SyclQueue):
290290
props = _parse_queue_properties(
291291
kwargs.pop('property', _queue_property_type._DEFAULT_PROPERTY)
292292
)
293+
if (kwargs):
294+
raise TypeError(
295+
f"Unsupported keyword arguments {kwargs} to "
296+
"SyclQueue constructor encountered."
297+
)
293298
len_args = len(args)
294299
if len_args == 0:
295300
status = self._init_queue_default(props)

dpctl/tests/test_sycl_queue.py

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -84,6 +84,14 @@ def test_invalid_filter_selectors(invalid_filter):
8484
dpctl.SyclQueue(invalid_filter)
8585

8686

87+
def test_unexpected_keyword():
88+
"""
89+
An unexpected keyword use raises TypeError.
90+
"""
91+
with pytest.raises(TypeError):
92+
dpctl.SyclQueue(device="cpu")
93+
94+
8795
def test_context_not_equals():
8896
try:
8997
gpuQ = dpctl.SyclQueue("gpu")

0 commit comments

Comments
 (0)