add exercise circular-buffer#52
add exercise circular-buffer#52EduardoBautista merged 1 commit intoexercism:masterfrom ekse:exercise-circular-buffer
Conversation
There was a problem hiding this comment.
Maybe could add a test for using overwrite in a non-full buffer too (like in Ruby) - create with size 2, write 1, overwrite 2, read should return 1 instead of 2
There was a problem hiding this comment.
good point, I will add this test tonight.
There was a problem hiding this comment.
I added the test. I actually had a bug in there due to a typo, oopsies.
|
We don't usually include the |
|
I can remove it, the reason I included was to provide the Error enum definition. |
|
@ekse I don't think it's too unreasonable to expect the Enum to be defined as a part of the exercise. If you can remove it, please do. |
|
I removed the lib.rs, I also merged the commits together. |
|
Looks good to me! Thanks! |
add exercise circular-buffer
The tests were ported from the Ruby version, the example implementation is similar to the Go example.