metac icon indicating copy to clipboard operation
metac copied to clipboard

expose fields of structure to aid in unit testing

Open AmitKumarDas opened this issue 5 years ago • 0 comments

Take for example following structure used in dynamic/clientset/clientset.go file

type Clientset struct {
  config           rest.Config
  discoveryManager *dynamicdiscovery.APIResourceDiscovery
  dynamicClient    dynamic.Interface
}

Above structure does not provide any solution for the packages that depend on above struct during this package' unit tests. It will be easier to test if these fields were public. Alternate way will be to expose an interface for each & every public method of this struct. However, it does not seem the most idiomatic way to do.

AmitKumarDas avatar Jun 01 '20 06:06 AmitKumarDas