Skip to content

Conversation

@amoeba
Copy link
Member

@amoeba amoeba commented Sep 20, 2025

I saw while writing docs that we don't have an explicit test of the precedence chain we document so I created one. However, it current fails in a very strange way for me so this PR adds a failing test.

@amoeba amoeba requested a review from zeroshade September 20, 2025 22:27
@amoeba
Copy link
Member Author

amoeba commented Sep 20, 2025

This test fails early on and when I poke around the suite.tempdir, I see a situation that looks like mangled paths?

$ tree
.
├── driver_path
│   └── test-driver-1.1
│       ├── test-driver-1-not-valid.so
│       └── test-driver-1-not-valid.so.sig
└── driver_path:
    └── tmp
        └── TestSubcommandsTestInstallEnvironmentPrecedence1409964535
            └── 001
                └── venv_path
                    └── etc
                        └── adbc
                            └── drivers:
                                └── tmp
                                    └── TestSubcommandsTestInstallEnvironmentPrecedence1409964535
                                        └── 001
                                            └── conda_path
                                                └── etc
                                                    └── adbc
                                                        └── drivers
                                                            └── test-driver-1.toml

@amoeba amoeba merged commit de9c4f0 into columnar-tech:main Sep 22, 2025
7 checks passed
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.

2 participants