Code clean-up

serenity-bdd · Dec 30, 2023 · 0769a0e · 0769a0e
1 parent 8094d59
commit 0769a0e
Showing 1 changed file with 0 additions and 14 deletions.
diff --git a/...nity-model/src/test/java/net/thucydides/model/requirements/AggregateRequirementsTest.java b/...nity-model/src/test/java/net/thucydides/model/requirements/AggregateRequirementsTest.java
@@ -40,8 +40,6 @@ void should_treat_files_in_a_flat_directory_structure_as_representing_features()
 
             List<Requirement> requirements = requirementsFrom(pathTo("serenity-js/spec-0-levels"));
 
-            System.out.println(requirements);
-
             assertThat(requirements).hasSize(1);
 
             Requirement feature = requirements.get(0);
@@ -58,8 +56,6 @@ void should_treat_files_in_a_single_level_directory_structure_as_representing_ca
 
             List<Requirement> requirements = requirementsFrom(pathTo("serenity-js/spec-1-level"));
 
-            System.out.println(requirements);
-
             assertThat(requirements).hasSize(1);
 
             Requirement capability = requirements.get(0);
@@ -80,8 +76,6 @@ void should_treat_files_in_a_two_level_directory_structure_as_representing_theme
 
             List<Requirement> requirements = requirementsFrom(pathTo("serenity-js/spec-2-levels"));
 
-            System.out.println(requirements);
-
             assertThat(requirements).hasSize(1);
 
             Requirement theme = requirements.get(0);
@@ -112,14 +106,6 @@ private List<Requirement> requirementsFrom(Path exampleRootDirectory) {
 
         EnvironmentVariables env = new MockEnvironmentVariables();
 
-        // todo: TestOutcomeRequirementsTagProvider doesn't use the jsonOutcomesDirectory
-        //  injected into AggregateRequirements, but instead uses the one from the environment variables
-        env.setProperty(ThucydidesSystemProperty.SERENITY_OUTPUT_DIRECTORY.getPropertyName(), jsonOutcomesDirectory.toString());
-
-        // todo: some parts of the AggregateRequirements machinery ignore the injected environment variables
-        //  and instead use the global singleton SystemEnvironmentVariables
-        ModelInfrastructure.getConfiguration().withEnvironmentVariables(env);
-
         Requirements requirements = new AggregateRequirements(
                 jsonOutcomesDirectory,
                 requirementsDirectory.toString(),