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 names pollute student namespace #15

Closed
petertseng opened this issue Jan 15, 2017 · 3 comments
Closed

Test names pollute student namespace #15

petertseng opened this issue Jan 15, 2017 · 3 comments
Labels
paused Work paused until further notice

Comments

@petertseng
Copy link
Member

petertseng commented Jan 15, 2017

Currently, tests are on the top-level. This means their names can conflict with student names. For example, the student can't name anything cases since that's the name used to list the test cases.

We could move the tests inside a class or object.

This would require making the test functions shared (ahhh, that's the situation where they have to be shared, I was wondering in #14) - confirm that if moving them to a test class/object.

Currently there is an obstacle: ceylon.language.meta.model.TypeApplicationException "Cannot apply a member declaration with no container type: use memberApply" on an attempt to use parameters on a member. Apparently a fix is planned for 1.3.2 1.3.3 1.4.0 (See eclipse-archived/ceylon-sdk#635), so we may have to wait until then.

Thus, I believe this issue is unactionable as of Ceylon 1.3.3 (please notify in this issue if a newer Ceylon version comes out), but if there are alternative ways of solving this problem please say the word.

@petertseng
Copy link
Member Author

Another possible option that may work today: Put the tests in a separate package. I'm a little reluctant to add another package for each exercise just for this, though.

@petertseng
Copy link
Member Author

Update. 1.3.2 is released, but issue is still open. It has been bumped to 1.3.3.

@petertseng
Copy link
Member Author

petertseng commented Sep 15, 2017

1.3.3 was released on August 21, but without the necessary change (verified not working in https://travis-ci.org/exercism/ceylon/builds/276053644). So it will have to wait until 1.4.0.

@kytrinyx kytrinyx added the paused Work paused until further notice label Nov 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
paused Work paused until further notice
Projects
None yet
Development

No branches or pull requests

2 participants