[cueman] Add unit tests and validation for frame operations

cueman/README.md

@@ -217,10 +217,10 @@ cueman -retry job_name -state DEAD
 ## Frame Manipulation
 
 ```bash
-# Stagger frames by increment
+# Stagger frames by increment (must be positive integer)
 cueman -stagger job_name 1-100 5
 
-# Reorder frames
+# Reorder frames (position must be FIRST, LAST, or REVERSE)
 cueman -reorder job_name 1-100 FIRST
 cueman -reorder job_name 50-100 LAST
 cueman -reorder job_name 1-100 REVERSE

cueman/cueman/main.py

@@ -567,6 +567,9 @@ def handleArgs(args):
     elif args.stagger:
         name, frame_range, increment = args.stagger
         try:
+            if not increment.isdigit() or int(increment) < 1:
+                logger.error("Error: Increment must be a positive integer.")
+                sys.exit(1)
             job = opencue.api.findJob(name)
             layers = args.layer
             common.confirm(
@@ -591,6 +594,9 @@ def handleArgs(args):
     elif args.reorder:
         name, frame_range, position = args.reorder
         try:
+            if position not in ["FIRST", "LAST", "REVERSE"]:
+                logger.error("Error: Position must be one of FIRST, LAST, or REVERSE.")
+                sys.exit(1)
             job = opencue.api.findJob(name)
             layers = args.layer
             common.confirm(
@@ -725,6 +731,14 @@ def buildFrameSearch(args):
     if args.layer:
         s["layer"] = args.layer
     if args.range:
+        if not re.match(r"^\d+$|^\d+-\d+$", args.range):
+            logger.error("Invalid range format: %s", args.range)
+            sys.exit(1)
+        if "-" in args.range:
+            r = args.range.partition("-")
+            if r[0] > r[2]:
+                logger.error("Invalid range format: %s", args.range)
+                sys.exit(1)
         s["range"] = args.range
     if args.state:
         s["state"] = [common.Convert.strToFrameState(st) for st in args.state]

cueman/cueman_tutorial.md

@@ -376,6 +376,8 @@ cueman -stagger job_name 1-100 5
 cueman -stagger job_name 1-50 10 -layer sim_layer
 ```
 
+**Note:** Increment must be a positive integer. Zero, negative, and non-numeric values will be rejected.
+
 ### Reorder Frames
 Change frame execution order:
 
@@ -393,6 +395,8 @@ cueman -reorder job_name 1-100 REVERSE
 cueman -reorder job_name 1-50 FIRST -layer hero_layer
 ```
 
+**Note:** Position must be one of: `FIRST`, `LAST`, or `REVERSE`. Other values will be rejected.
+
 ## Real-World Scenarios
 
 ### Scenario 1: Handling Stuck Frames