Closed
Bug 757279
Opened 12 years ago
Closed 12 years ago
[testing] testTransactionExecutor should use the mock module instead
Categories
(Socorro :: General, task)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: peterbe, Assigned: peterbe)
Details
The testTransactionExecutor uses ad-hoc MockConnection and MockLogging hacks. Since we bank on the mock module we should use it through out all new code.
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → peterbe
Assignee | ||
Comment 1•12 years ago
|
||
Pull request up: https://github.com/mozilla/socorro/pull/605
Comment 2•12 years ago
|
||
Commit pushed to master at https://github.com/mozilla/socorro https://github.com/mozilla/socorro/commit/d5430e894e29de1addeb88bd1fc8c54da6109e51 bug757279 - mocking logging with mock instead, r=twobraids
Assignee | ||
Updated•12 years ago
|
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Comment 3•12 years ago
|
||
Commit pushed to stage at https://github.com/mozilla/socorro https://github.com/mozilla/socorro/commit/d5430e894e29de1addeb88bd1fc8c54da6109e51 bug757279 - mocking logging with mock instead, r=twobraids
Comment 4•12 years ago
|
||
Commit pushed to stage at https://github.com/mozilla/socorro https://github.com/mozilla/socorro/commit/7a664f3e58e29ca685f7043e720e97d357ac0f77 bug757279 - mocking logging with mock instead, r=twobraids
Comment 5•12 years ago
|
||
Commit pushed to admin-refactor at https://github.com/mozilla/socorro https://github.com/mozilla/socorro/commit/d5430e894e29de1addeb88bd1fc8c54da6109e51 bug757279 - mocking logging with mock instead, r=twobraids
Comment 6•12 years ago
|
||
Commit pushed to stage at https://github.com/mozilla/socorro https://github.com/mozilla/socorro/commit/b9e70c14e06cb60a708ac60314262bb84937027a bug757279 - mocking logging with mock instead, r=twobraids
Comment 7•12 years ago
|
||
Commit pushed to https://github.com/mozilla/socorro https://github.com/mozilla/socorro/commit/b9e70c14e06cb60a708ac60314262bb84937027a bug757279 - mocking logging with mock instead, r=twobraids
You need to log in
before you can comment on or make changes to this bug.
Description
•