Experimental conformance suite: Extended skipped tests not being reported

[sunjayBhatia]

With a conformance suite config similar to the below:

	cSuite, err := suite.NewExperimentalConformanceTestSuite(suite.ExperimentalConformanceOptions{
		Options: suite.Options{
                        ...
                        // Note this vs. using the EnableAllSupportedFeatures and ExemptFeatures field
			SupportedFeatures: suite.AllFeatures.Delete(suite.MeshCoreFeatures.UnsortedList()...),
			SkipTests: []string{
				tests.HTTPRouteRedirectPortAndScheme.ShortName,
			},
		},
		Implementation: conformance_v1alpha1.Implementation{
                         ...
		},
		ConformanceProfiles: sets.New(
			suite.HTTPConformanceProfileName,
			suite.TLSConformanceProfileName,
		),
	})

The resulting conformance report does not show any tests are skipped:

        apiVersion: gateway.networking.k8s.io/v1alpha1
        date: "2023-07-19T20:34:17Z"
        gatewayAPIVersion: TODO
        implementation:
          contact:
          - '@projectcontour/maintainers'
          organization: projectcontour
          project: contour
          url: https://github.com/projectcontour/contour
          version: ""
        kind: ConformanceReport
        profiles:
        - core:
            result: success
            statistics:
              Failed: 0
              Passed: 10
              Skipped: 0
            summary: ""
          name: TLS
        - core:
            result: success
            statistics:
              Failed: 0
              Passed: 28
              Skipped: 0
            summary: ""
          extended:
            result: success
            statistics:
              Failed: 0
              Passed: 9
              Skipped: 0
            summary: ""
            supportedFeatures:
            - HTTPRouteQueryParamMatching
            - HTTPRouteSchemeRedirect
            - HTTPRouteHostRewrite
            - HTTPRoutePathRewrite
            - HTTPRouteRequestMirror
            - HTTPResponseHeaderModification
            - HTTPRoutePortRedirect
            - HTTPRoutePathRedirect
            - HTTPRouteMethodMatching
          name: HTTP

[slayer321]

Hey @shaneutt, I did like to work on this issue. Will need some pointers how can I start working on it.

[shaneutt]

I'm sorry I made a mistake with this one. I accepted it but now in retrospect as I look at it again, I think this was intended behavior @sunjayBhatia: the idea being that extended tests are optional, so you don't "opt out" of them, you only "opt in". 🤔

I would be curious to hear what other conformance reviewers and approvers think about making this change?

/cc @arkodg @mlavacca @michaelbeaumont @LiorLieberman @Xunzhuo