Build has been canceled!

1
59%
master: 60%

Ran 09 Oct 2019 09:07PM UTC

Files 53

Run time 5s

Badge

Embed ▾

Committed 09 Oct 2019 09:02PM UTC coverage: 59.479% (+0.7%) from 58.763%

Job # GO111MODULE=on

Build Type

Pull #87

travis-ci

Committed by

web-flow

Commit Message

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.

Pull Request Pull Request #87: Properly handle multiple blocks with same name in a single file

Run Details

3586 of 6029 relevant lines covered (59.48%)

80.49 hits per line

coveooss / terragrunt / 701 / 1
59%
master: 60%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 701.1 (GO111MODULE=on)

coveooss / terragrunt / 701 / 1 59% master: 60%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 701.1 (GO111MODULE=on)

coveooss / terragrunt / 701 / 1
59%
master: 60%

README BADGES
x