Review Chaneg: Change to --force-refill.

spencer-tb · spencer-tb · commit 7e2ad59c9082 · 2023-03-07T15:57:10.000-06:00
diff --git a/src/ethereum_test_filling_tool/filler.py b/src/ethereum_test_filling_tool/filler.py
@@ -119,7 +119,7 @@ def fill_fixture(self, filler, t8n, b11r):
             and not is_module_modified(
                 path, self.options.filler_path, module_path
             )
-            and not self.options.overwrite
+            and not self.options.force_refill
         ):
             self.log.debug(f"skipping - {full_name}")
             return
diff --git a/src/ethereum_test_filling_tool/main.py b/src/ethereum_test_filling_tool/main.py
@@ -85,7 +85,7 @@ def parse_arguments() -> argparse.Namespace:
     )
 
     parser.add_argument(
-        "--overwrite",
+        "--force-refill",
         action="store_true",
         help="fill all test fillers and don't skip any tests \
               overwriting where necessary",

Original file line number	Diff line number	Diff line change
`@@ -119,7 +119,7 @@ def fill_fixture(self, filler, t8n, b11r):`
`119`	`119`	`and not is_module_modified(`
`120`	`120`	`path, self.options.filler_path, module_path`
`121`	`121`	`)`
`122`		`- and not self.options.overwrite`
	`122`	`+ and not self.options.force_refill`
`123`	`123`	`):`
`124`	`124`	`self.log.debug(f"skipping - {full_name}")`
`125`	`125`	`return`
Original file line number	Diff line number	Diff line change
`@@ -85,7 +85,7 @@ def parse_arguments() -> argparse.Namespace:`
`85`	`85`	`)`
`86`	`86`
`87`	`87`	`parser.add_argument(`
`88`		`- "--overwrite",`
	`88`	`+ "--force-refill",`
`89`	`89`	`action="store_true",`
`90`	`90`	`help="fill all test fillers and don't skip any tests \`
`91`	`91`	`overwriting where necessary",`