Commit 7c2500b8 by Tyler Hallada

Fix tracking view JS tests

parent b2058669
...@@ -72,7 +72,8 @@ define(['jquery', 'models/course-model', 'models/tracking-model', 'models/user-m ...@@ -72,7 +72,8 @@ define(['jquery', 'models/course-model', 'models/tracking-model', 'models/user-m
// segment view should call segment methods when data is set // segment view should call segment methods when data is set
courseModel.set({ courseModel.set({
courseId: 'this/is/a/course' courseId: 'this/is/a/course',
org: 'org'
}); });
trackingModel.set({ trackingModel.set({
segmentApplicationId: 'applicationId', segmentApplicationId: 'applicationId',
...@@ -84,6 +85,7 @@ define(['jquery', 'models/course-model', 'models/tracking-model', 'models/user-m ...@@ -84,6 +85,7 @@ define(['jquery', 'models/course-model', 'models/tracking-model', 'models/user-m
expect(view.segment.identify).toHaveBeenCalled(); expect(view.segment.identify).toHaveBeenCalled();
expect(view.segment.page).toHaveBeenCalledWith({ expect(view.segment.page).toHaveBeenCalledWith({
courseId: 'this/is/a/course', courseId: 'this/is/a/course',
org: 'org',
label: 'mypage' label: 'mypage'
}); });
expect(view.segment.load).toHaveBeenCalled(); expect(view.segment.load).toHaveBeenCalled();
...@@ -122,7 +124,8 @@ define(['jquery', 'models/course-model', 'models/tracking-model', 'models/user-m ...@@ -122,7 +124,8 @@ define(['jquery', 'models/course-model', 'models/tracking-model', 'models/user-m
it('should call segment::track() when segment events are triggers', function() { it('should call segment::track() when segment events are triggers', function() {
var view, var view,
courseModel = new CourseModel({ courseModel = new CourseModel({
courseId: 'my/course/id' courseId: 'my/course/id',
org: 'org'
}), }),
trackingModel = new TrackingModel({ trackingModel = new TrackingModel({
page: 'mypage' page: 'mypage'
...@@ -158,6 +161,7 @@ define(['jquery', 'models/course-model', 'models/tracking-model', 'models/user-m ...@@ -158,6 +161,7 @@ define(['jquery', 'models/course-model', 'models/tracking-model', 'models/user-m
'trackingEvent', { 'trackingEvent', {
label: 'mypage', label: 'mypage',
courseId: 'my/course/id', courseId: 'my/course/id',
org: 'org',
param: 'my-param' param: 'my-param'
}); });
}); });
...@@ -165,7 +169,8 @@ define(['jquery', 'models/course-model', 'models/tracking-model', 'models/user-m ...@@ -165,7 +169,8 @@ define(['jquery', 'models/course-model', 'models/tracking-model', 'models/user-m
it('should call segment::page()', function() { it('should call segment::page()', function() {
var view, var view,
courseModel = new CourseModel({ courseModel = new CourseModel({
courseId: 'my/course/id' courseId: 'my/course/id',
org: 'org'
}), }),
trackingModel = new TrackingModel({ trackingModel = new TrackingModel({
page: 'mypage' page: 'mypage'
...@@ -201,6 +206,7 @@ define(['jquery', 'models/course-model', 'models/tracking-model', 'models/user-m ...@@ -201,6 +206,7 @@ define(['jquery', 'models/course-model', 'models/tracking-model', 'models/user-m
'pageName', { 'pageName', {
label: 'mypage', label: 'mypage',
courseId: 'my/course/id', courseId: 'my/course/id',
org: 'org',
param: 'my-param' param: 'my-param'
}); });
}); });
...@@ -218,7 +224,8 @@ define(['jquery', 'models/course-model', 'models/tracking-model', 'models/user-m ...@@ -218,7 +224,8 @@ define(['jquery', 'models/course-model', 'models/tracking-model', 'models/user-m
function setupTest() { function setupTest() {
var view, var view,
courseModel = new CourseModel({ courseModel = new CourseModel({
courseId: 'my/course/id' courseId: 'my/course/id',
org: 'org'
}), }),
trackingModel = new TrackingModel({ trackingModel = new TrackingModel({
page: 'mypage', page: 'mypage',
...@@ -272,6 +279,7 @@ define(['jquery', 'models/course-model', 'models/tracking-model', 'models/user-m ...@@ -272,6 +279,7 @@ define(['jquery', 'models/course-model', 'models/tracking-model', 'models/user-m
'trackingEvent', { 'trackingEvent', {
label: 'mypage', label: 'mypage',
courseId: 'my/course/id', courseId: 'my/course/id',
org: 'org',
param: 'my-param', param: 'my-param',
foo: 'bar' foo: 'bar'
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment