Comments (4)
I'm not sure (not familiar with Rust). @bitborn do you remember what this was about?
@petertseng @EduardoBautista do you have any ideas here?
from rust.
In its initial form, it appears this test just wanted conversion to a string. Asking for that may be clearer than as_ref
. I would have suggested https://doc.rust-lang.org/std/string/trait.ToString.html, but apparently you shouldn't implement this directly but instead should implement https://doc.rust-lang.org/std/fmt/trait.Display.html which requires students to think about Formatters. Might be a bit much.
Of course there's the question of "what's the value in asking students to be able to convert their RNA to a string anyway?". That I'd want to take some time to survey this problem in other languages to understand why. I've not thought about this exercise before.
from rust.
FWIW, I'm familiar with the Elixir and Ruby version of this exercise and they don't have any tests like this.
from rust.
I'm not by any means an expert on best practices in Rust (which is only relevant if there's some Rust practices reason for keeping this test in), but my current thinking is to agree - the test on converting to a string is not really useful (it's not to do with the problem statement, there's no clear use case for converting the RNA to a string, other languages don't do it, and other tests in xrust don't do it). I'm in favor of removing it.
from rust.
Related Issues (20)
- Spell checks / Typos HOT 2
- Question/Possible documentation issue: How to enable features? HOT 2
- Remove test_ prefix from test functions HOT 1
- Test in CI that we're in sync with problem-specifications HOT 2
- Simply linked list test error HOT 1
- Pin test runner to a specific version HOT 1
- Document / reconsider `topics` field in track config HOT 1
- Improve example solution testing scripts
- CamelCase test unintentionally removed from acronym exercise HOT 2
- Building a training set of tags for rust HOT 25
- Tests fail with no output HOT 16
- Tests don't pass when sending Cargo.toml with optional dependency HOT 1
- Remove all util functions from test files HOT 6
- Test in CI that stubs don't generate clippy warnings HOT 1
- Move more CI tests to rust tooling
- Concepts for this track are not displayed HOT 1
- word-count: change interface from u32 to usize
- test type_override fails even when correct solution is made in macros lesson HOT 1
- Performance issue about the Isogram problem approach HOT 1
- Add test that templates match generated test suite HOT 5
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from rust.