Skip to content

Commit 7f2ec8d

Browse files
authored
Merge pull request #67 from jmesnil/tck_env_vars
[tck] fix instructions for env variables
2 parents 99559dd + c7e5b90 commit 7f2ec8d

File tree

2 files changed

+13
-13
lines changed

2 files changed

+13
-13
lines changed

tck/running_the_tck.asciidoc

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -54,10 +54,10 @@ To enable the tests in your project you need to add the following dependency to
5454
Some tests are asserting statements related to environment variables.
5555
The following environment variable and their values must be present in the test runner environment:
5656

57-
* `my_int_property` with the value `45`
58-
* `MY_BOOLEAN_PROPERTY` with the value `true`
59-
* `my_string_property` with the value `haha`
60-
* `MY_STRING_PROPERTY` with the value `woohoo`
57+
* `envconfig_my_int_property` with the value `45`
58+
* `ENVCONFIG_MY_BOOLEAN_PROPERTY` with the value `true`
59+
* `envconfig_my_string_property` with the value `haha`
60+
* `ENVCONFIG_MY_STRING_PROPERTY` with the value `woohoo`
6161

6262
See below for an example configuration to provide these environment variables in a Maven pom.xml.
6363

@@ -95,12 +95,12 @@ If you use Apache Maven then the tests are run via the `maven-surefire-plugin`
9595
<suiteXmlFiles>
9696
<suiteXmlFile>tck-suite.xml</suiteXmlFile>
9797
</suiteXmlFiles>
98-
<!-- These env variables are required fororg.eclipse.configjsr.CDIPropertyNameMatchingTest -->
98+
<!-- These env variables are required for org.eclipse.configjsr.CDIPropertyNameMatchingTest -->
9999
<environmentVariables>
100-
<my_int_property>45</my_int_property>
101-
<MY_BOOLEAN_PROPERTY>true</MY_BOOLEAN_PROPERTY>
102-
<my_string_property>haha</my_string_property>
103-
<MY_STRING_PROPERTY>woohoo</MY_STRING_PROPERTY>
100+
<envconfig_my_int_property>45</envconfig_my_int_property>
101+
<ENVCONFIG_MY_BOOLEAN_PROPERTY>true</ENVCONFIG_MY_BOOLEAN_PROPERTY>
102+
<envconfig_my_string_property>haha</envconfig_my_string_property>
103+
<ENVCONFIG_MY_STRING_PROPERTY>woohoo</ENVCONFIG_MY_STRING_PROPERTY>
104104
</environmentVariables>
105105
</configuration>
106106
</plugin>

tck/src/main/java/org/eclipse/configjsr/CDIPropertyNameMatchingTest.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -45,10 +45,10 @@
4545
*
4646
* Prerequisite:
4747
* The following environment variables must be set prior to running this test:
48-
* "my_int_property" with the value of "45"
49-
* "MY_BOOLEAN_PROPERTY" with the value of "true"
50-
* "my_string_property" with the value of "haha"
51-
* "MY_STRING_PROPERTY" with the value of "woohoo"
48+
* "envconfig_my_int_property" with the value of "45"
49+
* "ENVCONFIG_MY_BOOLEAN_PROPERTY" with the value of "true"
50+
* "envconfig_my_string_property" with the value of "haha"
51+
* "ENVCONFIG_MY_STRING_PROPERTY" with the value of "woohoo"
5252
* @author Emily Jiang
5353
*/
5454
public class CDIPropertyNameMatchingTest extends Arquillian {

0 commit comments

Comments
 (0)