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

Re-enable Firebird tests (Issue178_Firebird) #1917

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

0xced
Copy link
Contributor

@0xced 0xced commented Jun 13, 2023

No description provided.

@0xced
Copy link
Contributor Author

0xced commented Jun 13, 2023

Well, it looks like running Firebird in a docker container on AppVeyor is not as easy as it ought to be. 😣

Downloading and configuring and running Firebird could also be a possibility, but that's quite a lot of code.

@mgravell
Copy link
Member

mgravell commented Jun 13, 2023

I'm fine with it, but appveyor says "nope"

specifically it says:

docker: error during connect: Post http://%2F%2F.%2Fpipe%2Fdocker_engine/v1.40/containers/create?name=Dapper.Tests.Firebird: open //./pipe/docker_engine: The system cannot find the file specified. In the default daemon configuration on Windows, the docker client must be run elevated to connect. This error may also indicate that the docker daemon is not running.

@0xced
Copy link
Contributor Author

0xced commented Jun 13, 2023

Unfortunately I'm not familiar enough with AppVeyor.

There's this //./pipe/docker_engine access denied? discussion on the AppVeyor forum but it doesn't seem to really offer a solution.

Alternatively, Firebird could probably be downloaded, configured and executed with scripts, like nhibernate-core does. But that's too much YAML and PowerShell for me. 😅

@mgravell
Copy link
Member

Me too :)

I think this is one of those things where if it is going to fight us, it is going to stay disabled (unless @NickCraver has some magic CI-jitsu to employ)

@NickCraver
Copy link
Member

Indeed this isn't something we can really pull in unless the platform already runs the server, we just don't have the test environment from it. Even in say GitHub Actions we have another version of the problem: we can't build and test .NET Framework (Windows) because service containers can only run on Ubuntu. Clients testing against server bits isn't something well supported on most test platforms, unfortunately :(

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 this pull request may close these issues.

3 participants