Skip to content

[tests] remove test_pipelines_common.py - #14669

Open
sayakpaul wants to merge 19 commits into
mainfrom
remove-test-pipelines-common
Open

sayakpaul wants to merge 19 commits into
mainfrom
remove-test-pipelines-common

Conversation

@sayakpaul

@sayakpaul sayakpaul commented Sep 1, 2026 •

Copy link
Copy Markdown
Member
  • Move FluxIPAdapterTesterMixin to the IP Adapter testing utils since it's shared by more than Flux.
  • Migrate old imports in the test suites of DeepFloyd IF, etc. to use new ones.
  • Migrate the Stable Diffusion Adapter test suite to use new mixins.
  • Remove test_pipelines_common.py.

@github-actions github-actions Bot added tests size/L PR with diff > 200 LOC labels Sep 1, 2026
# support `"pt"`; adding that here is a `src/` change and out of scope for this test migration, so the affected
# tests are marked `xfail` rather than skipped: whoever adds the `"pt"` branch will see them XPASS and can drop
# these markers.
NO_PT_OUTPUT = pytest.mark.xfail(

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Open to other ideas but I chose to xfail these tests and not introduce anything really new to our shared tester mixins.

@sayakpaul

Copy link
Copy Markdown
Member Author

/diffusers-bot pytest tests/pipelines/stable_diffusion_adapter/test_stable_diffusion_adapter.py

@github-actions

github-actions Bot commented Sep 1, 2026 •

Copy link
Copy Markdown
Contributor

✅ pytest tests/pipelines/stable_diffusion_adapter/test_stable_diffusion_adapter.py passed on GPU — view logs.

Comment thread tests/pipelines/testing_utils/memory.py Outdated
Comment thread tests/pipelines/kandinsky/test_kandinsky_prior.py Outdated
@sayakpaul

Copy link
Copy Markdown
Member Author

/diffusers-bot pytest tests/pipelines/stable_diffusion_2/test_stable_diffusion_depth.py::TestStableDiffusionDepth2ImgPipelineMemory::test_group_offloading_inference

@github-actions

github-actions Bot commented Sep 1, 2026 •

Copy link
Copy Markdown
Contributor

⚠️ pytest tests/pipelines/stable_diffusion_2/test_stable_diffusion_depth.py::TestStableDiffusionDepth2ImgPipelineMemory::test_group_offloading_inference was cancelled on GPU — view logs.

@sayakpaul

Copy link
Copy Markdown
Member Author

/diffusers-bot pytest tests/pipelines/consisid/test_consisid.py tests/pipelines/stable_audio/test_stable_audio.py

@github-actions

github-actions Bot commented Sep 1, 2026 •

Copy link
Copy Markdown
Contributor

✅ pytest tests/pipelines/consisid/test_consisid.py tests/pipelines/stable_audio/test_stable_audio.py passed on GPU — view logs.

@sayakpaul

Copy link
Copy Markdown
Member Author

/diffusers-bot pytest tests/pipelines/stable_diffusion_2/test_stable_diffusion_depth.py::TestStableDiffusionDepth2ImgPipelineMemory::test_group_offloading_inference

@github-actions

github-actions Bot commented Sep 1, 2026 •

Copy link
Copy Markdown
Contributor

✅ pytest tests/pipelines/stable_diffusion_2/test_stable_diffusion_depth.py::TestStableDiffusionDepth2ImgPipelineMemory::test_group_offloading_inference passed on GPU — view logs.

@github-actions github-actions Bot added the CI label Sep 1, 2026
pytest -n 1 --max-worker-restart=0 --dist=loadfile \
--make-reports=tests_torch_minimum_version_cuda \
tests/models/test_modeling_common.py \
tests/pipelines/test_pipelines_common.py \

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No longer exists.

Comment on lines -17 to +19
- "tests/pipelines/test_pipelines_common.py"
- "tests/pipelines/testing_utils/**.py"
- "tests/models/test_modeling_common.py"
- "tests/models/testing_utils/**.py"

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To have parity in our CI when test changes happen.

pytest -n 1 --max-worker-restart=0 --dist=loadfile \
--make-reports=tests_torch_minimum_cuda \
tests/models/test_modeling_common.py \
tests/pipelines/test_pipelines_common.py \

@sayakpaul sayakpaul Sep 3, 2026 •

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No longer exists.

)


class AdapterPipelineTesterConfig(BasePipelineTesterConfig):

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The remaining test suite being refactored.

@sayakpaul

Copy link
Copy Markdown
Member Author

/diffusers-bot pytest tests/pipelines/kandinsky/ tests/pipelines/kandinsky2_2/

@github-actions

github-actions Bot commented Sep 3, 2026 •

Copy link
Copy Markdown
Contributor

✅ pytest tests/pipelines/kandinsky/ tests/pipelines/kandinsky2_2/ passed on GPU — view logs.

Comment thread tests/pipelines/testing_utils/ip_adapter.py Outdated
Comment thread tests/pipelines/flux/testing_utils.py
@sayakpaul
sayakpaul requested a review from DN6 September 11, 2026 05:28
@hf-security-analysis

hf-security-analysis Bot commented Sep 16, 2026 •

Copy link
Copy Markdown
Contributor

⚠️ Workflow security review — 20 high, 15 medium

Scanned 3 workflow file(s) with zizmor, pinact, OSV/GHSA, Claude, in full.

.github/workflows/nightly_tests.yml

  • ⚠️ HIGH unpinned-images — .github/workflows/nightly_tests.yml:34 (via zizmor)
    unpinned image references — container image is unpinned

    Fix: unpinned image references

  • ⚠️ HIGH unpinned-images — .github/workflows/nightly_tests.yml:71 (via zizmor)
    unpinned image references — container image is unpinned

    Fix: unpinned image references

  • ⚠️ HIGH unpinned-images — .github/workflows/nightly_tests.yml:117 (via zizmor)
    unpinned image references — container image is unpinned

    Fix: unpinned image references

  • ⚠️ HIGH unpinned-images — .github/workflows/nightly_tests.yml:189 (via zizmor)
    unpinned image references — container image is unpinned

    Fix: unpinned image references

  • ⚠️ HIGH unpinned-images — .github/workflows/nightly_tests.yml:233 (via zizmor)
    unpinned image references — container image is unpinned

    Fix: unpinned image references

  • ⚠️ HIGH unpinned-images — .github/workflows/nightly_tests.yml:282 (via zizmor)
    unpinned image references — container image is unpinned

    Fix: unpinned image references

  • ⚠️ HIGH unpinned-images — .github/workflows/nightly_tests.yml:355 (via zizmor)
    unpinned image references — container image is unpinned

    Fix: unpinned image references

  • ⚠️ HIGH unpinned-images — .github/workflows/nightly_tests.yml:411 (via zizmor)
    unpinned image references — container image is unpinned

    Fix: unpinned image references

  • ⚠️ HIGH unpinned-images — .github/workflows/nightly_tests.yml:469 (via zizmor)
    unpinned image references — container image is unpinned

    Fix: unpinned image references

  • 🟡 MEDIUM template-injection — .github/workflows/nightly_tests.yml:97 (via zizmor)
    code injection via template expansion — may expand into attacker-controllable code

    Fix: Map the expression into the step's env: block and reference the shell variable instead of interpolating it into run:.

  • 🟡 MEDIUM template-injection — .github/workflows/nightly_tests.yml:98 (via zizmor)
    code injection via template expansion — may expand into attacker-controllable code

    Fix: Map the expression into the step's env: block and reference the shell variable instead of interpolating it into run:.

  • 🟡 MEDIUM template-injection — .github/workflows/nightly_tests.yml:99 (via zizmor)
    code injection via template expansion — may expand into attacker-controllable code

    Fix: Map the expression into the step's env: block and reference the shell variable instead of interpolating it into run:.

  • 🟡 MEDIUM template-injection — .github/workflows/nightly_tests.yml:103 (via zizmor)
    code injection via template expansion — may expand into attacker-controllable code

    Fix: Map the expression into the step's env: block and reference the shell variable instead of interpolating it into run:.

  • 🟡 MEDIUM template-injection — .github/workflows/nightly_tests.yml:104 (via zizmor)
    code injection via template expansion — may expand into attacker-controllable code

    Fix: Map the expression into the step's env: block and reference the shell variable instead of interpolating it into run:.

.github/workflows/pr_tests_gpu.yml

  • ⚠️ HIGH unpinned-images — .github/workflows/pr_tests_gpu.yml:91 (via zizmor)
    unpinned image references — container image uses the floating 'latest' tag

    Fix: unpinned image references

  • ⚠️ HIGH unpinned-images — .github/workflows/pr_tests_gpu.yml:130 (via zizmor)
    unpinned image references — container image uses the floating 'latest' tag

    Fix: unpinned image references

  • ⚠️ HIGH unpinned-images — .github/workflows/pr_tests_gpu.yml:194 (via zizmor)
    unpinned image references — container image uses the floating 'latest' tag

    Fix: unpinned image references

  • ⚠️ HIGH unpinned-images — .github/workflows/pr_tests_gpu.yml:264 (via zizmor)
    unpinned image references — container image uses the floating 'latest' tag

    Fix: unpinned image references

  • 🟡 MEDIUM template-injection — .github/workflows/pr_tests_gpu.yml:164 (via zizmor)
    code injection via template expansion — may expand into attacker-controllable code

    Fix: Map the expression into the step's env: block and reference the shell variable instead of interpolating it into run:.

  • 🟡 MEDIUM template-injection — .github/workflows/pr_tests_gpu.yml:166 (via zizmor)
    code injection via template expansion — may expand into attacker-controllable code

    Fix: Map the expression into the step's env: block and reference the shell variable instead of interpolating it into run:.

  • 🟡 MEDIUM template-injection — .github/workflows/pr_tests_gpu.yml:167 (via zizmor)
    code injection via template expansion — may expand into attacker-controllable code

    Fix: Map the expression into the step's env: block and reference the shell variable instead of interpolating it into run:.

  • 🟡 MEDIUM template-injection — .github/workflows/pr_tests_gpu.yml:172 (via zizmor)
    code injection via template expansion — may expand into attacker-controllable code

    Fix: Map the expression into the step's env: block and reference the shell variable instead of interpolating it into run:.

  • 🟡 MEDIUM template-injection — .github/workflows/pr_tests_gpu.yml:179 (via zizmor)
    code injection via template expansion — may expand into attacker-controllable code

    Fix: Map the expression into the step's env: block and reference the shell variable instead of interpolating it into run:.

  • 🟡 MEDIUM template-injection — .github/workflows/pr_tests_gpu.yml:180 (via zizmor)
    code injection via template expansion — may expand into attacker-controllable code

    Fix: Map the expression into the step's env: block and reference the shell variable instead of interpolating it into run:.

.github/workflows/release_tests_fast.yml

  • ⚠️ HIGH unpinned-images — .github/workflows/release_tests_fast.yml:34 (via zizmor)
    unpinned image references — container image is unpinned

    Fix: unpinned image references

  • ⚠️ HIGH unpinned-images — .github/workflows/release_tests_fast.yml:74 (via zizmor)
    unpinned image references — container image is unpinned

    Fix: unpinned image references

  • ⚠️ HIGH unpinned-images — .github/workflows/release_tests_fast.yml:119 (via zizmor)
    unpinned image references — container image is unpinned

    Fix: unpinned image references

  • ⚠️ HIGH unpinned-images — .github/workflows/release_tests_fast.yml:176 (via zizmor)
    unpinned image references — container image is unpinned

    Fix: unpinned image references

  • ⚠️ HIGH unpinned-images — .github/workflows/release_tests_fast.yml:234 (via zizmor)
    unpinned image references — container image is unpinned

    Fix: unpinned image references

  • ⚠️ HIGH unpinned-images — .github/workflows/release_tests_fast.yml:277 (via zizmor)
    unpinned image references — container image is unpinned

    Fix: unpinned image references

  • ⚠️ HIGH unpinned-images — .github/workflows/release_tests_fast.yml:320 (via zizmor)
    unpinned image references — container image is unpinned

    Fix: unpinned image references

  • 🟡 MEDIUM template-injection — .github/workflows/release_tests_fast.yml:100 (via zizmor)
    code injection via template expansion — may expand into attacker-controllable code

    Fix: Map the expression into the step's env: block and reference the shell variable instead of interpolating it into run:.

  • 🟡 MEDIUM template-injection — .github/workflows/release_tests_fast.yml:101 (via zizmor)
    code injection via template expansion — may expand into attacker-controllable code

    Fix: Map the expression into the step's env: block and reference the shell variable instead of interpolating it into run:.

  • 🟡 MEDIUM template-injection — .github/workflows/release_tests_fast.yml:105 (via zizmor)
    code injection via template expansion — may expand into attacker-controllable code

    Fix: Map the expression into the step's env: block and reference the shell variable instead of interpolating it into run:.

  • 🟡 MEDIUM template-injection — .github/workflows/release_tests_fast.yml:106 (via zizmor)
    code injection via template expansion — may expand into attacker-controllable code

    Fix: Map the expression into the step's env: block and reference the shell variable instead of interpolating it into run:.

ℹ️ No change could be applied automatically.

Pinning and static findings are deterministic. Findings marked (via Claude) are model judgements — check them before acting.

@sayakpaul sayakpaul added this to the Release 0.41.0 milestone Sep 30, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI size/L PR with diff > 200 LOC tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants