Skip to content

Add proper windows CI test coverage #249

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

Closed
dschep opened this issue Sep 17, 2018 · 5 comments · Fixed by #254
Closed

Add proper windows CI test coverage #249

dschep opened this issue Sep 17, 2018 · 5 comments · Fixed by #254
Labels

Comments

@dschep
Copy link
Contributor

dschep commented Sep 17, 2018

as #247 and #248 make obvious, testing on windows machines is not part of the usual release flow. This should change, but I'm not sure how to best go about running the bats test suite on windows or if it should be ported(not preferable, since then you have to maintain multiple test suites)

Also, it would be of interest to run this the various ways you can run it on windows (now of course, some of these make it much easier to use bats, eg when using WSL):

  • vanilla windows + docker for windows
  • vanilla windows + docker toolbox
  • WSL + "wsl" docker
  • WSL + docker for windows
  • cygwin + docker for windows
  • more? less? windows is so complicated.

CCing those I know to be windows-based users&contributors: @kichik @heri16 @vecchp @hubbins @ajshukury @LucasNP (others? If you know, plz tag them)

@hubbins
Copy link

hubbins commented Sep 17, 2018

IMHO, "windows + docker for windows" is going to cover almost all Windows users. I don't think there are many docker toolbox users anymore. WSL should essentially be covered by your existing tests, since there are no drive letters (/mnt/c and /mnt/d are used for C: and D: for example). If there are users that fall into the docker toolbox or cygwin buckets, they should speak up if they exist. Or focus on those users later.

@kichik
Copy link
Contributor

kichik commented Sep 17, 2018

I use Docker Toolbox. On one computer because I didn't want to pay extra for Windows Pro, and on another computer because my work requires me to have VirtualBox there. Docker for Windows is sadly still a limitation for some. I strongly believe we should support both.

@dschep
Copy link
Contributor Author

dschep commented Sep 18, 2018

@kichik, note, regardless of it being tested or not, docker toolbox won't be dropped. even windows with docker for windows tests benefits windows with docker toolbox 😄

@hubbins
Copy link

hubbins commented Sep 18, 2018

@kichik I'm just making a suggestion around prioritization, not whether to support or not.

@heri16
Copy link
Contributor

heri16 commented Sep 18, 2018

"windows is so complicated" - I think people perceive this is simply because of the lack of documentation out there that explains the architecture of POSIX on windows.

This was referenced Sep 19, 2018
@dschep dschep added the Windows label Sep 20, 2018
dschep added a commit that referenced this issue Sep 20, 2018
Main advantage is that these can run on windows more easily, thus fixing #249
dschep added a commit that referenced this issue Sep 20, 2018
Main advantage is that these can run on windows more easily, thus fixing #249
dschep added a commit that referenced this issue Oct 11, 2018
Main advantage is that these can run on windows more easily, thus fixing #249
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants