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

Transition from tests3.rs to test3.rs is confusing #244

Closed
cincodenada opened this issue Dec 4, 2019 · 8 comments
Closed

Transition from tests3.rs to test3.rs is confusing #244

cincodenada opened this issue Dec 4, 2019 · 8 comments
Assignees
Labels
A-exercises Area: Exercises

Comments

@cincodenada
Copy link

So I was going through rustlings (which has been great, thanks!) and as it happens, the next unit after tests3.rs is test3.rs in the root. It took me some hard staring to figure out why the code didn't look like I expected it to before I noticed the one-character difference in the filenames.

I'm not sure what the best solution here is - perhaps test*.rs in the root directory could be renamed to quiz*.rs, since that also describes what they are, and that doesn't collide with a concept being taught?

@jrvidal
Copy link
Contributor

jrvidal commented Dec 4, 2019

I too have thought about this. Maybe tests<X>.rs could be renamed to testing<X>.rs?

@shadows-withal
Copy link
Member

Maybe something like test-yourself<X>?

@ScarboroughCoral
Copy link

Oh, I am confused too.I think it's necessary to rename it.

@sanjaykdragon
Copy link
Contributor

Maybe something like test-yourself<X>?

Yeah I think this is a better name

@jrial
Copy link

jrial commented May 1, 2020

My $0.02: renaming to "quiz" is the best suggestion I read in this thread so far. If the idea is to avoid confusion with the previous "test" file, it makes little sense to keep "test" in the name.

@tonyito
Copy link

tonyito commented May 16, 2020

Does anyone know of the status of this issue? Still says test 1-4, when 'quiz' would be the perfect alternative as @jrial said.

@shadows-withal shadows-withal self-assigned this May 16, 2020
@shadows-withal
Copy link
Member

I think it's a good idea, and if someone doesn't snipe it from me, I'll do it as soon as I can.

@sanjaykdragon
Copy link
Contributor

sniped @fmoko
#402

samtay pushed a commit to samtay/rustlings-solutions that referenced this issue May 28, 2020
BREAKING CHANGE

* changed test to quiz: fixes issues in rust-lang#244
* fixed info.toml: rust-lang#244
* fixed naming related issues
MendelMonteiro pushed a commit to MendelMonteiro/rustlings that referenced this issue Jun 28, 2020
BREAKING CHANGE

* changed test to quiz: fixes issues in rust-lang#244
* fixed info.toml: rust-lang#244
* fixed naming related issues
dlemel8 pushed a commit to dlemel8/rustlings that referenced this issue Aug 2, 2020
BREAKING CHANGE

* changed test to quiz: fixes issues in rust-lang#244
* fixed info.toml: rust-lang#244
* fixed naming related issues
dlemel8 pushed a commit to dlemel8/rustlings that referenced this issue Aug 2, 2020
BREAKING CHANGE

* changed test to quiz: fixes issues in rust-lang#244
* fixed info.toml: rust-lang#244
* fixed naming related issues
ppp3 pushed a commit to ppp3/rustlings that referenced this issue May 23, 2022
BREAKING CHANGE

* changed test to quiz: fixes issues in rust-lang#244
* fixed info.toml: rust-lang#244
* fixed naming related issues
dmoore04 pushed a commit to dmoore04/rustlings that referenced this issue Sep 11, 2022
BREAKING CHANGE

* changed test to quiz: fixes issues in rust-lang#244
* fixed info.toml: rust-lang#244
* fixed naming related issues
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-exercises Area: Exercises
Projects
None yet
Development

No branches or pull requests

7 participants