Skip to content

Commit ca0288a

Browse files
Merge branch 'master' into dependabot/maven/com.github.spotbugs-spotbugs-maven-plugin-4.9.4.0
2 parents 209b692 + c046075 commit ca0288a

File tree

7 files changed

+12
-105
lines changed

7 files changed

+12
-105
lines changed

.github/workflows/build.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ jobs:
1010
steps:
1111
- uses: actions/checkout@v5
1212
- name: Set up JDK
13-
uses: actions/setup-java@v4
13+
uses: actions/setup-java@v5
1414
with:
1515
java-version: 21
1616
distribution: 'temurin'

.github/workflows/codeql.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ jobs:
2424
uses: actions/checkout@v5
2525

2626
- name: Set up JDK
27-
uses: actions/setup-java@v4
27+
uses: actions/setup-java@v5
2828
with:
2929
java-version: 21
3030
distribution: 'temurin'

.github/workflows/infer.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ jobs:
1818
- uses: actions/checkout@v5
1919

2020
- name: Set up JDK
21-
uses: actions/setup-java@v4
21+
uses: actions/setup-java@v5
2222
with:
2323
java-version: 21
2424
distribution: 'temurin'

src/main/java/com/thealgorithms/datastructures/stacks/ReverseStack.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,9 @@ private ReverseStack() {
3838
* @param stack the stack to reverse; should not be null
3939
*/
4040
public static void reverseStack(Stack<Integer> stack) {
41+
if (stack == null) {
42+
throw new IllegalArgumentException("Stack cannot be null");
43+
}
4144
if (stack.isEmpty()) {
4245
return;
4346
}

src/main/java/com/thealgorithms/others/ReverseStackUsingRecursion.java

Lines changed: 0 additions & 44 deletions
This file was deleted.

src/test/java/com/thealgorithms/datastructures/stacks/ReverseStackTest.java

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,19 @@
11
package com.thealgorithms.datastructures.stacks;
22

33
import static org.junit.jupiter.api.Assertions.assertEquals;
4+
import static org.junit.jupiter.api.Assertions.assertThrows;
45
import static org.junit.jupiter.api.Assertions.assertTrue;
56

67
import java.util.Stack;
78
import org.junit.jupiter.api.Test;
89

910
class ReverseStackTest {
1011

12+
@Test
13+
void testReverseNullStack() {
14+
assertThrows(IllegalArgumentException.class, () -> ReverseStack.reverseStack(null), "Reversing a null stack should throw an IllegalArgumentException.");
15+
}
16+
1117
@Test
1218
void testReverseEmptyStack() {
1319
Stack<Integer> stack = new Stack<>();

src/test/java/com/thealgorithms/others/ReverseStackUsingRecursionTest.java

Lines changed: 0 additions & 58 deletions
This file was deleted.

0 commit comments

Comments
 (0)