Skip to content
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

Struggling with tasty-golden under Windows 10 (2004) #42

Closed
jrp2014 opened this issue Jun 21, 2020 · 7 comments
Closed

Struggling with tasty-golden under Windows 10 (2004) #42

jrp2014 opened this issue Jun 21, 2020 · 7 comments

Comments

@jrp2014
Copy link
Contributor

jrp2014 commented Jun 21, 2020

I suggest that, to improve the reliability of testing under Windows, you could

  • Bump the limits for System.Process to 1.6.8.1 (Fix job support haskell/process#168) and set use_process_jobs=True when constructing the diff process
  • Mention that the diff process should not call exec as the ExitCode will be unreliable. (This is a pity as I suspect that git diff probably does this.)
  • Mention that the test harness should be compiled -threaded

(I am having stability problems with a test suite on Windows that don't occur on Linux and MacOS.)

@UnkindPartition
Copy link
Owner

No objections from me — could you send a PR? Ideally with explanations of what each of these does/why they are needed (in the docs for doc changes and in the comments for code changes.)

@jrp2014
Copy link
Contributor Author

jrp2014 commented Jun 22, 2020

Will do. When you say "docs" what are you referring to?

@UnkindPartition
Copy link
Owner

I mean your points (2) and (3). E.g. when mentioning that the test harness should be compiled -threaded, please also add an explanation on why that's needed (and whether it applies to Windows only).

@jrp2014
Copy link
Contributor Author

jrp2014 commented Jun 22, 2020

Ok, but where should this go? Is there some user-facing documentation somewhere?

@UnkindPartition
Copy link
Owner

Yes, in the module-level haddocks. There's already some advice there about file names, line endings etc.: https://hackage.haskell.org/package/tasty-golden-2.3.4/docs/Test-Tasty-Golden.html

@UnkindPartition
Copy link
Owner

Maybe it could also benefit from some headers to establish a bit of structure.

@jrp2014
Copy link
Contributor Author

jrp2014 commented Jun 22, 2020

Sure. As a side-note. probably a separate issue, there is the question of ctrl-c handling, and whether it should be delegated to the child process.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants