Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TestProfile's getEnabledAlternatives CDI BEAN leaks to unrelated TestProfile's test #44117

Open
yuan-kuan opened this issue Oct 26, 2024 · 3 comments · May be fixed by #44638
Open

TestProfile's getEnabledAlternatives CDI BEAN leaks to unrelated TestProfile's test #44117

yuan-kuan opened this issue Oct 26, 2024 · 3 comments · May be fixed by #44638
Labels
area/arc Issue related to ARC (dependency injection) area/testing kind/enhancement New feature or request

Comments

@yuan-kuan
Copy link

Describe the bug

Running a normal @QuarkusTest with TestProfile without any special getEnabledAlternatives() definition will mistakenly use another TestProfile's mock CDI BEAN declared in getEnabledAlternatives().

My Profiles:

public class MockProfile implements QuarkusTestProfile {
  public static class ATag implements QuarkusTestProfile {
    @Override
    public String getConfigProfile() {
      return "test-a-tag";
    }

    @Override
    public Set<String> tags() {
      return Set.of("a");
    }
    @Override
    public Set<Class<?>> getEnabledAlternatives() {
      return Collections.singleton(MockService.class);
    }
    
  }
  
  public static class BTag implements QuarkusTestProfile {
    @Override
    public String getConfigProfile() {
      return "test-b-tag";
    }

    @Override
    public Set<String> tags() {
      return Set.of("b");
    }
    @Override
    public Set<Class<?>> getEnabledAlternatives() {
      return Collections.emptySet();
    }
  }
}

TestProfile without mock service

@QuarkusTest
@TestProfile(MockProfile.BTag.class)
public class NormalTest {
    @Inject
    GreetingService greetingService;
    @Test
    public void testOk() {
      System.out.println("B tag test run");
      
      Assertions.assertEquals("special greeting", greetingService.getSpecialGreet());
    }
}

TestProfile with mock service

@QuarkusMainTest
@TestProfile(MockProfile.ATag.class)
public class BugTest {
    @Test
    @Launch({"person"})
    public void bugTest(LaunchResult result) {
        System.out.println("B tag test run"); 
        Assertions.assertTrue(result.getOutput().contains("Hello person, mocked greeting!"));
    }
}

Expected behavior

CDI BEAN used in tests under a different TestProfile should be isolated from each other.

Actual behavior

NormalTest failed because it uses the Mock Service defined in another TestProfile.

How to Reproduce?

  1. clone my small reproducible project: https://github.com/yuan-kuan/quarkus-test-tags-bug/tree/main
  2. quarkus dev
  3. press o and press r to run the unit tests with output on.
  4. Observe that okTest's failure

Output of uname -a or ver

Darwin 24.0.0 Darwin Kernel Version 24.0.0: Wed Aug 7 03:09:57 PDT 2024; root:xnu-11215.1.9~22/RELEASE_ARM64_T8112 arm64

Output of java -version

openjdk version "21.0.5" 2024-10-15 LTS

Quarkus version or git rev

quarkus 3.15.1

Build tool (ie. output of mvnw --version or gradlew --version)

Apache Maven 3.9.8 (36645f6c9b5079805ea5009217e36f2cffd34256)

Additional information

No response

@yuan-kuan yuan-kuan added the kind/bug Something isn't working label Oct 26, 2024
@quarkus-bot quarkus-bot bot added the area/arc Issue related to ARC (dependency injection) label Oct 26, 2024
Copy link

quarkus-bot bot commented Oct 26, 2024

/cc @Ladicek (arc), @manovotn (arc), @mkouba (arc)

@Ladicek
Copy link
Contributor

Ladicek commented Oct 29, 2024

Mixing @QuarkusTest and @QuarkusMainTest isn't documented in https://quarkus.io/guides/getting-started-testing#mixing-quarkustest-with-other-type-of-tests, but it feels like it isn't allowed either. If I move the @QuarkusTest execution into a separate execution of the Maven Surefire plugin (as documented), both tests pass.

Not sure who knows whether this is allowed. CC @geoand maybe?

@geoand
Copy link
Contributor

geoand commented Oct 30, 2024

Hm, My guess is that mixing those two works as long as you don't have profiles. I think this is fixable

@gsmet gsmet added kind/enhancement New feature or request and removed kind/bug Something isn't working labels Nov 16, 2024
geoand added a commit to geoand/quarkus that referenced this issue Nov 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/arc Issue related to ARC (dependency injection) area/testing kind/enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants