influxdb icon indicating copy to clipboard operation
influxdb copied to clipboard

test: remove group skips

Open onelson opened this issue 3 years ago • 1 comments

  • Refs https://github.com/influxdata/flux/pull/5172

Required checklist

  • [ ] Sample config files updated (both /etc folder and NewDemoConfig methods) (influxdb and plutonium)
  • [ ] openapi swagger.yml updated (if modified API) - link openapi PR
  • [ ] Signed CLA (if not already signed)

Description

In https://github.com/influxdata/flux/pull/5172 various tests which have been skipped long-term were repaired. This diff removes them from the skip list during the flux integration tests run as a part of CI here.

Context

Adds additional coverage to the flux integration tests.

Affected areas (delete section if not relevant):

CI, specifically test selection for fluxtest.

Note for reviewers:

Check the semantic commit type:

  • Feat: a feature with user-visible changes
  • Fix: a bug fix that we might tell a user “upgrade to get this fix for your issue”
  • Chore: version bumps, internal doc (e.g. README) changes, code comment updates, code formatting fixes… must not be user facing (except dependency version changes)
  • Build: build script changes, CI config changes, build tool updates
  • Refactor: non-user-visible refactoring
  • Check the PR title: we should be able to put this as a one-liner in the release notes

onelson avatar Sep 09 '22 18:09 onelson

Tests are still failing, but it looks like this is because the fixes were in flux v0.182 or v0.183, but we still have v0.181 here.

onelson avatar Sep 16 '22 19:09 onelson

:sweat_smile: bit of a shame to need to inline these, but I'd rather keep the coverage until we can work out something better.

onelson avatar Sep 27 '22 19:09 onelson