580
89%

Ran 17 Oct 2017 11:17PM UTC

Jobs 2

Files 37

Run time 2min

Badge

Embed ▾

pending completion

Build # 580

Build Type

push

travis-ci

Committed by

web-flow

Commit Message

Fix bug whereby BuildEmptyTree + Merge results in empty struct. (#96)

* Fix bug whereby BuildEmptyTree + Merge results in empty struct.

 * (M) ygot/struct_validation_map.go
   - Fix an issue whereby an empty dst struct ptr was always used
     (created with reflect.New) in the case that structs were being
     merged. This means that if BuildEmptyTree had been used on the
     structs being merged, then even empty branches would be
     overwritten.
   - Return an error if a non-struct ptr is non-nil in two different
     structs.
  * (M) ygot/struct_validation_map_test.go
   - Add testing for returned errors.
   - Add testing to validate the BuildEmptyTree issue above.

* Add comment related to merging of structs to function comment.

Run Details

5864 of 6650 relevant lines covered (88.18%)

159.75 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	580.1	17 Oct 2017 11:17PM UTC	0	88.18	Travis Job 580.1
2	580.2	17 Oct 2017 11:19PM UTC	0	88.18	Travis Job 580.2

Source Files on build 580

Detailed source file information is not available for this build.

openconfig / ygot / 580
89%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 580

openconfig / ygot / 580 89%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 580

openconfig / ygot / 580
89%

README BADGES
x