Mercurial > p > roundup > code
comparison test/test_mailgw.py @ 8261:28c5030757d3
fix: cae1bbf2536b - expression errors not setting result properly
Rather than setting 'result' in renderContext, I was wrirting the
output directly, but was stil falling through to where result would be
used.
This caused email to be sent to the admin and when test_mailgw was
called the mail-test.log was somehow created and caused the test there
to fail.
Fixed renderContext to use result variable and output contents of
mail-test.log to make debugging easier.
This should fix CI failure.
| author | John Rouillard <rouilj@ieee.org> |
|---|---|
| date | Sun, 05 Jan 2025 00:59:12 -0500 |
| parents | c12377fb4144 |
| children | 520075b29474 |
comparison
equal
deleted
inserted
replaced
| 8260:617d85ce4ac3 | 8261:28c5030757d3 |
|---|---|
| 537 Subject: [issue] Testing... | 537 Subject: [issue] Testing... |
| 538 | 538 |
| 539 This is a test submission of a new issue. | 539 This is a test submission of a new issue. |
| 540 ''') | 540 ''') |
| 541 userlist = self.db.user.list() | 541 userlist = self.db.user.list() |
| 542 if os.path.exists(SENDMAILDEBUG): | |
| 543 with open(SENDMAILDEBUG) as f: | |
| 544 print(f.read()) | |
| 542 assert not os.path.exists(SENDMAILDEBUG) | 545 assert not os.path.exists(SENDMAILDEBUG) |
| 543 self.assertEqual(userlist, self.db.user.list(), | 546 self.assertEqual(userlist, self.db.user.list(), |
| 544 "user created when it shouldn't have been") | 547 "user created when it shouldn't have been") |
| 545 | 548 |
| 546 def testNewIssueNoClass(self): | 549 def testNewIssueNoClass(self): |