Uploaded image for project: 'Doctrine 2 - ORM'
  1. Doctrine 2 - ORM
  2. DDC-222

Create unit tests for CLI components

    Details

    • Type: Task
    • Status: Closed
    • Priority: 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
        romanb Roman S. Borschel added a comment -

        Whats the status here? Do we have any?

        Show
        romanb Roman S. Borschel added a comment - Whats the status here? Do we have any?
        Hide
        guilhermeblanco 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
        guilhermeblanco 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
        beberlei 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
        beberlei 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
        beberlei 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
        beberlei 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
        romanb 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
        romanb 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
        guilhermeblanco Guilherme Blanco added a comment -

        This is not valid anymore as we use Symfony CLI component

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

          People

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

            Dates

            • Created:
              Updated:
              Resolved: