Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pipeline test config panics if fields is empty #2051

Open
jsoriano opened this issue Aug 28, 2024 · 0 comments
Open

Pipeline test config panics if fields is empty #2051

jsoriano opened this issue Aug 28, 2024 · 0 comments

Comments

@jsoriano
Copy link
Member

jsoriano commented Aug 28, 2024

Parsing pipeline test config panics if the fields attribute is empty. This can be reproduced for example by trying to skip a test with something like the following:

skip:
  reason: New value in network.community_id when running on newer versions
  link: https://github.com/elastic/integrations/issues/10904

Configuring it like this works:

skip:
  reason: New value in network.community_id when running on newer versions
  link: https://github.com/elastic/integrations/issues/10904
fields: 
  tags: []

It cannot be reproduced configuring the common test config only. Maybe the issue is related to merging the common test config and the config specific to the test.

Seen in elastic/integrations#10919, trying to skip pipeline tests in the cisco_ios package.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant