Skip to content

Test fixes #137

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
simonkurtz-MSFT merged 6 commits into from
Jan 30, 2026
Merged

Test fixes #137

simonkurtz-MSFT merged 6 commits into from
Jan 30, 2026

Conversation

@simonkurtz-MSFT
Copy link
Member

@simonkurtz-MSFT simonkurtz-MSFT commented Jan 30, 2026

No description provided.

@simonkurtz-MSFT simonkurtz-MSFT self-assigned this Jan 30, 2026
@github-actions
Copy link

github-actions bot commented Jan 30, 2026

🐍 Python 3.14 Results

Metric Status Value
Pylint Score ✅ 10.00/10
Unit Tests ✅ success
Code Coverage 📊 100%

Full Workflow Logs

@github-actions
Copy link

github-actions bot commented Jan 30, 2026

🐍 Python 3.12 Results

Metric Status Value
Pylint Score ✅ 10.00/10
Unit Tests ✅ success
Code Coverage 📊 100%

Full Workflow Logs

@github-actions
Copy link

github-actions bot commented Jan 30, 2026

🐍 Python 3.13 Results

Metric Status Value
Pylint Score ✅ 10.00/10
Unit Tests ✅ success
Code Coverage 📊 100%

Full Workflow Logs

@github-actions
Copy link

github-actions bot commented Jan 30, 2026

Python 3.12 Detailed Test Results

1 458 tests  +6   1 458 ✅ +6   16s ⏱️ -3s
    1 suites ±0       0 💤 ±0 
    1 files   ±0       0 ❌ ±0 

Results for commit 4f806b4. ± Comparison against base commit ee33f72.

This pull request removes 15 and adds 21 tests. Note that renamed tests count towards both. 
test_verify_local_setup ‑ test_check_azure_providers_subprocess_error
test_verify_local_setup ‑ test_check_bicep_cli_case_insensitive_version
test_verify_local_setup ‑ test_check_bicep_cli_empty_version
test_verify_local_setup ‑ test_check_bicep_cli_uses_first_line_only
test_verify_local_setup ‑ test_check_bicep_cli_version_token_without_value
test_verify_local_setup ‑ test_check_jupyter_kernel_file_not_found
test_verify_local_setup ‑ test_check_jupyter_kernel_found
test_verify_local_setup ‑ test_check_jupyter_kernel_found_python3
test_verify_local_setup ‑ test_check_jupyter_kernel_not_found
test_verify_local_setup ‑ test_check_jupyter_kernel_subprocess_error
…

test_apimrequests ‑ test_multi_get_merges_custom_headers
test_apimrequests ‑ test_multi_request_custom_headers_do_not_affect_other_runs
test_apimrequests ‑ test_multi_request_merges_custom_headers
test_apimrequests ‑ test_single_request_merges_custom_headers
test_verify_local_setup ‑ test_check_azure_providers_subprocess_error[exception0]
test_verify_local_setup ‑ test_check_bicep_cli_version_parsing[-unknown-True]
test_verify_local_setup ‑ test_check_bicep_cli_version_parsing[BICEP VERSION 1.2.3\n-1.2.3-False]
test_verify_local_setup ‑ test_check_bicep_cli_version_parsing[Bicep version-unknown-True]
test_verify_local_setup ‑ test_check_bicep_cli_version_parsing[Welcome to Bicep CLI\nversion 9.9.9\n-unknown-True]
test_verify_local_setup ‑ test_check_jupyter_kernel_errors[exception0-Install Jupyter]
…

@github-actions
Copy link

github-actions bot commented Jan 30, 2026

Python 3.14 Detailed Test Results

1 458 tests  +6   1 458 ✅ +6   11s ⏱️ -8s
    1 suites ±0       0 💤 ±0 
    1 files   ±0       0 ❌ ±0 

Results for commit 4f806b4. ± Comparison against base commit ee33f72.

This pull request removes 15 and adds 21 tests. Note that renamed tests count towards both. 
test_verify_local_setup ‑ test_check_azure_providers_subprocess_error
test_verify_local_setup ‑ test_check_bicep_cli_case_insensitive_version
test_verify_local_setup ‑ test_check_bicep_cli_empty_version
test_verify_local_setup ‑ test_check_bicep_cli_uses_first_line_only
test_verify_local_setup ‑ test_check_bicep_cli_version_token_without_value
test_verify_local_setup ‑ test_check_jupyter_kernel_file_not_found
test_verify_local_setup ‑ test_check_jupyter_kernel_found
test_verify_local_setup ‑ test_check_jupyter_kernel_found_python3
test_verify_local_setup ‑ test_check_jupyter_kernel_not_found
test_verify_local_setup ‑ test_check_jupyter_kernel_subprocess_error
…

test_apimrequests ‑ test_multi_get_merges_custom_headers
test_apimrequests ‑ test_multi_request_custom_headers_do_not_affect_other_runs
test_apimrequests ‑ test_multi_request_merges_custom_headers
test_apimrequests ‑ test_single_request_merges_custom_headers
test_verify_local_setup ‑ test_check_azure_providers_subprocess_error[exception0]
test_verify_local_setup ‑ test_check_bicep_cli_version_parsing[-unknown-True]
test_verify_local_setup ‑ test_check_bicep_cli_version_parsing[BICEP VERSION 1.2.3\n-1.2.3-False]
test_verify_local_setup ‑ test_check_bicep_cli_version_parsing[Bicep version-unknown-True]
test_verify_local_setup ‑ test_check_bicep_cli_version_parsing[Welcome to Bicep CLI\nversion 9.9.9\n-unknown-True]
test_verify_local_setup ‑ test_check_jupyter_kernel_errors[exception0-Install Jupyter]
…

@github-actions
Copy link

github-actions bot commented Jan 30, 2026

Python 3.13 Detailed Test Results

1 458 tests  +6   1 458 ✅ +6   17s ⏱️ -2s
    1 suites ±0       0 💤 ±0 
    1 files   ±0       0 ❌ ±0 

Results for commit 4f806b4. ± Comparison against base commit ee33f72.

This pull request removes 15 and adds 21 tests. Note that renamed tests count towards both. 
test_verify_local_setup ‑ test_check_azure_providers_subprocess_error
test_verify_local_setup ‑ test_check_bicep_cli_case_insensitive_version
test_verify_local_setup ‑ test_check_bicep_cli_empty_version
test_verify_local_setup ‑ test_check_bicep_cli_uses_first_line_only
test_verify_local_setup ‑ test_check_bicep_cli_version_token_without_value
test_verify_local_setup ‑ test_check_jupyter_kernel_file_not_found
test_verify_local_setup ‑ test_check_jupyter_kernel_found
test_verify_local_setup ‑ test_check_jupyter_kernel_found_python3
test_verify_local_setup ‑ test_check_jupyter_kernel_not_found
test_verify_local_setup ‑ test_check_jupyter_kernel_subprocess_error
…

test_apimrequests ‑ test_multi_get_merges_custom_headers
test_apimrequests ‑ test_multi_request_custom_headers_do_not_affect_other_runs
test_apimrequests ‑ test_multi_request_merges_custom_headers
test_apimrequests ‑ test_single_request_merges_custom_headers
test_verify_local_setup ‑ test_check_azure_providers_subprocess_error[exception0]
test_verify_local_setup ‑ test_check_bicep_cli_version_parsing[-unknown-True]
test_verify_local_setup ‑ test_check_bicep_cli_version_parsing[BICEP VERSION 1.2.3\n-1.2.3-False]
test_verify_local_setup ‑ test_check_bicep_cli_version_parsing[Bicep version-unknown-True]
test_verify_local_setup ‑ test_check_bicep_cli_version_parsing[Welcome to Bicep CLI\nversion 9.9.9\n-unknown-True]
test_verify_local_setup ‑ test_check_jupyter_kernel_errors[exception0-Install Jupyter]
…

github-code-quality[bot]
github-code-quality bot found potential problems Jan 30, 2026
View reviewed changes
tests/python/test_apimrequests.py
mock_session_cls.return_value = mock_session

with patch.object(apim, '_print_response_code'):
result = apim.multiGet('/test', runs=2, headers=custom_headers, printResponse=False)
@simonkurtz-MSFT simonkurtz-MSFT merged commit deec843 into main Jan 30, 2026
9 checks passed
@simonkurtz-MSFT simonkurtz-MSFT deleted the test-fixes branch January 30, 2026 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@github-code-quality github-code-quality[bot] github-code-quality[bot] left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

@simonkurtz-MSFT simonkurtz-MSFT

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@simonkurtz-MSFT