Skip to content

fix: correct isort import ordering in test files to unblock run-lint CI#1052

Draft
gastori with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-run-lint-failure
Draft

fix: correct isort import ordering in test files to unblock run-lint CI#1052
gastori with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-run-lint-failure

Conversation

Copilot AI commented Jul 9, 2026

Copy link
Copy Markdown

The run-lint CI job was failing because isort flagged incorrect import ordering in 8 test files. The linter treats all non-stdlib imports as a single alphabetically-sorted section with no blank-line separators between third-party and local modules.

Changes

  • test_activities/test_gather_enrichment.py — move import activities.* before import httpx; drop blank line separator
  • test_activities/test_resolve_owner.py — same pattern as above for activities.resolve_owner
  • test_clients/test_arg_client.py — move import clients.arg_client before import httpx; drop blank line separator
  • test_clients/test_ado_client.py — drop blank line between import respx and from clients.*
  • test_clients/test_graph_client.py — drop blank line between import respx and from clients.*
  • test_clients/test_mdc_client.py — drop blank line between import respx and from clients.*
  • test_function_app.py — drop blank line between import azure.functions and import function_app
  • test_models/test_mdc_payload.py — drop blank line between import pytest and from models.*

Before / after example:

# Before (isort error: blank line separates same-section imports)
import httpx
import pytest
import respx

import activities.gather_enrichment as gather

# After (single alphabetically-sorted block)
import activities.gather_enrichment as gather
import httpx
import pytest
import respx

Copilot AI changed the title [WIP] Fix failing GitHub Actions job run-lint fix: correct isort import ordering in test files to unblock run-lint CI Jul 9, 2026
Copilot AI requested a review from gastori July 9, 2026 17:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants