define brands, companies, as separate function

ajlopez · Mar 3, 2014 · 2201c3c · 2201c3c
1 parent aab4ca3
commit 2201c3c
Show file tree

Hide file tree

Showing 3 changed files with 24 additions and 9 deletions.
diff --git a/samples/preciosa/preciosa.js b/samples/preciosa/preciosa.js
@@ -20,7 +20,7 @@ function initialize() {
 }
 
 function loadRemoteMarcas(cb) {
-    var url = 'http://preciosdeargentina.com.ar/api/v1/marcas/?page=1&format=json&page_size=100';
+    var url = 'http://preciosdeargentina.com.ar/api/v1/marcas/?page=1&format=json&page_size=5000';
 
     http.get(url, function(res) {
         var body = '';
@@ -57,14 +57,6 @@ function loadMarcasFabricantes() {
         else if (row.model == 'precios.marca')
             addMarca(row);
     });
-
-    fabricantes.forEach(function (item) {
-        defineFabricante(item);
-    });
-
-    marcas.forEach(function (item) {
-        defineMarca(item);
-    });
 }
 
 function addFabricante(row) {
@@ -77,6 +69,16 @@ function addFabricante(row) {
     fabricantes[item.id] = item;
 }
 
+function defineMarcasFabricantes() {    
+    fabricantes.forEach(function (item) {
+        defineFabricante(item);
+    });
+
+    marcas.forEach(function (item) {
+        defineMarca(item);
+    });
+}
+
 function defineFabricante(item) {
     var item2 = {
         id: item.id,
@@ -320,6 +322,8 @@ function getProvinciaPais(fullname) {
 module.exports = {
     initialize: initialize,
     loadMarcasFabricantes: loadMarcasFabricantes,
+    loadRemoteMarcas: loadRemoteMarcas,
+    defineMarcasFabricantes: defineMarcasFabricantes,
     loadCategorias: loadCategorias,
     loadProductos: loadProductos,
     loadCiudades: loadCiudades,

diff --git a/samples/preciosa/test/marcas.js b/samples/preciosa/test/marcas.js
@@ -21,4 +21,14 @@ exports['Load brands, companies'] = function (test) {
     test.ok(result.id);
     test.ok(result.nombre);
 };
+
+exports['Load remote brands'] = function (test) {
+    test.async();
+
+    preciosa.loadRemoteMarcas(function (err, data) {
+        test.equal(err, null);
+        test.done();
+    });
+};
+
 
diff --git a/samples/preciosa/test/preciosa.js b/samples/preciosa/test/preciosa.js
@@ -7,6 +7,7 @@ exports['initialize'] = function (test) {
 
 exports['load brands and companies'] = function (test) {
     preciosa.loadMarcasFabricantes();
+    preciosa.defineMarcasFabricantes();
 
     var marca = preciosa.getMarca(1);
     test.ok(marca);