Write automated tests for Events service

RESOLVED DUPLICATE of bug 1000342

Status

Webmaker
Events
RESOLVED DUPLICATE of bug 1000342
4 years ago
3 years ago

People

(Reporter: gvn, Assigned: inactivate account for kate)

Tracking

Details

Attachments

(1 attachment)

(Reporter)

Description

4 years ago
This may serve as a meta-ticket, but we *really* need to start writing some tests for the Events service.
Status: NEW → ASSIGNED
Whiteboard: [june27][tech-debt] → [july11][tech-debt]

Updated

4 years ago
Whiteboard: [july11][tech-debt] → [july25][tech-debt]

Updated

4 years ago
Whiteboard: [july25][tech-debt] → [july11][tech-debt]
(Assignee)

Comment 1

4 years ago
I started working on fixing the existing tests here https://github.com/k88hudson/webmaker-events-service/compare/mozilla:master...k88hudson:tests
(Reporter)

Comment 2

4 years ago
Nice! I'm actually starting with front end tests via Protractor, so I'm going to make a new ticket for FE tests. Do you want to take this one for your service tests? We should probably move both to next sprint realistically.
Flags: needinfo?(kate)
(Assignee)

Comment 3

4 years ago
yeah let's move this to next sprint
Flags: needinfo?(kate)
(Assignee)

Updated

4 years ago
Assignee: gavin → kate
(Assignee)

Updated

4 years ago
Whiteboard: [july11][tech-debt] → [july25][tech-debt]
* Is this a Kate thing or a Gavin thing?
(Reporter)

Updated

3 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → DUPLICATE
Duplicate of bug: 1000342
(Reporter)

Updated

3 years ago
Whiteboard: [july25][tech-debt]
(Assignee)

Comment 6

3 years ago
Created attachment 8459818 [details] [review]
https://github.com/mozilla/webmaker-events-service/pull/81
Attachment #8459818 - Flags: review?(gavin)
(Assignee)

Updated

3 years ago
Attachment #8459818 - Flags: review?(gavin)

Comment 7

3 years ago
Commit pushed to master at https://github.com/mozilla/webmaker-events-service

https://github.com/mozilla/webmaker-events-service/commit/cf232fc0901658bc3a9bdcf2d4278003f5c8aaee
Bug 1030285 - Fixed tests
You need to log in before you can comment on or make changes to this bug.