Re: [Django] #29024: `TestContextDecorator` never exits if `setUp` fails in tests

[Django]

#29024: `TestContextDecorator` never exits if `setUp` fails in tests
---+
 Reporter:  Anthony King   |Owner:  Kamil
 Type:  Bug|   Status:  closed
Component:  Testing framework  |  Version:  master
 Severity:  Normal |   Resolution:  fixed
 Keywords: | Triage Stage:  Accepted
Has patch:  1  |  Needs documentation:  0
  Needs tests:  1  |  Patch needs improvement:  1
Easy pickings:  1  |UI/UX:  0
---+
Changes (by Tim Graham ):

 * status:  assigned => closed
 * resolution:   => fixed


Comment:

 In [changeset:"3d4080f19c606865f8f76d30d91c49d989a7f76c" 3d4080f]:
 {{{
 #!CommitTicketReference repository=""
 revision="3d4080f19c606865f8f76d30d91c49d989a7f76c"
 Fixed #29024 -- Made TestContextDecorator call disable() if setUp() raises
 an exception.
 }}}

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/067.4832d8ad52004533ab1e2d4d7ef3b4b4%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #29024: `TestContextDecorator` never exits if `setUp` fails in tests

[Django]

#29024: `TestContextDecorator` never exits if `setUp` fails in tests
---+
 Reporter:  Anthony King   |Owner:  Kamil
 Type:  Bug|   Status:  assigned
Component:  Testing framework  |  Version:  master
 Severity:  Normal |   Resolution:
 Keywords: | Triage Stage:  Accepted
Has patch:  1  |  Needs documentation:  0
  Needs tests:  1  |  Patch needs improvement:  1
Easy pickings:  1  |UI/UX:  0
---+
Changes (by Jeff):

 * needs_better_patch:  0 => 1


Comment:

 Recommendations present on PR

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/067.104fd09422de656827c459feef5add87%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #29024: `TestContextDecorator` never exits if `setUp` fails in tests

[Django]

#29024: `TestContextDecorator` never exits if `setUp` fails in tests
---+
 Reporter:  Anthony King   |Owner:  Kamil
 Type:  Bug|   Status:  assigned
Component:  Testing framework  |  Version:  master
 Severity:  Normal |   Resolution:
 Keywords: | Triage Stage:  Accepted
Has patch:  1  |  Needs documentation:  0
  Needs tests:  1  |  Patch needs improvement:  0
Easy pickings:  1  |UI/UX:  0
---+
Changes (by Kamil):

 * owner:  (none) => Kamil
 * status:  new => assigned


-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/067.e9578c9c452b53ba790cd3fd2dfe5a4d%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #29024: `TestContextDecorator` never exits if `setUp` fails in tests

[Django]

#29024: `TestContextDecorator` never exits if `setUp` fails in tests
---+
 Reporter:  Anthony King   |Owner:  (none)
 Type:  Bug|   Status:  new
Component:  Testing framework  |  Version:  master
 Severity:  Normal |   Resolution:
 Keywords: | Triage Stage:  Accepted
Has patch:  1  |  Needs documentation:  0
  Needs tests:  1  |  Patch needs improvement:  0
Easy pickings:  1  |UI/UX:  0
---+
Changes (by Shahbaj Sayyad):

 * owner:  Shahbaj Sayyad => (none)
 * status:  assigned => new


-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/067.4a5578d1e03cbc4577e404dc1d3c25e8%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #29024: `TestContextDecorator` never exits if `setUp` fails in tests

[Django]

#29024: `TestContextDecorator` never exits if `setUp` fails in tests
-+-
 Reporter:  Anthony King |Owner:  Shahbaj
 |  Sayyad
 Type:  Bug  |   Status:  assigned
Component:  Testing framework|  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  1|  Patch needs improvement:  0
Easy pickings:  1|UI/UX:  0
-+-
Changes (by Tim Graham):

 * needs_tests:  0 => 1


-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/067.99f709bf0f8315d1a3081f21b2551fd7%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #29024: `TestContextDecorator` never exits if `setUp` fails in tests

[Django]

#29024: `TestContextDecorator` never exits if `setUp` fails in tests
-+-
 Reporter:  Anthony King |Owner:  Shahbaj
 |  Sayyad
 Type:  Bug  |   Status:  assigned
Component:  Testing framework|  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  1|UI/UX:  0
-+-
Changes (by Shahbaj Sayyad):

 * has_patch:  1 => 0


-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/067.69ca7651484cf106b7b5206ca3b61537%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #29024: `TestContextDecorator` never exits if `setUp` fails in tests

[Django]

#29024: `TestContextDecorator` never exits if `setUp` fails in tests
-+-
 Reporter:  Anthony King |Owner:  Shahbaj
 |  Sayyad
 Type:  Bug  |   Status:  assigned
Component:  Testing framework|  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  1|UI/UX:  0
-+-
Changes (by Shahbaj Sayyad):

 * has_patch:  0 => 1


-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/067.d4f55fbb7ef913b0a007520d4980e347%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #29024: `TestContextDecorator` never exits if `setUp` fails in tests

[Django]

#29024: `TestContextDecorator` never exits if `setUp` fails in tests
-+-
 Reporter:  Anthony King |Owner:  Shahbaj
 |  Sayyad
 Type:  Bug  |   Status:  assigned
Component:  Testing framework|  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  1|UI/UX:  0
-+-

Comment (by Anthony King):

 Hi Shahbaj,

 I saw your post in the mailing list.


 So how we've done it internally is like this:

 {{{
 #!python

 class TestContextDecorator(DjangoTestContextDecorator):
 def decorate_class(self, cls):
 # https://code.djangoproject.com/ticket/29024

 if issubclass(cls, TestCase):
 decorated_setUp = cls.setUp

 @wraps(decorated_setUp)
 def setUp(inner_self):
 context = self.enable()
 if self.attr_name:
 setattr(inner_self, self.attr_name, context)
 inner_self.addCleanup(self.disable)
 decorated_setUp(inner_self)

 cls.setUp = setUp
 return cls
 raise TypeError('Can only decorate subclasses of
 unittest.TestCase')
 }}}

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/067.905d96498d4948ef2badf453a567655b%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #29024: `TestContextDecorator` never exits if `setUp` fails in tests

[Django]

#29024: `TestContextDecorator` never exits if `setUp` fails in tests
-+-
 Reporter:  Anthony King |Owner:  Shahbaj
 |  Sayyad
 Type:  Bug  |   Status:  assigned
Component:  Testing framework|  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  1|UI/UX:  0
-+-
Description changed by Anthony King:

Old description:

> when using `TestContextDecorator` as a class decorator, `enable` is
> called just before `Test.setUp`, and `disable` is called after
> `Test.tearDown`.
>
> unfortunately, `tearDown` is not called in the event of a failure inside
> `setUp`. This can result in unexpected behaviour.
>
> Even though this class is not documented, there are decorators that use
> it that are.
>

> example:
>

> {{{#!python
> class example_decorator(TestContextDecorator):
> some_var = False
>
> def enable(self):
> self.__class__.some_var = True
>
> def disable(self):
> self.__class__.some_var = False
>
> class Example1TestCase(TestCase):
> def test_var_is_false(self):
> # some_var will be False
> self.assertFalse(example_decorator.some_var)
>

> @example_decorator()
> class Example2TestCase(TestCase):
> def setUp(self):
> raise Exception
>
> def test_var_is_true(self):
> # won't be hit due to exception in setUp
> self.assertTrue(example_decorator.some_var)
>

> class Example3TestCase(TestCase):
> def test_var_is_false(self):
> # some_var will be True now due to no cleanup in Example2TestCase
> self.assertFalse(example_decorator.some_var)
>
> }}}
> output:
> {{{
> .EF
> ==
> ERROR: test_var_is_true (sharefile.tests.Example2TestCase)
> --
> ... Traceback
>
> ==
> FAIL: test_var_is_false (sharefile.tests.Example3TestCase)
> --
> ...
> AssertionError: True is not false
>
> Ran 3 tests in 0.007s
>
> FAILED (failures=1, errors=1)
> }}}
>
> There are 2 potential fixes here:
> - decorate `setUpClass` and `tearDownClass`
> - use `addCleanup` inside the `setUp` decorator
>
> `addCleanup` will always run, and will only ever be registered if the
> context manager was successfully enabled.
>
> It would also be useful to have this class documented, however that's out
> of scope of this ticket.

New description:

 when using `TestContextDecorator` as a class decorator, `enable` is called
 just before `Test.setUp`, and `disable` is called after `Test.tearDown`.

 unfortunately, `tearDown` is not called in the event of a failure inside
 `setUp`. This can result in unexpected behaviour.

 Even though this class is not documented, there are decorators that use it
 that are.


 example:


 {{{#!python
 class example_decorator(TestContextDecorator):
 some_var = False

 def enable(self):
 self.__class__.some_var = True

 def disable(self):
 self.__class__.some_var = False

 class Example1TestCase(TestCase):
 def test_var_is_false(self):
 # some_var will be False
 self.assertFalse(example_decorator.some_var)


 @example_decorator()
 class Example2TestCase(TestCase):
 def setUp(self):
 raise Exception

 def test_var_is_true(self):
 # won't be hit due to exception in setUp
 self.assertTrue(example_decorator.some_var)


 class Example3TestCase(TestCase):
 def test_var_is_false(self):
 # some_var will be True now due to no cleanup in Example2TestCase
 self.assertFalse(example_decorator.some_var)

 }}}
 output:
 {{{
 .EF
 ==
 ERROR: test_var_is_true (example.tests.Example2TestCase)
 --
 ... Traceback

 ==
 FAIL: test_var_is_false (example.tests.Example3TestCase)
 --
 ...
 AssertionError: True is not false

 Ran 3 tests in 0.007s

 FAILED (failures=1, errors=1)
 }}}

 There are 2 potential fixes here:
 - decorate `setUpClass` and `tearDownClass`
 - use `addCleanup` inside the `setUp` decorator

 `addCleanup` will always run, and will only ever be register

Re: [Django] #29024: `TestContextDecorator` never exits if `setUp` fails in tests

[Django]

#29024: `TestContextDecorator` never exits if `setUp` fails in tests
-+-
 Reporter:  Anthony King |Owner:  Shahbaj
 |  Sayyad
 Type:  Bug  |   Status:  assigned
Component:  Testing framework|  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  1|UI/UX:  0
-+-
Changes (by Shahbaj Sayyad):

 * owner:  nobody => Shahbaj Sayyad
 * status:  new => assigned


-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/067.be867b8ea348952961f6d020e25092c7%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #29024: `TestContextDecorator` never exits if `setUp` fails in tests

[Django]

#29024: `TestContextDecorator` never exits if `setUp` fails in tests
---+
 Reporter:  Anthony King   |Owner:  nobody
 Type:  Bug|   Status:  new
Component:  Testing framework  |  Version:  master
 Severity:  Normal |   Resolution:
 Keywords: | Triage Stage:  Accepted
Has patch:  0  |  Needs documentation:  0
  Needs tests:  0  |  Patch needs improvement:  0
Easy pickings:  1  |UI/UX:  0
---+
Changes (by Simon Charette):

 * version:  1.11 => master
 * stage:  Unreviewed => Accepted


Comment:

 Using `addCleanup` instead of hooking up on `tearDown` makes sense here.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/067.2cb238d389294ac7250505e7a7ac1823%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

[Django] #29024: `TestContextDecorator` never exits if `setUp` fails in tests

[Django]

#29024: `TestContextDecorator` never exits if `setUp` fails in tests
-+
   Reporter:  Anthony King   |  Owner:  nobody
   Type:  Bug| Status:  new
  Component:  Testing framework  |Version:  1.11
   Severity:  Normal |   Keywords:
   Triage Stage:  Unreviewed |  Has patch:  0
Needs documentation:  0  |Needs tests:  0
Patch needs improvement:  0  |  Easy pickings:  1
  UI/UX:  0  |
-+
 when using `TestContextDecorator` as a class decorator, `enable` is called
 just before `Test.setUp`, and `disable` is called after `Test.tearDown`.

 unfortunately, `tearDown` is not called in the event of a failure inside
 `setUp`. This can result in unexpected behaviour.

 Even though this class is not documented, there are decorators that use it
 that are.


 example:


 {{{#!python
 class example_decorator(TestContextDecorator):
 some_var = False

 def enable(self):
 self.__class__.some_var = True

 def disable(self):
 self.__class__.some_var = False

 class Example1TestCase(TestCase):
 def test_var_is_false(self):
 # some_var will be False
 self.assertFalse(example_decorator.some_var)


 @example_decorator()
 class Example2TestCase(TestCase):
 def setUp(self):
 raise Exception

 def test_var_is_true(self):
 # won't be hit due to exception in setUp
 self.assertTrue(example_decorator.some_var)


 class Example3TestCase(TestCase):
 def test_var_is_false(self):
 # some_var will be True now due to no cleanup in Example2TestCase
 self.assertFalse(example_decorator.some_var)

 }}}
 output:
 {{{
 .EF
 ==
 ERROR: test_var_is_true (sharefile.tests.Example2TestCase)
 --
 ... Traceback

 ==
 FAIL: test_var_is_false (sharefile.tests.Example3TestCase)
 --
 ...
 AssertionError: True is not false

 Ran 3 tests in 0.007s

 FAILED (failures=1, errors=1)
 }}}

 There are 2 potential fixes here:
 - decorate `setUpClass` and `tearDownClass`
 - use `addCleanup` inside the `setUp` decorator

 `addCleanup` will always run, and will only ever be registered if the
 context manager was successfully enabled.

 It would also be useful to have this class documented, however that's out
 of scope of this ticket.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/052.3ccb058a9ed903ace6fc26e54008982a%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.