make non-empty githubLink for tests to pass in all fixtures

iliakan · iliakan · commit 73d35b9f7b2b · 2015-10-03T13:06:06.000+02:00
'
diff --git a/fixture/tutorial.js b/fixture/tutorial.js
diff --git a/handlers/auth/test/fixtures/db.js b/handlers/auth/test/fixtures/db.js
diff --git a/handlers/tutorial/test/fixture/article.js b/handlers/tutorial/test/fixture/article.js
@@ -10,7 +10,7 @@ exports.Article = [
     slug:     "article-1",
     content:  "Content 1",
     modified: new Date(2014, 0, 1),
-    githubLink: '',
+    githubLink: 'http://not.exists.com',
     isFolder: true,
     weight:   0
   },
@@ -20,7 +20,7 @@ exports.Article = [
     slug:     "article-2",
     content:  "Content 2",
     modified: new Date(2014, 0, 2),
-    githubLink: '',
+    githubLink: 'http://not.exists.com',
     isFolder: true,
     weight:   1
   },
@@ -32,7 +32,7 @@ exports.Article = [
     modified: new Date(2014, 0, 2),
     isFolder: false,
     weight:   0,
-    githubLink: '',
+    githubLink: 'http://not.exists.com',
     parent:   ids[0]
   },
   {
@@ -43,7 +43,7 @@ exports.Article = [
     modified: new Date(2014, 0, 3),
     isFolder: false,
     weight:   1,
-    githubLink: '',
+    githubLink: 'http://not.exists.com',
     parent:   ids[0]
   },
   {
@@ -53,7 +53,7 @@ exports.Article = [
     content:  "Content 2.1",
     modified: new Date(2014, 0, 4),
     isFolder: false,
-    githubLink: '',
+    githubLink: 'http://not.exists.com',
     weight:   0,
     parent:   ids[1]
   },
@@ -64,7 +64,7 @@ exports.Article = [
     content:  "Content 2.2",
     modified: new Date(2014, 0, 5),
     isFolder: false,
-    githubLink: '',
+    githubLink: 'http://not.exists.com',
     weight:   1,
     parent:   ids[1]
   }
diff --git a/handlers/tutorial/test/model/article.js b/handlers/tutorial/test/model/article.js
@@ -22,7 +22,7 @@ describe('Article', function() {
       slug: 'slug',
       content: "Content",
       isFolder: false,
-      githubLink: '',
+      githubLink: 'http://not.exists.com',
       weight: 0
     });
 
diff --git a/handlers/tutorial/test/renderer/articleRenderer.js b/handlers/tutorial/test/renderer/articleRenderer.js
@@ -15,7 +15,7 @@ describe("ArticleRenderer", function() {
     const article = new Article({
       title:   "Title",
       slug:    "test",
-      githubLink: '',
+      githubLink: 'http://not.exists.com',
       content: "# Title\n\n## Title\n\nMy html *string*."
     });
     const renderer = new ArticleRenderer();
@@ -35,7 +35,7 @@ describe("ArticleRenderer", function() {
       weight:  0,
       isFolder: false,
       content: "# Title\n\n## Title\n\nMy html *string*.",
-      githubLink: ''
+      githubLink: 'http://not.exists.com'
     }).persist();
 
     const article = new Article({
@@ -44,7 +44,7 @@ describe("ArticleRenderer", function() {
       weight:  0,
       isFolder: false,
       content: "# Title\n\n[](/test)",
-      githubLink: ''
+      githubLink: 'http://not.exists.com'
     });
 
     const renderer = new ArticleRenderer();
diff --git a/modules/lib/dataUtil.js b/modules/lib/dataUtil.js
@@ -117,7 +117,7 @@ function *loadModel(Model, data) {
       yield model.persist();
     } catch (e) {
       if (e.name == 'ValidationError') {
-        log.error("loadModel persist validation error", e, e.errors);
+        log.error("loadModel persist validation error", e, e.errors, model.toObject());
       }
       throw e;
     }

Original file line number	Diff line number	Diff line change
`@@ -117,7 +117,7 @@ function *loadModel(Model, data) {`
`117`	`117`	`yield model.persist();`
`118`	`118`	`} catch (e) {`
`119`	`119`	`if (e.name == 'ValidationError') {`
`120`		`- log.error("loadModel persist validation error", e, e.errors);`
	`120`	`+ log.error("loadModel persist validation error", e, e.errors, model.toObject());`
`121`	`121`	`}`
`122`	`122`	`throw e;`
`123`	`123`	`}`