Skip to content

Commit 7caec2a

Browse files
authored
Merge pull request #100 from machow/fix-pin-name-validate
fix(rsconnect): error in pin name starts with slash
2 parents e7d9587 + ae7305f commit 7caec2a

File tree

2 files changed

+11
-1
lines changed

2 files changed

+11
-1
lines changed

pins/boards.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -770,7 +770,7 @@ def pin_versions_prune(self, *args, **kwargs):
770770
super().pin_versions_prune(*args, **kwargs)
771771

772772
def validate_pin_name(self, name) -> None:
773-
if name.count("/") > 1:
773+
if name.count("/") > 1 or name.lstrip().startswith("/"):
774774
raise ValueError(f"Invalid pin name: {name}")
775775

776776
def sort_pin_versions(self, versions) -> Sequence[VersionRaw]:

pins/tests/test_boards.py

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,16 @@ def board(backend):
3131
backend.teardown()
3232

3333

34+
# misc ========================================================================
35+
36+
37+
def test_board_validate_pin_name_root(board):
38+
with pytest.raises(ValueError) as exc_info:
39+
board.path_to_pin("/some_pin")
40+
41+
assert "Invalid pin name" in exc_info.value.args[0]
42+
43+
3444
# pin_write ===================================================================
3545

3646

0 commit comments

Comments
 (0)