Skip to content

Commit a589db0

Browse files
aronweileraronweiler
committed
debug
1 parent 05580ac commit a589db0

File tree

2 files changed

+32
-39
lines changed

2 files changed

+32
-39
lines changed

‎.github/workflows/test.yml‎

Lines changed: 31 additions & 38 deletions
Original file line numberDiff line numberDiff line change
@@ -55,25 +55,6 @@ jobs:
5555
python -c "import sys, os; sys.path.insert(0, os.path.join(os.getcwd(), 'tests')); import conftest; print('✅ conftest imported successfully')" || echo "❌ conftest import failed"
5656
python -c "from tests.test_compatibility import TestCompatibility; print('✅ test_compatibility imported successfully')" || echo "❌ test_compatibility import failed"
5757
58-
- name: Create test environment file
59-
run: |
60-
echo "Creating .env.test file..."
61-
cat > .env.test << EOF
62-
# GitHub Projects MCP Server Test Configuration
63-
TEST_GITHUB_TOKEN=\${TEST_GITHUB_TOKEN}
64-
TEST_ORG_NAME=\${TEST_ORG_NAME}
65-
TEST_PROJECT_ID=\${TEST_PROJECT_ID}
66-
TEST_REPO_OWNER=\${TEST_REPO_OWNER}
67-
TEST_REPO_NAME=\${TEST_REPO_NAME}
68-
MCP_TEST_HOST=\${MCP_TEST_HOST}
69-
MCP_TEST_PORT_SSE=\${MCP_TEST_PORT_SSE}
70-
MCP_TEST_PORT_HTTP=\${MCP_TEST_PORT_HTTP}
71-
API_MAX_RETRIES=\${API_MAX_RETRIES}
72-
API_RETRY_DELAY=\${API_RETRY_DELAY}
73-
LOG_LEVEL=\${LOG_LEVEL}
74-
TEST_ITEM_PREFIX=\${TEST_ITEM_PREFIX}
75-
EOF
76-
7758
- name: Run linting checks
7859
run: |
7960
echo "Running code quality checks..."
@@ -88,8 +69,11 @@ jobs:
8869
- name: Check test configuration
8970
env:
9071
TEST_GITHUB_TOKEN: ${{ secrets.TEST_GITHUB_TOKEN }}
91-
TEST_ORG_NAME: ${{ vars.TEST_ORG_NAME || 'redducklabs' }}
92-
TEST_PROJECT_ID: ${{ vars.TEST_PROJECT_ID || 'PVT_kwDOCdCYe84A-VAN' }}
72+
TEST_ORG_NAME: ${{ vars.TEST_ORG_NAME }}
73+
TEST_PROJECT_ID: ${{ vars.TEST_PROJECT_ID }}
74+
API_MAX_RETRIES: ${{ vars.API_MAX_RETRIES }}
75+
API_RETRY_DELAY: ${{ vars.API_RETRY_DELAY }}
76+
LOG_LEVEL: ${{ vars.LOG_LEVEL }}
9377
run: |
9478
echo "Checking test configuration..."
9579
if [ -z "$TEST_GITHUB_TOKEN" ]; then
@@ -99,32 +83,38 @@ jobs:
9983
echo "✅ TEST_GITHUB_TOKEN is configured (length: ${#TEST_GITHUB_TOKEN})"
10084
fi
10185
echo "Test environment: $TEST_ORG_NAME/$TEST_PROJECT_ID"
102-
echo "Environment variables for pytest:"
86+
echo "Required variables:"
10387
echo "- TEST_GITHUB_TOKEN: $(if [ -z "$TEST_GITHUB_TOKEN" ]; then echo "NOT SET"; else echo "SET"; fi)"
10488
echo "- TEST_ORG_NAME: $TEST_ORG_NAME"
10589
echo "- TEST_PROJECT_ID: $TEST_PROJECT_ID"
90+
echo "- API_MAX_RETRIES: $API_MAX_RETRIES"
91+
echo "- API_RETRY_DELAY: $API_RETRY_DELAY"
92+
echo "- LOG_LEVEL: $LOG_LEVEL"
10693
10794
- name: Run unit tests
10895
env:
10996
# Test configuration from repository variables/secrets
11097
TEST_GITHUB_TOKEN: ${{ secrets.TEST_GITHUB_TOKEN }}
111-
TEST_ORG_NAME: ${{ vars.TEST_ORG_NAME || 'redducklabs' }}
112-
TEST_PROJECT_ID: ${{ vars.TEST_PROJECT_ID || 'PVT_kwDOCdCYe84A-VAN' }}
113-
TEST_REPO_OWNER: ${{ vars.TEST_REPO_OWNER || 'redducklabs' }}
114-
TEST_REPO_NAME: ${{ vars.TEST_REPO_NAME || 'github-projects-mcp' }}
115-
MCP_TEST_HOST: ${{ vars.MCP_TEST_HOST || 'localhost' }}
116-
MCP_TEST_PORT_SSE: ${{ vars.MCP_TEST_PORT_SSE || '8001' }}
117-
MCP_TEST_PORT_HTTP: ${{ vars.MCP_TEST_PORT_HTTP || '8002' }}
118-
API_MAX_RETRIES: ${{ vars.API_MAX_RETRIES || '1' }}
119-
API_RETRY_DELAY: ${{ vars.API_RETRY_DELAY || '1' }}
120-
LOG_LEVEL: ${{ vars.LOG_LEVEL || 'ERROR' }}
121-
TEST_ITEM_PREFIX: ${{ vars.TEST_ITEM_PREFIX || '[MCP-TEST]' }}
98+
TEST_ORG_NAME: ${{ vars.TEST_ORG_NAME }}
99+
TEST_PROJECT_ID: ${{ vars.TEST_PROJECT_ID }}
100+
TEST_REPO_OWNER: ${{ vars.TEST_REPO_OWNER }}
101+
TEST_REPO_NAME: ${{ vars.TEST_REPO_NAME }}
102+
MCP_TEST_HOST: ${{ vars.MCP_TEST_HOST }}
103+
MCP_TEST_PORT_SSE: ${{ vars.MCP_TEST_PORT_SSE }}
104+
MCP_TEST_PORT_HTTP: ${{ vars.MCP_TEST_PORT_HTTP }}
105+
API_MAX_RETRIES: ${{ vars.API_MAX_RETRIES }}
106+
API_RETRY_DELAY: ${{ vars.API_RETRY_DELAY }}
107+
LOG_LEVEL: ${{ vars.LOG_LEVEL }}
108+
TEST_ITEM_PREFIX: ${{ vars.TEST_ITEM_PREFIX }}
122109
# GitHub API configuration for the library
123110
GITHUB_TOKEN: ${{ secrets.TEST_GITHUB_TOKEN }}
124111
run: |
125112
echo "Running test suite..."
126113
echo "Running full test suite with live GitHub API tests"
127114
echo "DEBUG: Environment check before pytest:"
115+
echo "- API_MAX_RETRIES: $API_MAX_RETRIES"
116+
echo "- API_RETRY_DELAY: $API_RETRY_DELAY"
117+
echo "- LOG_LEVEL: $LOG_LEVEL"
128118
echo "- TEST_GITHUB_TOKEN: $(if [ -z "$TEST_GITHUB_TOKEN" ]; then echo "NOT SET"; else echo "SET (${#TEST_GITHUB_TOKEN} chars)"; fi)"
129119
echo "- GITHUB_TOKEN: $(if [ -z "$GITHUB_TOKEN" ]; then echo "NOT SET"; else echo "SET (${#GITHUB_TOKEN} chars)"; fi)"
130120
echo "- TEST_ORG_NAME: $TEST_ORG_NAME"
@@ -209,14 +199,17 @@ jobs:
209199
env:
210200
# Test configuration from repository variables/secrets
211201
TEST_GITHUB_TOKEN: ${{ secrets.TEST_GITHUB_TOKEN }}
212-
TEST_ORG_NAME: ${{ vars.TEST_ORG_NAME || 'redducklabs' }}
213-
TEST_PROJECT_ID: ${{ vars.TEST_PROJECT_ID || 'PVT_kwDOCdCYe84A-VAN' }}
214-
TEST_REPO_OWNER: ${{ vars.TEST_REPO_OWNER || 'redducklabs' }}
215-
TEST_REPO_NAME: ${{ vars.TEST_REPO_NAME || 'github-projects-mcp' }}
202+
TEST_ORG_NAME: ${{ vars.TEST_ORG_NAME }}
203+
TEST_PROJECT_ID: ${{ vars.TEST_PROJECT_ID }}
204+
TEST_REPO_OWNER: ${{ vars.TEST_REPO_OWNER }}
205+
TEST_REPO_NAME: ${{ vars.TEST_REPO_NAME }}
216206
GITHUB_TOKEN: ${{ secrets.TEST_GITHUB_TOKEN }}
217-
LOG_LEVEL: ERROR
207+
API_MAX_RETRIES: ${{ vars.API_MAX_RETRIES }}
208+
API_RETRY_DELAY: ${{ vars.API_RETRY_DELAY }}
209+
LOG_LEVEL: ${{ vars.LOG_LEVEL }}
218210
run: |
219211
echo "Running canary test (live integration)..."
212+
echo "Config check - API_MAX_RETRIES: $API_MAX_RETRIES, API_RETRY_DELAY: $API_RETRY_DELAY, LOG_LEVEL: $LOG_LEVEL"
220213
pytest tests/test_canary.py tests/test_live_integration.py -v -s
221214
222215
- name: Upload test results

‎tests/conftest.py‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ def event_loop() -> Generator[asyncio.AbstractEventLoop, None, None]:
3434
@pytest.fixture(scope="session")
3535
def test_config() -> Dict[str, Any]:
3636
"""Load test configuration from environment"""
37-
# Load test environment variables
37+
# Load .env.test if it exists (overrides environment variables)
3838
load_dotenv(".env.test", override=True)
3939

4040
required_vars = [

0 commit comments

Comments
 (0)