Skip to content

Reorganize tests#113

Merged
ChenZhao44 merged 15 commits intomasterfrom
chen/reorganize-tests
Oct 22, 2025
Merged

Reorganize tests#113
ChenZhao44 merged 15 commits intomasterfrom
chen/reorganize-tests

Conversation

@ChenZhao44
Copy link
Member

No description provided.

@ChenZhao44 ChenZhao44 marked this pull request as ready for review October 22, 2025 03:03
@ChenZhao44 ChenZhao44 requested a review from Copilot October 22, 2025 03:04
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR reorganizes the test suite structure by introducing module-based subdirectories, standardizing code formatting, and updating CI configuration.

Key Changes

  • Test files reorganized into module-specific subdirectories (ZX, Utils, ZXW, PMG, ZW, Application)
  • Whitespace cleanup: removed trailing blank lines and unnecessary line breaks
  • Julia syntax modernization: replaced deprecated = with in in for loops and comprehensions
  • Removed nightly Julia version from CI workflow

Reviewed Changes

Copilot reviewed 17 out of 24 changed files in this pull request and generated no comments.

Show a summary per file
File Description
test/runtests.jl Updated test organization with module-based testset hierarchy and new file paths
test/ZXW/zxw_rules.jl Removed blank lines and reformatted multi-line assignments
test/ZXW/zxw_diagram.jl Cleaned up whitespace and changed loop syntax from = to in
test/ZXW/utils.jl Removed blank lines, fixed trailing commas, adjusted spacing in slicing expressions
test/ZX/zx_graph.jl Changed loop syntax from = to in and removed blank line
test/ZX/zx_diagram.jl Changed loop comprehension syntax from to in
test/ZX/rules.jl Changed loop syntax from = to in and removed trailing commas
test/ZX/plots.jl Removed blank line
test/ZX/ir.jl Cleaned up whitespace, reformatted multi-line assignments, changed loop syntax
test/ZX/challenge.jl Changed loop syntax from = to in and removed trailing commas
test/ZX/abstract_zx_diagram.jl Removed blank line
test/ZW/zw_utils.jl Removed blank lines, fixed trailing commas, adjusted spacing
test/ZW/zw_diagram.jl Removed blank lines and trailing commas
test/Utils/parameter.jl Removed blank lines
test/PMG/planar_multigraph.jl Removed blank lines and trailing commas throughout
test/Application/to_eincode.jl Reformatted multi-line array literal to single line
.github/workflows/CI.yml Removed nightly Julia version from test matrix

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@ChenZhao44 ChenZhao44 merged commit 5823be3 into master Oct 22, 2025
8 checks passed
@ChenZhao44 ChenZhao44 deleted the chen/reorganize-tests branch October 22, 2025 03:08
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.

1 participant