1
100%
master: 100%

Ran 24 Apr 2016 05:43PM UTC

Files 3

Run time 0s

Badge

Embed ▾

Committed 24 Apr 2016 05:41PM UTC coverage: 100.0%. Remained the same

Job # 191.1

Build Type

push

travis-ci

Committed by

ceejbot

Commit Message

A major change to transform() with tests (#39)

* A major change to transform() with tests

The new behavior is documented at the top of lib/transform.js.

`config` might look like this:

```js
{
     'var.app.group': 'foo',
     'var.app.group2': 'bar',
     'var.other': 'baz',
     var: 'qux',
     var2: 'yo'
}
```

We want the following output:

```js
transform(config, 'app', 'group2') => { var: bar, var2: yo }
transform(config, 'app', 'group') => { var: foo, var2: yo }
transform(config, 'other', 'some-other-group') => { var: baz, var2: yo }
transform(config, 'some-other-app') => { var: qux, var2: yo }
```

Keys not relevant to the app & group specified should be trimmed. Keys that ARE
relevant remain, but only in their top-level forms. (This is the only form used
by the configuration templates, after all!) In other words, only not-dots key
forms remain in the output.

WHY?

In production, some config keys are set only in fully-qualified versions,
without any top level versions. Those keys weren't being looked up. Also, keys
that were set without group vars being present were being looked up in an unexpected
manner. E.g., `mail_secret.npm-website` was being resolved to `mail_secret` and NOT
to the set key `mail_secret.npm-website` which existed in etcd!

* Lint doesn't like my C-isms.

Run Details

52 of 52 relevant lines covered (100.0%)

7.25 hits per line

npm / etcetera / 191 / 1
100%
master: 100%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 191.1

npm / etcetera / 191 / 1 100% master: 100%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 191.1

npm / etcetera / 191 / 1
100%
master: 100%

README BADGES
x