Rename children -> assets in Resource

Author: s0ph1e

README.md

@@ -62,7 +62,7 @@ Makes requests to `urls` and saves all files found with `sources` to `directory`
   - `result`: if error - `null`, if success - array of [Resource](https://github.com/s0ph1e/node-website-scraper/blob/master/lib/resource.js) objects containing:
     - `url`: url of loaded page
     - `filename`: filename where page was saved (relative to `directory`)
-    - `children`: array of children Resources
+    - `assets`: array of children Resources
 
 ### Filename Generators
 The filename generator determines where the scraped files are saved.

lib/resource.js

@@ -13,7 +13,7 @@ function getTypeByHtmlData (htmlData) {
 function Resource (url, filename) {
 	this.url = url;
 	this.filename = filename;
-	this.children = [];
+	this.assets = [];
 }
 
 Resource.prototype.createChild = function createChild (url, filename) {
@@ -24,22 +24,18 @@ Resource.prototype.createChild = function createChild (url, filename) {
 	child.setParent(this);
 	child.setDepth(++currentDepth);
 
-	this.children.push(child);
+	this.assets.push(child);
 
 	return child;
 };
 
 Resource.prototype.updateChild = function updateChild (oldChild, newChild) {
-	var index = _.indexOf(this.children, oldChild);
+	var index = _.indexOf(this.assets, oldChild);
 	if (index >= 0) {
-		this.children[index] = newChild;
+		this.assets[index] = newChild;
 	}
 };
 
-Resource.prototype.getChildren = function getChildren () {
-	return this.children;
-};
-
 Resource.prototype.getUrl = function getUrl () {
 	return this.url;
 };

test/functional/base-test.js

@@ -69,19 +69,19 @@ describe('Functional base', function() {
 			result.should.be.instanceOf(Array).and.have.length(3);
 
 			result[0].should.have.properties({ url: 'http://example.com/', filename: 'index.html' });
-			result[0].should.have.properties('children');
-			result[0].children.should.be.instanceOf(Array).and.have.length(4);
-			result[0].children[0].should.be.instanceOf(Resource);
+			result[0].should.have.properties('assets');
+			result[0].assets.should.be.instanceOf(Array).and.have.length(4);
+			result[0].assets[0].should.be.instanceOf(Resource);
 
 			result[1].should.have.properties({ url: 'http://example.com/about', filename: 'about.html' });
-			result[1].should.have.properties('children');
-			result[1].children.should.be.instanceOf(Array).and.have.length(4);
-			result[1].children[0].should.be.instanceOf(Resource);
+			result[1].should.have.properties('assets');
+			result[1].assets.should.be.instanceOf(Array).and.have.length(4);
+			result[1].assets[0].should.be.instanceOf(Resource);
 
 			result[2].should.have.properties({ url: 'http://blog.example.com/', filename: 'blog.html' }); // url after redirect
-			result[2].should.have.properties('children');
-			result[2].children.should.be.instanceOf(Array).and.have.length(1);
-			result[2].children[0].should.be.instanceOf(Resource);
+			result[2].should.have.properties('assets');
+			result[2].assets.should.be.instanceOf(Array).and.have.length(1);
+			result[2].assets[0].should.be.instanceOf(Resource);
 
 			// should create directory and subdirectories
 			fs.existsSync(testDirname).should.be.eql(true);

test/unit/scraper-test.js

@@ -210,7 +210,7 @@ describe('Scraper', function () {
 			}).catch(done);
 		});
 
-		it('should return array of objects with url, filename and childre', function(done) {
+		it('should return array of objects with url, filename and assets', function(done) {
 			nock('http://first-url.com').get('/').reply(200, 'OK');
 			nock('http://second-url.com').get('/').reply(500);
 
@@ -225,8 +225,8 @@ describe('Scraper', function () {
 			s.prepare().bind(s).then(s.load).then(function(res) {
 				res.should.be.instanceOf(Array);
 				res.should.have.length(2);
-				res[0].should.be.instanceOf(Resource).and.have.properties(['url', 'filename', 'children']);
-				res[1].should.be.instanceOf(Resource).and.have.properties(['url', 'filename', 'children']);
+				res[0].should.be.instanceOf(Resource).and.have.properties(['url', 'filename', 'assets']);
+				res[1].should.be.instanceOf(Resource).and.have.properties(['url', 'filename', 'assets']);
 				done();
 			}).catch(done);
 		});