Skip to content

Commit bf30864

Browse files
committed
. dt document currently supported Reporters when using EnvironmentalVariableReporter
1 parent c7fb16a commit bf30864

File tree

4 files changed

+42
-31
lines changed

4 files changed

+42
-31
lines changed

approvaltests-tests/src/test/java/org/approvaltests/reporters/EnvironmentVariableReporterTest.java

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,10 +2,11 @@
22

33
import org.approvaltests.Approvals;
44
import org.approvaltests.core.Options;
5-
import org.junit.jupiter.api.Assertions;
65
import org.junit.jupiter.api.Test;
76

8-
import static org.junit.jupiter.api.Assertions.*;
7+
import java.util.List;
8+
import java.util.Set;
9+
import java.util.stream.Collectors;
910

1011
public class EnvironmentVariableReporterTest
1112
{
@@ -26,4 +27,11 @@ public void testEnvironmentVariable()
2627
EnvironmentVariableReporter.ENVIRONMENT_VARIABLES = System::getenv;
2728
}
2829
}
30+
@Test
31+
void testValidEnvironmentalValues()
32+
{
33+
Set<String> reporters = new EnvironmentVariableReporter().getReporterMapping().keySet();
34+
List<String> sortedReporters = reporters.stream().sorted().collect(Collectors.toList());
35+
Approvals.verifyAll("", sortedReporters, x -> x);
36+
}
2937
}
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,27 @@
1+
AraxisMergeReporter
2+
AutoApproveReporter
3+
AutoApproveWhenEmptyReporter
4+
BeyondCompareReporter
5+
ClipboardReporter
6+
CodeCompareReporter
7+
DelayedClipboardReporter
8+
DiffMergeReporter
9+
DiffReporter
10+
FileCaptureReporter
11+
ImageReporter
12+
ImageWebReporter
13+
IntelliJReporter
14+
JunitReporter
15+
KDiff3Reporter
16+
KaleidoscopeDiffReporter
17+
MeldMergeReporter
18+
P4MergeReporter
19+
PitReporter
20+
QuietReporter
21+
TestNgReporter
22+
TextWebReporter
23+
TkDiffReporter
24+
TortoiseDiffReporter
25+
VisualStudioCodeReporter
26+
WinMergeReporter
27+
WindowsDiffReporter

approvaltests/docs/how_to/ConfigureReporters.md

Lines changed: 1 addition & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -91,35 +91,7 @@ Lastly, it is possible to set the `APPROVAL_TESTS_USE_REPORTER` environment vari
9191

9292
The environment variable can take any combination of the following values. Multiple values should be separated by a comma, without whitespace.
9393

94-
```java
95-
"AraxisMergeReporter",
96-
"AutoApproveReporter",
97-
"AutoApproveWhenEmptyReporter",
98-
"BeyondCompareReporter",
99-
"ClipboardReporter",
100-
"CodeCompareReporter",
101-
"DelayedClipboardReporter",
102-
"DiffMergeReporter",
103-
"DiffReporter",
104-
"FileCaptureReporter",
105-
"ImageReporter",
106-
"ImageWebReporter",
107-
"IntelliJReporter",
108-
"JunitReporter",
109-
"KDiff3Reporter",
110-
"KaleidoscopeDiffReporter",
111-
"MeldMergeReporter",
112-
"P4MergeReporter",
113-
"PitReporter",
114-
"QuietReporter",
115-
"TestNgReporter",
116-
"TextWebReporter",
117-
"TkDiffReporter",
118-
"TortoiseDiffReporter",
119-
"VisualStudioCodeReporter",
120-
"WinMergeReporter",
121-
"WindowsDiffReporter",
122-
```
94+
snippets: EnvironmentVariableReporterTest.testValidEnvironmentalValues.approved.txt
12395

12496
For example, setting `APPROVAL_TESTS_USE_REPORTER=AutoApproveReporter` allows you to approve all pending changes at once
12597
without modifying the source code and rebuilding the code to temporarily choose a different reporter. Similarly, setting

approvaltests/src/main/java/org/approvaltests/reporters/EnvironmentVariableReporter.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -78,4 +78,8 @@ public boolean report(String received, String approved)
7878
{ return false; }
7979
return reporter.report(received, approved);
8080
}
81+
public Map<String, Class> getReporterMapping()
82+
{
83+
return new HashMap<>(REPORTER_MAP);
84+
}
8185
}

0 commit comments

Comments
 (0)