Doctrine 2 - ORM
  1. Doctrine 2 - ORM
  2. DDC-222

Create unit tests for CLI components

    Details

    • Type: Task Task
    • Status: Closed
    • Priority: Critical Critical
    • Resolution: Invalid
    • Affects Version/s: 2.0-ALPHA3
    • Fix Version/s: 2.x
    • Component/s: ORM
    • Security Level: All
    • Labels:
      None

      Issue Links

        Activity

        Hide
        Roman S. Borschel added a comment -

        Whats the status here? Do we have any?

        Show
        Roman S. Borschel added a comment - Whats the status here? Do we have any?
        Hide
        Guilherme Blanco added a comment -

        Since we moved to Symfony Console I don't think this is needed anymore.
        The purpose of this ticket was actually to test our own CLI support, which was dropped.

        I'm closing the ticket due to this. Reopen if you have any other comment.

        Show
        Guilherme Blanco added a comment - Since we moved to Symfony Console I don't think this is needed anymore. The purpose of this ticket was actually to test our own CLI support, which was dropped. I'm closing the ticket due to this. Reopen if you have any other comment.
        Hide
        Benjamin Eberlei added a comment -

        I think we do need some basic functional tests of our Commands, they have been subject to many bugs in the past becaues they are not tested.

        Show
        Benjamin Eberlei added a comment - I think we do need some basic functional tests of our Commands, they have been subject to many bugs in the past becaues they are not tested.
        Hide
        Benjamin Eberlei added a comment -

        Fixed another fatal error in the command due to missing namespace dependency. We need tests for all the commands, there have been dozens of issues on these things so far.

        This commit shows a simple approach on how testing is easily possible for symfony commands:

        http://github.com/doctrine/doctrine2/commit/51e6681934a7cf4448b85c5670c04045f66c6056

        Show
        Benjamin Eberlei added a comment - Fixed another fatal error in the command due to missing namespace dependency. We need tests for all the commands, there have been dozens of issues on these things so far. This commit shows a simple approach on how testing is easily possible for symfony commands: http://github.com/doctrine/doctrine2/commit/51e6681934a7cf4448b85c5670c04045f66c6056
        Hide
        Roman S. Borschel added a comment -

        Can we expect some more tests for beta4 or is it unlikely that you find the time? Should we move this further back or does someone else want to step in?

        Show
        Roman S. Borschel added a comment - Can we expect some more tests for beta4 or is it unlikely that you find the time? Should we move this further back or does someone else want to step in?
        Hide
        Guilherme Blanco added a comment -

        This is not valid anymore as we use Symfony CLI component

        Show
        Guilherme Blanco added a comment - This is not valid anymore as we use Symfony CLI component

          People

          • Assignee:
            Guilherme Blanco
            Reporter:
            Roman S. Borschel
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: