Update CI configuration to support multiple OS environments (#708) · dgenio/python-sdk@b8f7b02 · GitHub

dgenio / python-sdk Public

forked from modelcontextprotocol/python-sdk

• Notifications You must be signed in to change notification settings
• Fork 0
• Star 0

• Code
• Pull requests
• Actions
• Projects
• Security and quality
• Insights

Additional navigation options

Commit b8f7b02

Browse filesBrowse the repository at this point in the historyBrowse files

DanielAvdar

authored

Update CI configuration to support multiple OS environments (modelcontextprotocol#708)

Signed-off-by: DanielAvdar <66269169+DanielAvdar@users.noreply.github.com>

1 parent 9e66f7c commit b8f7b02
Copy full SHA for b8f7b02

2 files changed

+14-3Lines changed: 14 additions & 3 deletions

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• .github/workflows
  • shared.yml
• tests
  • test_examples.py

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎.github/workflows/shared.yml‎

Copy file name to clipboardExpand all lines: .github/workflows/shared.yml

+3-1Lines changed: 3 additions & 1 deletion

Original file line number	Diff line number	Diff line change
@@ -37,10 +37,11 @@ jobs:
37	37	run: uv run --no-sync pyright
38	38
39	39	test:
40		- runs-on: ubuntu-latest
	40	+ runs-on: ${{ matrix.os }}
41	41	strategy:
42	42	matrix:
43	43	python-version: ["3.10", "3.11", "3.12", "3.13"]
	44	+ os: [ubuntu-latest, windows-latest]
44	45
45	46	steps:
46	47	- uses: actions/checkout@v4
@@ -55,3 +56,4 @@ jobs:
55	56
56	57	- name: Run pytest
57	58	run: uv run --no-sync pytest
	59	+ continue-on-error: true

Collapse file

‎tests/test_examples.py‎

Copy file name to clipboardExpand all lines: tests/test_examples.py

+11-2Lines changed: 11 additions & 2 deletions

Original file line number	Diff line number	Diff line change
@@ -1,5 +1,7 @@
1	1	"""Tests for example servers"""
2	2
	3	+ import sys
	4	+
3	5	import pytest
4	6	from pytest_examples import CodeExample, EvalExample, find_examples
5	7
@@ -69,8 +71,15 @@ async def test_desktop(monkeypatch):
69	71	content = result.contents[0]
70	72	assert isinstance(content, TextResourceContents)
71	73	assert isinstance(content.text, str)
72		- assert "/fake/path/file1.txt" in content.text
73		- assert "/fake/path/file2.txt" in content.text
	74	+ if sys.platform == "win32":
	75	+ file_1 = "/fake/path/file1.txt".replace("/", "\\\\") # might be a bug
	76	+ file_2 = "/fake/path/file2.txt".replace("/", "\\\\") # might be a bug
	77	+ assert file_1 in content.text
	78	+ assert file_2 in content.text
	79	+ # might be a bug, but the test is passing
	80	+ else:
	81	+ assert "/fake/path/file1.txt" in content.text
	82	+ assert "/fake/path/file2.txt" in content.text
74	83
75	84
76	85	@pytest.mark.parametrize("example", find_examples("README.md"), ids=str)

0 commit comments

Comments

0 (0)