refactor: change semantics to right exclusive

Author: peter-jerry-ye

crates/moon/src/cli/bench.rs

@@ -41,8 +41,8 @@ pub struct BenchSubcommand {
     #[clap(short, long, requires("package"))]
     pub file: Option<String>,
 
-    /// Run only the index-th test in the file. Accepts a single index or an inclusive
-    /// range like `0-2`. Only valid when `--file` is also specified.
+    /// Run only the index-th test in the file. Accepts a single index or a left-inclusive
+    /// right-exclusive range like `0-2`. Only valid when `--file` is also specified.
     #[clap(short, long, requires("file"))]
     pub index: Option<TestIndexRange>,
 

crates/moon/src/cli/test.rs

@@ -107,8 +107,8 @@ pub struct TestSubcommand {
     #[clap(short, long)]
     pub file: Option<String>,
 
-    /// Run only the index-th test in the file. Accepts a single index or an inclusive
-    /// range like `0-2`. Only valid when `--file` is also specified.
+    /// Run only the index-th test in the file. Accepts a single index or a left-inclusive
+    /// right-exclusive range like `0-2`. Only valid when `--file` is also specified.
     /// Implies `--include-skipped`.
     #[clap(short, long)]
     pub index: Option<TestIndexRange>,

crates/moon/tests/test_cases/test_filter/mod.rs

@@ -836,7 +836,7 @@ fn test_moon_test_filter_index_range() {
                 "-f",
                 "hello.mbt",
                 "-i",
-                "0-1",
+                "0-2",
             ],
         ),
         expect![[r#"

crates/moonutil/src/common.rs

@@ -596,9 +596,6 @@ impl FromStr for TestIndexRange {
             }
             let start = parse_index_bound(start, TestIndexRangeParseError::MissingStart)?;
             let end = parse_index_bound(end, TestIndexRangeParseError::MissingEnd)?;
-            let end = end
-                .checked_add(1)
-                .ok_or(TestIndexRangeParseError::EndOverflow)?;
             if start >= end {
                 return Err(TestIndexRangeParseError::InvalidRange);
             }

docs/dev/reference/tests.md

@@ -134,7 +134,7 @@ Filtering is handled by `TestFilter` (`runtest/filter.rs`). It stores allowed
 
 - `--index N` selects a single inline test (the index is taken from the generated
   metadata, displayed in failure output as `#N`).
-- `--index A-B` selects an inclusive range (end included).
+- `--index A-B` selects a left-inclusive right-exclusive range.
 - `--doc-index N` selects a single doc test block.
 - When a positional `PATH` points to a file, the indexes can be provided without an
   explicit `--file`. Otherwise, the CLI requires `--file` whenever `--index` or

docs/manual-zh/src/commands.md

@@ -256,7 +256,7 @@ Test the current package
 
 * `-p`, `--package <PACKAGE>` — Run test in the specified package
 * `-f`, `--file <FILE>` — Run test in the specified file. Only valid when `--package` is also specified
-* `-i`, `--index <INDEX>` — Run only the index-th test in the file. Accepts a single index or an inclusive range like `0-2`. Only valid when `--file` is also specified. Implies `--include-skipped`
+* `-i`, `--index <INDEX>` — Run only the index-th test in the file. Accepts a single index or a left-inclusive right-exclusive range like `0-2`. Only valid when `--file` is also specified. Implies `--include-skipped`
 * `--doc-index <DOC_INDEX>` — Run only the index-th doc test in the file. Only valid when `--file` is also specified. Implies `--include-skipped`
 * `-u`, `--update` — Update the test snapshot
 * `-l`, `--limit <LIMIT>` — Limit of expect test update passes to run, in order to avoid infinite loops
@@ -385,7 +385,7 @@ Run benchmarks in the current package
 
 * `-p`, `--package <PACKAGE>` — Run test in the specified package
 * `-f`, `--file <FILE>` — Run test in the specified file. Only valid when `--package` is also specified
-* `-i`, `--index <INDEX>` — Run only the index-th test in the file. Accepts a single index or an inclusive range like `0-2`. Only valid when `--file` is also specified
+* `-i`, `--index <INDEX>` — Run only the index-th test in the file. Accepts a single index or a left-inclusive right-exclusive range like `0-2`. Only valid when `--file` is also specified
 * `--frozen` — Do not sync dependencies, assuming local dependencies are up-to-date
 * `--build-only` — Only build, do not bench
 * `--no-parallelize` — Run the benchmarks in a target backend sequentially

docs/manual/src/commands.md

@@ -256,7 +256,7 @@ Test the current package
 
 * `-p`, `--package <PACKAGE>` — Run test in the specified package
 * `-f`, `--file <FILE>` — Run test in the specified file. Only valid when `--package` is also specified
-* `-i`, `--index <INDEX>` — Run only the index-th test in the file. Accepts a single index or an inclusive range like `0-2`. Only valid when `--file` is also specified. Implies `--include-skipped`
+* `-i`, `--index <INDEX>` — Run only the index-th test in the file. Accepts a single index or a left-inclusive right-exclusive range like `0-2`. Only valid when `--file` is also specified. Implies `--include-skipped`
 * `--doc-index <DOC_INDEX>` — Run only the index-th doc test in the file. Only valid when `--file` is also specified. Implies `--include-skipped`
 * `-u`, `--update` — Update the test snapshot
 * `-l`, `--limit <LIMIT>` — Limit of expect test update passes to run, in order to avoid infinite loops
@@ -385,7 +385,7 @@ Run benchmarks in the current package
 
 * `-p`, `--package <PACKAGE>` — Run test in the specified package
 * `-f`, `--file <FILE>` — Run test in the specified file. Only valid when `--package` is also specified
-* `-i`, `--index <INDEX>` — Run only the index-th test in the file. Accepts a single index or an inclusive range like `0-2`. Only valid when `--file` is also specified
+* `-i`, `--index <INDEX>` — Run only the index-th test in the file. Accepts a single index or a left-inclusive right-exclusive range like `0-2`. Only valid when `--file` is also specified
 * `--frozen` — Do not sync dependencies, assuming local dependencies are up-to-date
 * `--build-only` — Only build, do not bench
 * `--no-parallelize` — Run the benchmarks in a target backend sequentially