openconfig / ygot / 1389

89%

push

travis-ci

 Add testutil options for JSON, and ignoring fields. (#257)

* Fix build failures due to incorrect package naming.

* Add testutil options for JSON, and ignoring fields.

 * (M) testutil/testutil.go
  - Add an options framework to testutil's comparisons which allows
    for specific fields to be diff'd using custom comparers, and
    for fields to be ignored.
  - Add a diff mechanism that compares IETF JSON values by unmarshalling
    the JSON into map[string]interface{} which is used by default to
    ensure that syntatic differences in valid but equal JSON do not
    cause the value to be considered invalid.
  - Add a diff mechanism that is schema aware in IETF JSON values, which
    unmarshals JSON into a schema tree root, and then diffs the
    resulting roots. This allows for comparisons between semantically
    equal but syntactically different JSON contents.
  - Add an option to ignore the timestamp field of a Notification.
 * (M) testutil/testutil_test.go
  - Add unit tests for above functionality.

* Update comparison implementation to avoid package circular loop.

We now have a testcmp package which contains comparison code that
is dependent on ygot+ytypes, and testutil is made flexible to
deal with custom comparers being supplied by the caller.

This also allowed a refactor to use ytypes.Schema rather than a
new type for specifying unmarshal filds.

* Run gofmt -w -s.

* Update .travis.yml to support current go master.

144 of 144 new or added lines in 3 files covered. (100.0%)

8956 of 9820 relevant lines covered (91.2%)

1809.59 hits per line