Skip to content

Commit 5d74edb

Browse files
committed
Standardize order of test annotations
1 parent 302034a commit 5d74edb

10 files changed

Lines changed: 108 additions & 108 deletions

File tree

POLICIES.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ References: [[1](https://github.com/exercism/java/issues/178)]
4848
4949
### Ignore noninitial tests
5050

51-
> All but the first test in an exercise test suite should be annotated `@Ignore("Remove to run test")` (single test) or `@Ignore("Remove to run tests")` (parametrized test).
51+
> All but the first test in an exercise test suite should add `@Ignore("Remove to run test")` (single test) or `@Ignore("Remove to run tests")` (parametrized test) on the line before the corresponding `@Test` annotation.
5252
5353
References: [[1](https://github.com/exercism/java/issues/101#issuecomment-249349204)]
5454

exercises/binary-search-tree/src/test/java/BSTTest.java

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -20,8 +20,8 @@ public void dataIsRetained() {
2020
assertEquals(expected, actual);
2121
}
2222

23-
@Test
2423
@Ignore("Remove to run test")
24+
@Test
2525
public void insertsLess() {
2626
BST<Integer> bst = new BST();
2727
final int expectedRoot = 4;
@@ -41,8 +41,8 @@ public void insertsLess() {
4141
assertEquals(expectedRoot, actualRoot);
4242
}
4343

44-
@Test
4544
@Ignore("Remove to run test")
45+
@Test
4646
public void insertsSame() {
4747
BST<Integer> bst = new BST();
4848
final int expectedRoot = 4;
@@ -62,8 +62,8 @@ public void insertsSame() {
6262
assertEquals(expectedRoot, actualRoot);
6363
}
6464

65-
@Test
6665
@Ignore("Remove to run test")
66+
@Test
6767
public void insertsRight() {
6868
BST<Integer> bst = new BST();
6969
final int expectedRoot = 4;
@@ -83,8 +83,8 @@ public void insertsRight() {
8383
assertEquals(expectedRoot, actualRoot);
8484
}
8585

86-
@Test
8786
@Ignore("Remove to run test")
87+
@Test
8888
public void createsComplexTree() {
8989
BST<Integer> bst = new BST<>();
9090
List<Integer> expected = Collections.unmodifiableList(
@@ -100,8 +100,8 @@ public void createsComplexTree() {
100100
assertEquals(expected, actual);
101101
}
102102

103-
@Test
104103
@Ignore("Remove to run test")
104+
@Test
105105
public void sortsSingleElement() {
106106
BST<Integer> bst = new BST<>();
107107
List<Integer> expected = Collections.unmodifiableList(
@@ -114,8 +114,8 @@ public void sortsSingleElement() {
114114
assertEquals(expected, actual);
115115
}
116116

117-
@Test
118117
@Ignore("Remove to run test")
118+
@Test
119119
public void sortsCollectionOfTwoIfSecondInsertedIsSmallerThanFirst() {
120120
BST<Integer> bst = new BST<>();
121121
List<Integer> expected = Collections.unmodifiableList(
@@ -129,8 +129,8 @@ public void sortsCollectionOfTwoIfSecondInsertedIsSmallerThanFirst() {
129129
assertEquals(expected, actual);
130130
}
131131

132-
@Test
133132
@Ignore("Remove to run test")
133+
@Test
134134
public void sortsCollectionOfTwoIfSecondInsertedIsBiggerThanFirst() {
135135
BST<Integer> bst = new BST<>();
136136
List<Integer> expected = Collections.unmodifiableList(
@@ -144,8 +144,8 @@ public void sortsCollectionOfTwoIfSecondInsertedIsBiggerThanFirst() {
144144
assertEquals(expected, actual);
145145
}
146146

147-
@Test
148147
@Ignore("Remove to run test")
148+
@Test
149149
public void iteratesOverComplexTree() {
150150
BST<Integer> bst = new BST<>();
151151
List<Integer> expected = Collections.unmodifiableList(

0 commit comments

Comments
 (0)