Return an error if page format is invalid

markdalgleish · Jul 25, 2012 · e178265 · e178265
1 parent 7f39a13
commit e178265
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 0 deletions.
diff --git a/lib/lanyrd-scraper.js b/lib/lanyrd-scraper.js
@@ -20,6 +20,11 @@ exports.scrape = function(url, callback) {
 			return;
 		}
 
+		if ($('h1.summary').length === 0) {
+			callback(new Error('Invalid page format'), null);
+			return;
+		}
+
 		helper.setContext($);
 
 		var data = {};

diff --git a/test/lanyrd-scraper_test.js b/test/lanyrd-scraper_test.js
@@ -46,6 +46,13 @@ exports['multi-day event'] = {
 			done();
 		});
 	},
+	'returns an error if page content is invalid': function(test) {
+		test.expect(1);
+		lanyrd.scrape('http://localhost:' + PORT + '/invalid-page.html', function(err, res){
+			test.equal(err.message, 'Invalid page format');
+			test.done();
+		});
+	},
 	'has title': function(test) {
 		test.expect(1);
 		test.equal(multiDayEventData.title, 'Web Directions Code');

diff --git a/test/mock-pages/invalid-page.html b/test/mock-pages/invalid-page.html
@@ -0,0 +1,5 @@
+<!DOCTYPE html>
+<html>
+<head></head>
+<body></body>
+</html>