Skip to content

Commit 4945c2a

Browse files
committed
Update exit code logic for add command tests
1 parent eac7c26 commit 4945c2a

11 files changed

Lines changed: 15 additions & 15 deletions

File tree

src/main/scala/Main.scala

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -234,7 +234,7 @@ object AddCommand extends Command[CommonOptions]:
234234
}
235235

236236
logger.info(s"Skipped: $skipped")
237-
if added == 0 then sys.exit(0)
237+
if added == 0 then sys.exit(0) else sys.exit(1)
238238
}
239239

240240
/** Add command entry point.

src/test/resources/cases/add-all-handlers/case.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ command: add
33
args:
44
- "source/*"
55
ignore: [ ]
6-
expectExit: 0
6+
expectExit: 1
77
expectedOutput:
88
- "Added: source/one.py"
99
- "Added: source/one.scala"

src/test/resources/cases/add-all-rights-reserved/case.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ command: add
33
args:
44
- "source/*.py"
55
ignore: [ ]
6-
expectExit: 0
6+
expectExit: 1
77
expectedOutput:
88
- "Added: source/one.py"
99
- "Skipped: 0"

src/test/resources/cases/add-basic/case.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ args:
44
- "source/*.py"
55
ignore:
66
- "**/skip.py"
7-
expectExit: 0
7+
expectExit: 1
88
expectedOutput:
99
- "Added: source/one.py"
1010
- "Skipped: 0"

src/test/resources/cases/add-complex/case.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ args:
55
ignore:
66
- "source/vendor/**"
77
- "source/**/skip.py"
8-
expectExit: 0
8+
expectExit: 1
99
expectedOutput:
1010
- "Added: source/app/main.py"
1111
- "Added: source/app/utils/helper.py"

src/test/resources/cases/add-directory-arg/case.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,8 @@ name: add-directory-arg
22
command: add
33
args:
44
- "source"
5-
ignore: []
6-
expectExit: 0
5+
ignore: [ ]
6+
expectExit: 1
77
expectedOutput:
88
- "Added: source/app.py"
99
- "Added: source/utils/helper.py"

src/test/resources/cases/add-ignore-bare/case.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ args:
44
- "source/**/*.py"
55
ignore:
66
- "source/skip.py"
7-
expectExit: 0
7+
expectExit: 1
88
expectedOutput:
99
- "Added: source/src/skip.py"
1010
- "Added: source/src/main.py"

src/test/resources/cases/add-multiple-ignores/case.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ ignore:
66
- "source/vendor/**"
77
- "source/build/**"
88
- "**/test_*.py"
9-
expectExit: 0
9+
expectExit: 1
1010
expectedOutput:
1111
- "Added: source/app/main.py"
1212
- "Skipped: 0"

src/test/resources/cases/add-nested/case.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,8 @@ name: add-nested
22
command: add
33
args:
44
- "source/**/*.py"
5-
ignore: []
6-
expectExit: 0
5+
ignore: [ ]
6+
expectExit: 1
77
expectedOutput:
88
- "Added: source/a/b/c/deep.py"
99
- "Added: source/x/y/file.py"

src/test/resources/cases/add-range/case.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,8 @@ name: add-range
22
command: add
33
args:
44
- "source/*.py"
5-
ignore: []
6-
expectExit: 0
5+
ignore: [ ]
6+
expectExit: 1
77
expectedOutput:
88
- "Added: source/range.py"
99
- "Skipped: 0"

0 commit comments

Comments
 (0)