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

Why not use tests for some exercises #127

Closed
Hades32 opened this issue Mar 18, 2019 · 3 comments
Closed

Why not use tests for some exercises #127

Hades32 opened this issue Mar 18, 2019 · 3 comments
Labels
A-exercises Area: Exercises

Comments

@Hades32
Copy link

Hades32 commented Mar 18, 2019

There are several exercises of the style

if you_did_it_right {
  print("great");
} else {
  print("try again");
}

Why aren't these exercises done as tests?

@komaeda komaeda added C-question A-exercises Area: Exercises labels Mar 20, 2019
@MForever78
Copy link

Echo on this. some exercises can now pass by just not having compiling errors.

dylnuge added a commit to dylnuge/rustlings that referenced this issue Aug 18, 2019
This commit converts primitive_types4 to a test and asserts that the
slice given is equal to the expected slice.

The intent of the primitive_types4 exercise appears to be to ensure the
user understands inclusive and exclusive bounds as well as slice syntax.
`rustlings` commands using `compile` do not verify that a specific
println is reached and, in the case of `watch` and `verify` (but not
`run`), they do not output the `println`s at all.

This fix is semantically similar to rust-lang#198. It does not take a stance on
the correct way to handle this for all exercises; see rust-lang#127. There are
likely other exercises whose intent are masked by this issue.
bors added a commit that referenced this issue Aug 18, 2019
fix(primitive_types4): Fail on a slice covering the wrong area

I noticed this issue and it seems like a similar one was raised/fixed in #160 this way. This is my first contribution to this repo (or any Rust project) so let me know if I messed up or need to fix anything!

---
This commit converts primitive_types4 to a test and asserts that the
slice given is equal to the expected slice.

The intent of the primitive_types4 exercise appears to be to ensure the
user understands inclusive and exclusive bounds as well as slice syntax.
`rustlings` commands using `compile` do not verify that a specific
println is reached and, in the case of `watch` and `verify` (but not
`run`), they do not output the `println`s at all.

This fix is semantically similar to #198. It does not take a stance on
the correct way to handle this for all exercises; see #127. There are
likely other exercises whose intent are masked by this issue.
pedantic79 pushed a commit to pedantic79/rustlings that referenced this issue Apr 11, 2020
This commit converts primitive_types4 to a test and asserts that the
slice given is equal to the expected slice.

The intent of the primitive_types4 exercise appears to be to ensure the
user understands inclusive and exclusive bounds as well as slice syntax.
`rustlings` commands using `compile` do not verify that a specific
println is reached and, in the case of `watch` and `verify` (but not
`run`), they do not output the `println`s at all.

This fix is semantically similar to rust-lang#198. It does not take a stance on
the correct way to handle this for all exercises; see rust-lang#127. There are
likely other exercises whose intent are masked by this issue.
pedantic79 pushed a commit to pedantic79/rustlings that referenced this issue Apr 11, 2020
fix(primitive_types4): Fail on a slice covering the wrong area

I noticed this issue and it seems like a similar one was raised/fixed in rust-lang#160 this way. This is my first contribution to this repo (or any Rust project) so let me know if I messed up or need to fix anything!

---
This commit converts primitive_types4 to a test and asserts that the
slice given is equal to the expected slice.

The intent of the primitive_types4 exercise appears to be to ensure the
user understands inclusive and exclusive bounds as well as slice syntax.
`rustlings` commands using `compile` do not verify that a specific
println is reached and, in the case of `watch` and `verify` (but not
`run`), they do not output the `println`s at all.

This fix is semantically similar to rust-lang#198. It does not take a stance on
the correct way to handle this for all exercises; see rust-lang#127. There are
likely other exercises whose intent are masked by this issue.
ppp3 pushed a commit to ppp3/rustlings that referenced this issue May 23, 2022
This commit converts primitive_types4 to a test and asserts that the
slice given is equal to the expected slice.

The intent of the primitive_types4 exercise appears to be to ensure the
user understands inclusive and exclusive bounds as well as slice syntax.
`rustlings` commands using `compile` do not verify that a specific
println is reached and, in the case of `watch` and `verify` (but not
`run`), they do not output the `println`s at all.

This fix is semantically similar to rust-lang#198. It does not take a stance on
the correct way to handle this for all exercises; see rust-lang#127. There are
likely other exercises whose intent are masked by this issue.
ppp3 pushed a commit to ppp3/rustlings that referenced this issue May 23, 2022
fix(primitive_types4): Fail on a slice covering the wrong area

I noticed this issue and it seems like a similar one was raised/fixed in rust-lang#160 this way. This is my first contribution to this repo (or any Rust project) so let me know if I messed up or need to fix anything!

---
This commit converts primitive_types4 to a test and asserts that the
slice given is equal to the expected slice.

The intent of the primitive_types4 exercise appears to be to ensure the
user understands inclusive and exclusive bounds as well as slice syntax.
`rustlings` commands using `compile` do not verify that a specific
println is reached and, in the case of `watch` and `verify` (but not
`run`), they do not output the `println`s at all.

This fix is semantically similar to rust-lang#198. It does not take a stance on
the correct way to handle this for all exercises; see rust-lang#127. There are
likely other exercises whose intent are masked by this issue.
dmoore04 pushed a commit to dmoore04/rustlings that referenced this issue Sep 11, 2022
This commit converts primitive_types4 to a test and asserts that the
slice given is equal to the expected slice.

The intent of the primitive_types4 exercise appears to be to ensure the
user understands inclusive and exclusive bounds as well as slice syntax.
`rustlings` commands using `compile` do not verify that a specific
println is reached and, in the case of `watch` and `verify` (but not
`run`), they do not output the `println`s at all.

This fix is semantically similar to rust-lang#198. It does not take a stance on
the correct way to handle this for all exercises; see rust-lang#127. There are
likely other exercises whose intent are masked by this issue.
dmoore04 pushed a commit to dmoore04/rustlings that referenced this issue Sep 11, 2022
fix(primitive_types4): Fail on a slice covering the wrong area

I noticed this issue and it seems like a similar one was raised/fixed in rust-lang#160 this way. This is my first contribution to this repo (or any Rust project) so let me know if I messed up or need to fix anything!

---
This commit converts primitive_types4 to a test and asserts that the
slice given is equal to the expected slice.

The intent of the primitive_types4 exercise appears to be to ensure the
user understands inclusive and exclusive bounds as well as slice syntax.
`rustlings` commands using `compile` do not verify that a specific
println is reached and, in the case of `watch` and `verify` (but not
`run`), they do not output the `println`s at all.

This fix is semantically similar to rust-lang#198. It does not take a stance on
the correct way to handle this for all exercises; see rust-lang#127. There are
likely other exercises whose intent are masked by this issue.
@mo8it
Copy link
Contributor

mo8it commented Mar 27, 2024

I really appreciate the message of this issue, but it is too unspecific. Could you please point to some specific exercises?

@mo8it
Copy link
Contributor

mo8it commented Mar 27, 2024

Closing but feel free to open new issues pointing to specific exercises. You are also welcome to submit pull requests :D

@mo8it mo8it closed this as completed Mar 27, 2024
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

4 participants