708
60%

Ran 10 Oct 2019 11:29AM UTC

Jobs 1

Files 53

Run time 3s

Badge

Embed ▾

Committed 10 Oct 2019 11:24AM UTC coverage: 59.509% (+0.7%) from 58.763%

Build # 708

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

3589 of 6031 relevant lines covered (59.51%)

80.32 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	708.1 (GO111MODULE=on)	10 Oct 2019 11:29AM UTC	0	59.51	Travis Job 708.1

Source Files on build 708

Detailed source file information is not available for this build.

coveooss / terragrunt / 708
60%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 708

coveooss / terragrunt / 708 60%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 708

coveooss / terragrunt / 708
60%

README BADGES
x