carols10cents/cargo-open

Add a test for trying to run `cargo open` without a `Cargo.toml`

Open

#9 opened on Oct 4, 2015

View on GitHub
 (0 comments) (0 reactions) (0 assignees)Rust (21 stars) (5 forks)user submission
E-easyhelp wanted

Description

In main.rs, add a unit test for the cargo_dir function that checks that it panics if we try to run this command in a directory that doesn't have a Cargo.toml and thus would have no idea where to look to figure out where that crate might be!

Use the existing test marked should_panic for an example of how to make a test check that a function panics.

I doubt that this test will pass currently! Eventually, I'd like this to panic with a message like "Cannot find Cargo.toml file in the current directory" or similar.

Contributor guide

Tech stack
rust
Domain
cli
Issue type
test
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
under 1 hour
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
fresh
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
basic Rust knowledgeunderstanding of unit tests
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
90
Research direction
Look at the `main.rs` file in the repository to find the `cargo dir` function. Examine the existing test marked `should panic` to understand the pattern for testing panics. Write a new test that calls `cargo dir` in a directory without a `Cargo.toml` and expects it to panic with a message like "Cannot find `Cargo.toml` file in the current directory". Ensure the test is added in the same test module as the existing test.