darcs

Issue 1973 TEST: avoid_test_spam flag results in no mail being sent to lists

Title TEST: avoid_test_spam flag results in no mail being sent to lists
Priority invalid Status resolved
Milestone Resolved in
Superseder Nosy List dmitry.kurochkin, kowey
Assigned To
Topics

Created on 2010-10-15.15:08:21 by kowey, last changed 2012-12-14.15:24:24 by markstos.

Messages
msg12702 (view) Author: kowey Date: 2010-10-15.15:08:20
Nobody should be receiving this through a mailing list. My new
avoid_test_spam flag should let me test changes to the BTS/email
integration without spamming darcs-*@
msg12703 (view) Author: kowey Date: 2010-10-15.15:13:04
Sigh, no, really nobody should receive this now.

Fri Oct 15 17:10:08 CEST 2010  Eric Kow <kowey@darcs.net>
  * Two avoid_test_spam mistakes: boolean inversion and also type error.
    {
    hunk ./detectors/nosyreaction.py 39
    -    can_spam = getattr(db.config.detectors, 'avoid_test_spam', '') == 1
    +    can_spam = getattr(db.config.detectors, 'avoid_test_spam', '')
!= '1'
    hunk ./detectors/nosyreaction.py 80
    -    can_spam = getattr(db.config.detectors, 'avoid_test_spam', '') == 1
    +    can_spam = getattr(db.config.detectors, 'avoid_test_spam', '')
!= '1'
    }
msg12704 (view) Author: kowey Date: 2010-10-15.15:20:33
No, really! this time it should work

Fri Oct 15 17:16:54 CEST 2010  Eric Kow <kowey@darcs.net>
  * Oops: forgot the setting is in all caps.
    {
    hunk ./detectors/nosyreaction.py 39
    -    can_spam = getattr(db.config.detectors, 'avoid_test_spam', '')
!= '1'
    +    can_spam = getattr(db.config.detectors, 'AVOID_TEST_SPAM', '')
!= '1'
    hunk ./detectors/nosyreaction.py 80
    -    can_spam = getattr(db.config.detectors, 'avoid_test_spam', '')
!= '1'
    +    can_spam = getattr(db.config.detectors, 'AVOID_TEST_SPAM', '')
!= '1'
    }
msg12705 (view) Author: kowey Date: 2010-10-15.15:25:50
Sorry, everybody.  I'm sure you're all loving the
does-my-spam-avoidance-mechanism-work-now? spam

I'm doing this so that I can do more testing on roundup instance
(especially the bits that send email to people) without risking so much
collateral damage.  The goal is to be able to flip a single switch that
suppresses all noisy-behaviour to lists.


darcs changes --last=1 -v
Prehook ran successfully.
Fri Oct 15 17:20:33 CEST 2010  Eric Kow <kowey@darcs.net>
  * Oh, you need every setting to be in a section.
    {
    hunk ./detectors/nosyreaction.py 39
    -    can_spam = getattr(db.config.detectors, 'AVOID_TEST_SPAM', '')
!= '1'
    +    can_spam = getattr(db.config.detectors,
'DEBUG_AVOID_TEST_SPAM', '') != '1'
    hunk ./detectors/nosyreaction.py 80
    -    can_spam = getattr(db.config.detectors, 'AVOID_TEST_SPAM', '')
!= '1'
    +    can_spam = getattr(db.config.detectors,
'DEBUG_AVOID_TEST_SPAM', '') != '1'
    }
Posthook ran successfully.
msg12706 (view) Author: kowey Date: 2010-10-15.15:31:22
I keep forgetting I have to restart Apache for changes to take effect.
msg12707 (view) Author: kowey Date: 2010-10-15.15:37:24
I finally got it working!  (I had to restart Apache)

Sorry for the noise.  This is just a final test to make sure that
disabling the spam suppression flag results in messages being sent to
lists again.
msg16412 (view) Author: markstos Date: 2012-12-14.15:24:23
Looks like Kowey was doing some testing here which is done now. 
Resolving.
History
Date User Action Args
2010-10-15 15:08:21koweycreate
2010-10-15 15:13:05koweysetmessages: + msg12703
2010-10-15 15:20:34koweysetmessages: + msg12704
2010-10-15 15:25:51koweysetmessages: + msg12705
2010-10-15 15:31:22koweysetmessages: + msg12706
2010-10-15 15:37:25koweysetmessages: + msg12707
2012-12-14 15:24:24markstossetstatus: unknown -> resolved
messages: + msg16412