711
59%
master: 60%

Ran 15 Oct 2019 02:13PM UTC

Jobs 1

Files 53

Run time 4s

Badge

Embed ▾

Committed 10 Oct 2019 11:24AM UTC coverage: 59.493% (+0.6%) from 58.868%

Build # 711

Build Type

push

travis-ci

Committed by

web-flow

Commit Message

Properly handle multiple blocks with same name in a single file (#87)

* Properly handle multiple blocks with same name in a single file

If we declare a block with the same name, the normal behavior is to overwrite the previous definition with the latest one. That works if blocks are defined in separated files. But it did not work properly when blocks were declared in the same file.

I fixed this in the generic merge function, but that only worked if multiple blocks were declared in an included file but not for the main configuration file. So, I forced the main file to also go through the merge process (starting with an empty structure).

I also discovered a test `no-tf-variable` that was working even if it used the same block many times in the configuration. Fixing this problem caused that specific test to fail.

To ensure that we don't break existing configuration without being noticed, I added a warning when a block is overridden by another block with the same name in a file.

* Changes following review comments

Run Details

3588 of 6031 relevant lines covered (59.49%)

80.56 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	711.1 (GO111MODULE=on)	15 Oct 2019 02:13PM UTC	0	59.49	Travis Job 711.1

Source Files on build 711

Coverage	∆	File	Lines	Relevant	Covered	Missed	Hits/Line

coveooss / terragrunt / 711
59%
master: 60%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 711

coveooss / terragrunt / 711 59% master: 60%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 711

coveooss / terragrunt / 711
59%
master: 60%

README BADGES
x