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

Test runner fails in container/website for Beauty Salon Goes Global #62

Closed
msdeibel opened this issue Sep 13, 2021 · 2 comments · Fixed by #63
Closed

Test runner fails in container/website for Beauty Salon Goes Global #62

msdeibel opened this issue Sep 13, 2021 · 2 comments · Fixed by #63

Comments

@msdeibel
Copy link

msdeibel commented Sep 13, 2021

A mentee left the following message with their submission on the exercise.

The container that the tests run in doesn't appear to be returning any timezones from TimeZoneInfo.GetSystemTimeZones(). The tests run locally both on Linux and Windows so I'm a bit confused here. Is there some other config item that needs to be set in order for the time zones to be found or is this an environment issue with the tests being run in a container?

I have asked the member to add any additional info they might have on the issue.

@dgee2
Copy link
Contributor

dgee2 commented Sep 13, 2021

For further information, this was on the C# track. Link to the exercise is https://exercism.org/tracks/csharp/exercises/beauty-salon-goes-global

I won't post my solution immediately to avoid dropping a solution here but can add it if it helps solve this issue.

After my own investigation, TimeZoneInfo.GetSystemTimeZones() returns an empty IEnumerable. I have not dug into the container that runs the tests yet but it looks likely that it is similar to this announcement on the dotnet docker images here: dotnet/dotnet-docker#1366 For brevity I arrived at that from this Stack Overflow answer: https://stackoverflow.com/a/68120062/3697637

@dgee2
Copy link
Contributor

dgee2 commented Sep 13, 2021

I have added a PR that seems to fix this in the Dockerfile. Hopefully, this is helpful.

Additional dotnet-docker issues that I have found for background info:
dotnet/dotnet-docker#1366
dotnet/dotnet-docker#1264

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

Successfully merging a pull request may close this issue.

2 participants