-
Notifications
You must be signed in to change notification settings - Fork 587
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.
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
Expecto helper #1435
Expecto helper #1435
Conversation
@@ -0,0 +1,91 @@ | |||
[<AutoOpen>] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please remove autoopen - we don't use that for new helpers anymore
|> appendList this.Run "--run " | ||
|> toText | ||
|
||
static member defaultParams = |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please use DefaultParams
let Expecto (setParams : ExpectoParams -> ExpectoParams) (assemblies : string seq) = | ||
let args = setParams ExpectoParams.defaultParams | ||
let argsString = string args | ||
let runAssembly testAssembly = |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please trace the taskstart like in https://github.com/fsharp/FAKE/blob/master/src/app/FakeLib/DocFxHelper.fs#L45
awesome! |
Hi, this is a helper proposal for running unit tests with expecto. No extra dependencies to FAKE, the tests are run as executables. Please look at what could be improved