191
100%

Ran 24 Apr 2016 05:42PM UTC

Jobs 3

Files 3

Run time 18s

Badge

Embed ▾

pending completion

Build # 191

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%)

21.75 hits per line

Jobs

ID	Job ID	Ran	Coverage
1	191.1	24 Apr 2016 05:43PM UTC	100.0	Travis Job 191.1
2	191.2	24 Apr 2016 05:42PM UTC	100.0	Travis Job 191.2
3	191.3	24 Apr 2016 05:42PM UTC	100.0	Travis Job 191.3

Source Files on build 191

Detailed source file information is not available for this build.

npm / etcetera / 191
100%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 191

npm / etcetera / 191 100%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 191

npm / etcetera / 191
100%

README BADGES
x