Skip to content

Shrink relationComplexityError test size - #62754

Merged
Jake Bailey (jakebailey) merged 1 commit into
microsoft:mainfrom
jakebailey:relationComplexityErrorSmaller
Nov 13, 2025
Merged

Shrink relationComplexityError test size#62754
Jake Bailey (jakebailey) merged 1 commit into
microsoft:mainfrom
jakebailey:relationComplexityErrorSmaller

Conversation

@jakebailey

Copy link
Copy Markdown
Member

Getting tired of this test timing out on slower builders.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 reduces the size of the relationComplexityError compiler test to prevent timeouts on slower build machines. By reducing the Digits type from 10 literal types ('0'-'9') to 8 literal types ('0'-'7'), the number of type combinations drops from 10,000 to 4,096, significantly improving test performance while still reproducing the relation complexity error from issue #55630.

Key Changes

  • Reduced Digits type from 10 literals to 8 literals, decreasing type combinations from 10^4 to 8^4
  • Updated all baseline files to reflect the reduced Type Count (from 25,000 to 10,000) and adjusted symbol positions

Reviewed Changes

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

File Description
tests/cases/compiler/relationComplexityError.ts Removed '8' and '9' from the Digits type to reduce test complexity
tests/baselines/reference/relationComplexityError.types Updated type count from 25,000 to 10,000 and refreshed type baselines
tests/baselines/reference/relationComplexityError.symbols Updated symbol declaration positions to reflect shorter Digits type definition
tests/baselines/reference/relationComplexityError.errors.txt Updated error baseline to reflect the reduced Digits type

@jakebailey
Jake Bailey (jakebailey) merged commit ea48ded into microsoft:main Nov 13, 2025
36 of 37 checks passed
@github-project-automation github-project-automation Bot moved this from Needs merge to Done in PR Backlog Nov 13, 2025
@microsoft Microsoft (microsoft) locked as resolved and limited conversation to collaborators May 14, 2026
@jakebailey
Jake Bailey (jakebailey) deleted the relationComplexityErrorSmaller branch August 11, 2026 20:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Author: Team For Uncommitted Bug PR for untriaged, rejected, closed or missing bug

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants