# About Metricbeat modules integration test

**URL:** <https://discuss.elastic.co/t/about-metricbeat-modules-integration-test/354361>\
**Category:** Beats\
**Tags:** docker, metricbeat\
**Created:** [February 28, 2024, 3:11pm UTC](https://discuss.elastic.co/t/about-metricbeat-modules-integration-test/354361 "2024-02-28T15:11:56Z")\
**Posts on this page:** 5\
**Page:** 1

<div class="post-metadata">

**Author:** ![zeynepyz](https://sea2.discourse-cdn.com/elastic/user_avatar/discuss.elastic.co/zeynepyz/32/114656_2.png) [@zeynepyz](https://discuss.elastic.co/u/zeynepyz)\
**Post date:** [February 28, 2024, 3:11pm UTC](https://discuss.elastic.co/t/about-metricbeat-modules-integration-test/354361/1 "2024-02-28T15:11:56Z")

</div>

Hello i was trying to contribute metricbeat modules because of my school project. I made some things now i can see metrics through kibana and in logs on my local. But there is something that i can't done and that is integration test. My unit test is working fine it has %90 coverage. But i got stuck on integration test. What should i do now? Should i raise pr and get help about it or share my codes on here and trying to fix the error?

---

<div class="post-metadata">

**Author:** ![zeynepyz](https://sea2.discourse-cdn.com/elastic/user_avatar/discuss.elastic.co/zeynepyz/32/114656_2.png) [@zeynepyz](https://discuss.elastic.co/u/zeynepyz)\
**Post date:** [February 29, 2024, 9:41am UTC](https://discuss.elastic.co/t/about-metricbeat-modules-integration-test/354361/2 "2024-02-29T09:41:04Z")

</div>

hello can you help me? @kcreddy

---

<div class="post-metadata">

**Author:** ![kcreddy](https://avatars.discourse-cdn.com/v4/letter/k/6f9a4e/32.png) [@kcreddy](https://discuss.elastic.co/u/kcreddy)\
**Post date:** [March 4, 2024, 9:59am UTC](https://discuss.elastic.co/t/about-metricbeat-modules-integration-test/354361/3 "2024-03-04T09:59:36Z")

</div>

Hey @zeynepyz yes, please create PR and appropriate team should be assigned and you can discuss the failing tests there.

---

<div class="post-metadata">

**Author:** ![zeynepyz](https://sea2.discourse-cdn.com/elastic/user_avatar/discuss.elastic.co/zeynepyz/32/114656_2.png) [@zeynepyz](https://discuss.elastic.co/u/zeynepyz)\
**Post date:** [March 4, 2024, 7:33pm UTC](https://discuss.elastic.co/t/about-metricbeat-modules-integration-test/354361/4 "2024-03-04T19:33:45Z")

</div>

> <https://github.com/elastic/beats/pull/38179>
>
> \<!-- Type of change
> Please label this PR with one of the following labels, depe…nding on the scope of your change:
> \- Bug
> \- Enhancement
> \- Breaking change
> \- Deprecation
> \- Cleanup
> \- Docs
> \--\>
> 
> \## Proposed commit message
> I created new metricbeat module named k6. It takes k6 metrics via Rest API and indexes them then send them to elasticsearch. While doing this project, I schematized the metrics using mapstr. I am trying to improve myself and i want my github to fill with good projects that's why i did this.
> 
> \<!-- Mandatory
> 
> Please explain:
> 
> \- WHAT: patterns used, algorithms implemented, design architecture, message processing, etc.
> \- WHY: the rationale/motivation for the changes
> 
> This text will be pasted into the squash dialog when the change is committed and will be
> a long term historical record of the change to help future contributors understand the
> change, please help them by making it clear and comprehensive, they may be you.
> 
> If the commit title is adequate to describe both of these things, The text here may be omitted
> or replaced with "See title". The title of the PR will be used as the commit message title when
> the merge is made and the "See title" marker will be removed if present.
> 
> The text here and the PR title will be subject to the PR review process.
> \--\>
> 
> \## Checklist
> 
> \<!-- Mandatory
> Add a checklist of things that are required to be reviewed in order to have the PR approved
> 
> List here all the items you have verified BEFORE sending this PR. Please DO NOT remove any item, striking through those that do not apply. (Just in case, strikethrough uses two tildes. ~~Scratch this.~~ )
> \--\>
> 
> \- \[x\] My code follows the style guidelines of this project
> \- \[x\] I have commented my code, particularly in hard-to-understand areas
> \- \[x\] I have made corresponding changes to the documentation
> \- \[\] I have made corresponding change to the default configuration files
> \- \[x\] I have added tests that prove my fix is effective or that my feature works
> \- \[\] I have added an entry in \`CHANGELOG.next.asciidoc\` or \`CHANGELOG-developer.next.asciidoc\`.
> 
> 
> 
> 
> \## Screenshots
> 
> \<!-- Optional
> Add here screenshots about how the project will be changed after the PR is applied. They could be related to web pages, terminal, etc, or any other image you consider important to be shared with the team.
> \--\>
> This is the expected error when k6 is not running.
> 
> !\[Screenshot from 2024-01-29 10-01-56\](https://github.com/elastic/beats/assets/56611927/57c9d2b3-b0cc-4235-8e96-43013c816039)
> 
> \## Logs
> 
> \<!-- Recommended
> Paste here output logs discovered while creating this PR, such as stack traces or integration logs, or any other output you consider important to be shared with the team.
> \--\>
> This will show that the k6 module is working. You will receive metrics too.
> 
> {"log.level":"warn","@timestamp":"2024-03-04T22:01:07.399+0300","log.logger":"cfgwarn","log.origin":{"function":"github.com/elastic/beats/v7/metricbeat/module/k6/k6metricset.New","file.name":"k6metricset/k6metricset.go","file.line":65},"message":"BETA: The k6 k6metricset metricset is beta.","service.name":"metricbeat","ecs.version":"1.6.0"}

Here is my pr. I wanna share this too.

---

<div class="post-metadata">

**Author:** ![system](https://us1.discourse-cdn.com/elastic/original/3X/1/a/1ac57faf039f6b580b3f104ef42a2a89e41014de.png) [@system](https://discuss.elastic.co/u/system)\
**Post date:** [April 1, 2024, 9:34pm UTC](https://discuss.elastic.co/t/about-metricbeat-modules-integration-test/354361/5 "2024-04-01T21:34:13Z")

</div>

This topic was automatically closed 28 days after the last reply. New replies are no longer allowed.