DEV: Add tests for no_ads_for_categories

This commit is contained in:
Neil Lalonde 2019-09-11 12:18:42 -04:00
parent 0612054da8
commit b500ea171f
2 changed files with 35 additions and 2 deletions

View File

@ -5,6 +5,7 @@ acceptance("AdSense", {
loggedIn: true,
settings: {
no_ads_for_groups: "discourse",
no_ads_for_categories: "1",
adsense_publisher_code: "MYADSENSEID",
adsense_through_trust_level: 2,
adsense_topic_list_top_code: "list_top_ad_unit",
@ -15,7 +16,9 @@ acceptance("AdSense", {
adsense_post_bottom_ad_sizes: "728*90 - leaderboard",
adsense_mobile_post_bottom_code: "mobile_post_bottom_ad_unit",
adsense_mobile_post_bottom_ad_size: "300*250 - medium rectangle",
adsense_nth_post_code: 6
adsense_nth_post_code: 6,
adsense_topic_above_post_stream_code: "above_post_stream_ad_unit",
adsense_topic_above_post_stream_ad_sizes: "728*90 - leaderboard"
},
site: {
house_creatives: {
@ -34,6 +37,13 @@ acceptance("AdSense", {
test("correct number of ads should show", async assert => {
updateCurrentUser({ staff: false, trust_level: 1 });
await visit("/t/280"); // 20 posts
assert.equal(
find(".google-adsense.adsense-topic-above-post-stream").length,
1,
"it should render 1 ad"
);
const ads = find(".google-adsense.adsense-post-bottom");
assert.equal(ads.length, 3, "it should render 3 ads");
assert.equal(
@ -82,3 +92,13 @@ test("can omit ads based on groups", async assert => {
"it should render 0 ads"
);
});
test("can omit ads based on category", async assert => {
updateCurrentUser({ staff: false, trust_level: 1 });
await visit("/t/28830");
assert.equal(
find(".google-adsense.adsense-topic-above-post-stream").length,
0,
"it should render 0 ads"
);
});

View File

@ -5,6 +5,7 @@ acceptance("DFP Ads", {
loggedIn: true,
settings: {
no_ads_for_groups: "discourse",
no_ads_for_categories: "1",
dfp_publisher_id: "MYdfpID",
dfp_through_trust_level: 2,
dfp_topic_list_top_code: "list_top_ad_unit",
@ -15,7 +16,9 @@ acceptance("DFP Ads", {
dfp_post_bottom_ad_sizes: "728*90 - leaderboard",
dfp_mobile_post_bottom_code: "mobile_post_bottom_ad_unit",
dfp_mobile_post_bottom_ad_size: "300*250 - medium rectangle",
dfp_nth_post_code: 6
dfp_nth_post_code: 6,
dfp_topic_above_post_stream_code: "list_top_ad_unit",
dfp_topic_above_post_stream_ad_sizes: "728*90 - leaderboard"
},
site: {
house_creatives: {
@ -82,3 +85,13 @@ test("can omit ads based on groups", async assert => {
"it should render 0 ads"
);
});
test("can omit ads based on category", async assert => {
updateCurrentUser({ staff: false, trust_level: 1 });
await visit("/t/28830");
assert.equal(
find(".google-dfp-ad.dfp-ad-topic-above-post-stream").length,
0,
"it should render 0 ads"
);
});