Skip to content

refactor: Clean up the /tests in the root #26525

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
hbjORbj merged 2 commits into from
Jan 7, 2026
Merged

refactor: Clean up the /tests in the root #26525

hbjORbj merged 2 commits into from
Jan 7, 2026

Conversation

@hbjORbj
Copy link
Contributor

@hbjORbj hbjORbj commented Jan 7, 2026 •
edited
Loading

What does this PR do?

Removed redundant test helpers from /tests at the root level as they are not consumed anywhere

Mandatory Tasks (DO NOT REMOVE)

  • I have self-reviewed the code (A decent size PR without self-review might be rejected).
  • N/A - I have updated the developer docs in /docs if this PR makes changes that would require a documentation change. If N/A, write N/A here and check the checkbox.
  • I confirm automated tests are in place that prove my fix is effective or that my feature works.

How should this be tested?

  • CI jobs are sufficient

@vercel
Copy link

vercel bot commented Jan 7, 2026 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for GitHub.

3 Skipped Deployments
Project Deployment Review Updated (UTC)
cal Ignored Ignored Jan 7, 2026 6:18am
cal-companion Ignored Ignored Preview Jan 7, 2026 6:18am
cal-eu Ignored Ignored Jan 7, 2026 6:18am

@graphite-app graphite-app bot added foundation core area: core, team members only labels Jan 7, 2026
@graphite-app graphite-app bot requested a review from a team January 7, 2026 06:11
@hbjORbj hbjORbj marked this pull request as draft January 7, 2026 06:11
@hbjORbj hbjORbj changed the title chore: remove redundant files in tests at root refactor: Clean up the /tests in the root Jan 7, 2026
cubic-dev-ai[bot]
cubic-dev-ai bot reviewed Jan 7, 2026
View reviewed changes
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 3 files

@hbjORbj hbjORbj marked this pull request as ready for review January 7, 2026 06:17
@hbjORbj hbjORbj enabled auto-merge (squash) January 7, 2026 06:18
cubic-dev-ai[bot]
cubic-dev-ai bot reviewed Jan 7, 2026
View reviewed changes
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 5 files

@hbjORbj hbjORbj requested a review from keithwillcode January 7, 2026 09:55
@hbjORbj hbjORbj merged commit 6d2491b into main Jan 7, 2026
71 of 72 checks passed
@hbjORbj hbjORbj deleted the chore/remove-redundant-files-in-tests-at-root branch January 7, 2026 10:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cubic-dev-ai cubic-dev-ai[bot] cubic-dev-ai[bot] left review comments

@keithwillcode keithwillcode keithwillcode approved these changes

Assignees

No one assigned

Labels

core area: core, team members only foundation ready-for-e2e size/L

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@hbjORbj @keithwillcode