From fcbf5cc411de2f6195f53fe70c54458386fc588a Mon Sep 17 00:00:00 2001 From: Petr Vobornik Date: Mon, 8 Apr 2013 16:49:56 +0200 Subject: [PATCH] Spec modification by diff object https://fedorahosted.org/freeipa/ticket/3235 --- install/ui/src/freeipa/_base/Builder.js | 19 ++- install/ui/src/freeipa/_base/Spec_mod.js | 177 +++++++++++++++++++++++ install/ui/test/all_tests.html | 6 +- install/ui/test/build_tests.html | 22 +++ install/ui/test/build_tests.js | 119 +++++++++++++++ install/ui/test/index.html | 1 + install/ui/test/jsl.conf | 3 +- 7 files changed, 342 insertions(+), 5 deletions(-) create mode 100644 install/ui/src/freeipa/_base/Spec_mod.js create mode 100644 install/ui/test/build_tests.html create mode 100644 install/ui/test/build_tests.js diff --git a/install/ui/src/freeipa/_base/Builder.js b/install/ui/src/freeipa/_base/Builder.js index 10c544cce..9fc116ba8 100644 --- a/install/ui/src/freeipa/_base/Builder.js +++ b/install/ui/src/freeipa/_base/Builder.js @@ -21,8 +21,9 @@ define(['dojo/_base/declare', 'dojo/_base/array', 'dojo/_base/lang', - './construct' - ], function(declare, array, lang, construct) { + './construct', + './Spec_mod' + ], function(declare, array, lang, construct, Spec_mod) { var Builder = declare(null, { /** @@ -38,6 +39,11 @@ define(['dojo/_base/declare', */ registry: null, + /** + * Specification modifier + */ + spec_mod: null, + /** * Build object based on spec. * @@ -152,6 +158,9 @@ define(['dojo/_base/declare', if (preop_t === 'function') { cs.spec = preop(cs.spec || {}); } else if (preop_t === 'object') { + var temp = lang.clone(preop); + this.spec_mod.mod(cs.spec, temp); + this.spec_mod.del_rules(temp); lang.mixin(cs.spec, preop); } } @@ -159,6 +168,10 @@ define(['dojo/_base/declare', cs.spec = cs.spec || {}; + // do we want following?, remove? + this.spec_mod.mod(cs.spec, cs.spec); + this.spec_mod.del_rules(cs.spec); + if (cs.factory && typeof cs.factory === 'function') { obj = cs.factory(cs.spec); } else if (cs.constructor && typeof cs.constructor === 'function') { @@ -194,6 +207,8 @@ define(['dojo/_base/declare', spec = spec || {}; if (spec.registry) this.registry = spec.registry; + if (spec.spec_mod) this.spec_mod = spec.spec_mod; + else this.spec_mod = new Spec_mod(); } }); diff --git a/install/ui/src/freeipa/_base/Spec_mod.js b/install/ui/src/freeipa/_base/Spec_mod.js new file mode 100644 index 000000000..60d4b9229 --- /dev/null +++ b/install/ui/src/freeipa/_base/Spec_mod.js @@ -0,0 +1,177 @@ +/* Authors: + * Petr Vobornik + * + * Copyright (C) 2013 Red Hat + * see file 'COPYING' for use and warranty information + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation, either version 3 of the License, or + * (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with this program. If not, see . +*/ + +define(['dojo/_base/declare', + 'dojo/_base/lang' + ], function(declare, lang) { + + var Spec_mod = declare(null, { + + /** + * Modifies spec according to rules defined in diff object. + * + * Diff should have following structure: { + * $add: array of add rules + * $del: array of del rules + * $set: array of set rules + * } + * + * The order of modification is del, add, set. + * + * @param {Object} spec + * @param {Object} diff + */ + mod: function(spec, diff) { + + if (!diff) return spec; + + this.del(spec, diff.$del); + this.add(spec, diff.$add); + this.set(spec, diff.$set); + + return spec; + }, + + + /** + * Adds objects according to rules to array. + * + * A rule is a triple of path and a object and position to add: + * ['path.to.spec.array', {}, position] + * + */ + add: function(spec, rules) { + + return this._apply_rules(spec, rules, this._add); + }, + + /** + * Deletes objects according to rules from an array. + * + * A rule is a pair of path and delete conditions: + * ['path.to.spec.array', [ { name: 'foo'}, { name: 'baz'} ]] + * + * Deletes all objects with name 'baz' or 'foo'. + */ + del: function(spec, rules) { + + return this._apply_rules(spec, rules, this._del); + }, + + /** + * A rule is a pair of path and a object to set. + * ['path.to.spec.property', {}] + */ + set: function(spec, rules) { + + return this._apply_rules(spec, rules, this._set); + }, + + /** + * Removes all rule props + */ + del_rules: function(diff) { + delete diff.$add; + delete diff.$del; + delete diff.$set; + }, + + _apply_rules: function(spec, rules, method) { + if (!lang.isArrayLike(rules)) return spec; + + for (var i=0; i arr.length) pos = arr.length; + + arr.splice(pos, 0, value); + return spec; + }, + + _del: function(spec, rule) { + + var path = rule[0]; + var conds = rule[1]; + var arr = lang.getObject(path, false, spec); + + if (!arr) return spec; + + var del = []; + var i,j; + + for (i=0; i diff --git a/install/ui/test/build_tests.html b/install/ui/test/build_tests.html new file mode 100644 index 000000000..7b61a9ce5 --- /dev/null +++ b/install/ui/test/build_tests.html @@ -0,0 +1,22 @@ + + + + IPA utils test suite + + + + + + + + +

IPA Build test suite

+

+
+

+
    +
    + + \ No newline at end of file diff --git a/install/ui/test/build_tests.js b/install/ui/test/build_tests.js new file mode 100644 index 000000000..11c5a03e5 --- /dev/null +++ b/install/ui/test/build_tests.js @@ -0,0 +1,119 @@ +/* Authors: + * Petr Vobornik + * + * Copyright (C) 2013 Red Hat + * see file 'COPYING' for use and warranty information + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation, either version 3 of the License, or + * (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with this program. If not, see . + */ + +define([ + 'freeipa/_base/Spec_mod'], + function(Spec_mod) { return function() { + + +module('build',{ + + setup: function() { + }, + teardown: function() { + } +}); + +test('Testing builder', function() { +}); + +test('Testing Spec_mod', function() { + + var sm = new Spec_mod(); + + var spec = { + foo: { + arr1: [ + { name: 'i1', a: 'b' }, + { name: 'i2', a: 'b' }, + { name: 'i3', a: 'c' }, + { name: 'i4', a: 'c' }, + { name: 'i5', a: 'a' } + ], + arr2: ['item1'] + }, + baz: { + bar: 'a' + }, + bar: 'b' + }; + + var diff = { + $add: [ + ['foo.arr1', { name: 'foo', a: 'c' }], + ['foo.arr2', 'item2'], + ['foo.arr2', { name: 'foo' }], + ['arr3', 'a'] //creates new array + ], + $del: [ + [ + 'foo.arr1', + [ + { name: 'i1' }, //match + { a: 'c'}, // 2 matches + { name: 'i2', a:'c' }, //no match + { name: 'i5', a:'a' } // match + ] + ], + [ 'foo.arr2', ['item1'] ] //match + ], + $set: [ + [ 'arr4', ['b'] ], // new array in spec + [ 'baz.bar', 'c'], //overwrite 'a' + [ 'baz.baz.baz', 'a'], // new property + [ 'bar', { foo: 'baz' }] // replace string by object + ] + }; + + var ref = { + foo: { + arr1: [ + { name: 'i2', a: 'b' }, + { name: 'foo', a: 'c'} + ], + arr2: [ + 'item2', { name: 'foo' } + ] + }, + arr3: [ 'a' ], + baz: { + bar: 'c', + baz: { baz: 'a' } + }, + bar: { foo: 'baz' }, + arr4: ['b'] + }; + + sm.mod(spec, diff); + + deepEqual(spec, ref, 'Complex Modification'); + + spec = { + a: [ 'a1', 'a2', 'a3' ] + }; + var rules = [[ 'a', 'new', 1]]; + sm.add(spec, rules); + + deepEqual(spec, { a: ['a1', 'new', 'a2', 'a3'] }, 'Add on position'); +}); + + + +};}); \ No newline at end of file diff --git a/install/ui/test/index.html b/install/ui/test/index.html index 0a135188e..948f9a448 100644 --- a/install/ui/test/index.html +++ b/install/ui/test/index.html @@ -35,6 +35,7 @@
  1. Widget Test Suite
  2. IP Addresses Test Suite
  3. Utils Test Suite +
  4. Build Test Suite diff --git a/install/ui/test/jsl.conf b/install/ui/test/jsl.conf index 94ac10c50..ba5bb2079 100644 --- a/install/ui/test/jsl.conf +++ b/install/ui/test/jsl.conf @@ -148,4 +148,5 @@ +process ordered_map_tests.js +process widget_tests.js +process ip_tests.js -+process utils_tests.js \ No newline at end of file ++process utils_tests.js ++process build_tests.js \ No newline at end of file