kubectl: remove redundant test

[seans3]

• Removes cmd_printing_test.go
• This file has two tests which both test the "IllegalPackageSourceChecker".
• The "IllegalPackageSourceChecker" is located in another repo (cli-runtime), and it is unit tested there: https://github.com/kubernetes/cli-runtime/blob/master/pkg/genericclioptions/printers/sourcechecker_test.go
• Testing this in kubectl requires an internal type, which is not allowed.

helps fix:
kubernetes/kubectl#80

NONE

/sig cli
/kind cleanup
/area kubectl
/assign

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: seans3

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• pkg/kubectl/OWNERS [seans3]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[seans3]

/cc @soltysh
/cc @apelisse

[apelisse]

LGTM, you have to update bazel BUILD file

[seans3]

/cc @juanvallejo

[juanvallejo]

@seans3 I think we should move these two test cases to https://github.com/kubernetes/cli-runtime/blob/master/pkg/genericclioptions/printers/sourcechecker_test.go. We can also just delete the customcolumn printer pieces from this file

Nevermind, I see it's using the internal api

[juanvallejo]

/lgtm

[soltysh]

/priority cleanup
/milestone v1.13

[soltysh]

/lgtm