Skip to content

Commit 97ca618

Browse files
authored
Fix incorrect Select max option condition
1 parent 6643784 commit 97ca618

File tree

2 files changed

+40
-1
lines changed

2 files changed

+40
-1
lines changed

discord/ui/select.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -499,7 +499,7 @@ def append_option(self, option: SelectOption) -> None:
499499
The number of options exceeds 25.
500500
"""
501501

502-
if len(self._underlying.options) > 25:
502+
if len(self._underlying.options) >= 25:
503503
raise ValueError('maximum number of options already provided')
504504

505505
self._underlying.options.append(option)

tests/test_ui_selects.py

Lines changed: 39 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,39 @@
1+
"""
2+
The MIT License (MIT)
3+
4+
Copyright (c) 2015-present Rapptz
5+
6+
Permission is hereby granted, free of charge, to any person obtaining a
7+
copy of this software and associated documentation files (the "Software"),
8+
to deal in the Software without restriction, including without limitation
9+
the rights to use, copy, modify, merge, publish, distribute, sublicense,
10+
and/or sell copies of the Software, and to permit persons to whom the
11+
Software is furnished to do so, subject to the following conditions:
12+
13+
The above copyright notice and this permission notice shall be included in
14+
all copies or substantial portions of the Software.
15+
16+
THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS
17+
OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
18+
FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
19+
AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
20+
LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
21+
FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
22+
DEALINGS IN THE SOFTWARE.
23+
"""
24+
25+
from __future__ import annotations
26+
27+
import discord
28+
import pytest
29+
30+
31+
@pytest.mark.asyncio
32+
async def test_add_option():
33+
select = discord.ui.Select()
34+
35+
for i in range(1, 25 + 1):
36+
select.add_option(label=str(i), value=str(i))
37+
38+
with pytest.raises(ValueError):
39+
select.add_option(label="26", value="26")

0 commit comments

Comments
 (0)