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

Tests with extension .tt #535

Open
elliottslaughter opened this issue Apr 20, 2022 · 1 comment
Open

Tests with extension .tt #535

elliottslaughter opened this issue Apr 20, 2022 · 1 comment

Comments

@elliottslaughter
Copy link
Member

I just noticed there are two tests with the extension .tt. I'm not sure what the intention was, but I'm guessing these aren't being run right now.

$ ls ../tests/*.tt
../tests/emptytag.tt	../tests/tag.tt
@elliottslaughter
Copy link
Member Author

Looking in more depth, these are both tests that use #!../terra. This obviously does not work if Terra is not located precisely at ../terra. In fact there is a commit 0540bdd that seems to indicate as much. This is even more true with CMake, where the build directory could be located anywhere.

The only way I see to fix it is to use /usr/bin/env terra and make sure the Terra binary is on PATH. But I'm not really sure if it's worth the trouble.

Overall I don't think we're losing much by not having these tests.

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

1 participant