PicoContainer
  1. PicoContainer
  2. PICO-229

VerifyingVisitor does not work for array types.

    Details

    • Type: Bug Bug
    • Status: Resolved Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.1
    • Fix Version/s: 1.2
    • Component/s: PicoContainer (Java)
    • Labels:
      None
    • Environment:
      JDK 1.4
    • Number of attachments :
      0

      Description

      The following testcase should pass, but it fails with a PicoVerificationException.

      public static class MockRunnable implements Runnable{
      public void run() {
      }
      }

      public static class OtherRunnable implements Runnable{
      public void run() {
      }
      }

      public static class MockRunner{
      private final Runnable[] _runners;
      public MockRunner(Runnable[] runnable)

      { _runners = runnable; }

      }

      public void testArrayDependenciesAndVerification()

      { DefaultPicoContainer container = new DefaultPicoContainer(); container.registerComponentImplementation(MockRunnable.class); container.registerComponentImplementation(OtherRunnable.class); container.registerComponentImplementation(MockRunner.class); //this will fail to resolve the Runnable array on the MockRunner VerifyingVisitor visitor = new VerifyingVisitor(); visitor.traverse(container); container.start(); assertNotNull(container.getComponentInstanceOfType(MockRunner.class)); }

        Activity

        Jörg Schaible made changes -
        Field Original Value New Value
        Assignee Joerg Schaible [ joehni ]
        Hide
        Jörg Schaible added a comment -

        Fixed in CVS. Thanks for test case.

        Show
        Jörg Schaible added a comment - Fixed in CVS. Thanks for test case.
        Jörg Schaible made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 1.2 [ 11330 ]
        Resolution Fixed [ 1 ]

          People

          • Assignee:
            Jörg Schaible
            Reporter:
            Michael Rettig
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: