Skip to content

Commit b3efd12

Browse files
committed
Fix argument accumulation bug in integration test
The test reused a single Command, causing args to accumulate across cases. Each argument-count scenario now uses a fresh Command.
1 parent 6fdec08 commit b3efd12

1 file changed

Lines changed: 16 additions & 9 deletions

File tree

tests/int_tests.rs

Lines changed: 16 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,20 +1,27 @@
11
use assert_cmd::cargo;
22

33
#[test]
4-
fn test_different_amount_of_arguments() {
4+
fn test_no_arguments() {
55
let mut cmd = cargo::cargo_bin_cmd!("bitcoinvert");
6-
7-
// No argument
86
cmd.assert().success();
7+
}
98

10-
// 1 argument
11-
cmd.args(vec!["1"]).assert().success();
9+
#[test]
10+
fn test_one_argument() {
11+
let mut cmd = cargo::cargo_bin_cmd!("bitcoinvert");
12+
cmd.args(["1"]).assert().success();
13+
}
1214

13-
// 2 arguments
14-
cmd.args(vec!["BTC"]).assert().success();
15+
#[test]
16+
fn test_two_arguments() {
17+
let mut cmd = cargo::cargo_bin_cmd!("bitcoinvert");
18+
cmd.args(["1", "BTC"]).assert().success();
19+
}
1520

16-
// 3 arguments
17-
cmd.args(vec!["SAT"]).assert().stdout("100,000,000 SAT\n");
21+
#[test]
22+
fn test_three_arguments() {
23+
let mut cmd = cargo::cargo_bin_cmd!("bitcoinvert");
24+
cmd.args(["1", "BTC", "SAT"]).assert().stdout("100,000,000 SAT\n");
1825
}
1926

2027
#[test]

0 commit comments

Comments
 (0)