Doctrine 2 - ORM

Create unit tests for CLI components

Details

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

Issue Links

is referenced by

Bug - A problem which impairs or prevents the functions of the product. DDC-359 Specified, but empty CLI Options --option= causes Notice undefined offset: 0 [Test case for CliController included]

  • Minor - Minor loss of function, or other problem where easy workaround is present.
  • Resolved - A resolution has been taken, and it is awaiting verification by reporter. From here issues are either reopened, or are closed.

Activity

Descending order - Click to sort in ascending order
Hide
Permalink
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
Permalink
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
Permalink
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
Permalink
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
Permalink
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?

People

  • Assignee:
    Guilherme Blanco
    Reporter:
    Roman S. Borschel
Vote (0)
Watch (0)

Dates

  • Created:
    Updated: