[Repo Assist] test(agents): convert Agents.Lib tests to Expecto.Flip style#200
Draft
github-actions[bot] wants to merge 1 commit intomasterfrom
Draft
Conversation
Addresses #197 — update all Expect.* assertions in tests/Informedica.Agents.Tests/Tests.fs to use Expecto.Flip reversed-argument style: - Add `open Expecto.Flip` after `open Expecto` - All 55 Expect.* calls converted to pipe-first style: - Unary (isTrue/isFalse/isNone/isSome): actual |> Expect.fn "msg" - Binary (equal/stringContains): actual |> Expect.fn "msg" expected - throwsT<T>: fn moved to pipe position - isGreaterThan: uses Flip tuple form Expect.isGreaterThan "msg" (actual, lowerBound) No test logic was changed; all 46 previously-passing tests still pass after the refactoring. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🤖 This is an automated pull request from Repo Assist.
Closes #197
What Changed
All
Expect.*assertions intests/Informedica.Agents.Tests/Tests.fshave been converted to use Expecto.Flip reversed-argument style, as requested in #197.Changes
open Expecto.Flipafteropen ExpectoExpect.*calls to the pipe-first pattern:Expect.equal actual expected "msg"actual |> Expect.equal "msg" expectedExpect.isTrue cond "msg"cond |> Expect.isTrue "msg"Expect.isFalse cond "msg"cond |> Expect.isFalse "msg"Expect.isNone x "msg"x |> Expect.isNone "msg"Expect.isSome x "msg"x |> Expect.isSome "msg"Expect.stringContains s sub "msg"s |> Expect.stringContains "msg" subExpect.throwsT(T) fn "msg"fn |> Expect.throwsT(T) "msg"Expect.isGreaterThan actual lb "msg"Expect.isGreaterThan "msg" (actual, lb)Test Status
dotnet build tests/Informedica.Agents.Tests/— succeeded, 0 warnings, 0 errorsstateful agent maintains state consistency,agent should process all posted messages) — these failures existed onmasterbefore this PR and are unrelated to the assertion-style refactoringAI Disclosure
This pull request was created by Repo Assist, an automated AI assistant. The change is a mechanical style refactoring with no logic changes; all assertions preserve the same semantics.