Skip to content

[Repo Assist] Add tests for begin..end expressions and auto-property edge cases#3299

Draft
github-actions[bot] wants to merge 1 commit intomainfrom
repo-assist/improve-tests-begin-end-auto-props-eb4d56ac82265636
Draft

[Repo Assist] Add tests for begin..end expressions and auto-property edge cases#3299
github-actions[bot] wants to merge 1 commit intomainfrom
repo-assist/improve-tests-begin-end-auto-props-eb4d56ac82265636

Conversation

@github-actions
Copy link
Contributor

🤖 This is an automated PR from Repo Assist.

Summary

Adds 6 new tests to two under-covered test files to improve test suite coverage and document expected formatting behaviour.

BeginEndTests.fs (was 2 tests → now 6)

Test Scenario
begin end in value binding begin..end in a let-binding value
begin end in function body begin..end as a function body result
begin end in match branch begin..end in each arm of a match expression
begin end with comment inside begin..end block containing a line comment

AutoPropertiesTests.fs (was 4 tests → now 6)

Test Scenario
auto-property with attribute [(DefaultValue)] attribute on the preceding line of a member val
static auto-property with explicit type static member val with an explicit type annotation

Rationale

Both files had very few tests (2 and 4 respectively). These additions document common F# constructs that were not explicitly covered and act as regression guards for those formatting paths.

Test Status

  • ✅ Build: succeeded (0 warnings, 0 errors)
  • ✅ Tests: 2745 passed, 7 skipped, 0 failed
  • ✅ All 6 new tests pass
  • ✅ Source files already formatted correctly (no fantomas changes needed)

Generated by Repo Assist ·

To install this agentic workflow, run

gh aw add githubnext/agentics/workflows/repo-assist.md@346204513ecfa08b81566450d7d599556807389f

- BeginEndTests.fs: add 4 new tests covering value bindings, function
  bodies, match branches, and comments inside begin..end blocks
- AutoPropertiesTests.fs: add 2 new tests covering attributes on
  auto-properties and static auto-properties with explicit types

All 2745 existing tests continue to pass.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants