Skip to content

chore(config): added benchmark tests - #196

Draft
tirirote wants to merge 33 commits into
developfrom
config-test-benchmark
Draft

chore(config): added benchmark tests#196
tirirote wants to merge 33 commits into
developfrom
config-test-benchmark

Conversation

@tirirote

@tirirote tirirote commented May 19, 2025

Copy link
Copy Markdown
Contributor

No description provided.

tirirote and others added 9 commits April 16, 2025 12:34
@tirirote @alvarobernal2412
test(config): added controller (#99)
88c1025 
---------

Co-authored-by: Alvaro Jesus Bernal Caunedo <80347035+alvarobernal2412@users.noreply.github.com>
Co-authored-by: [[alvarobernal2412]] <alvarobc2412@gmail.com>
@tirirote
Merge branch 'develop' of https://github.com/statuscompliance/status-…
f243329 
…backend into develop
@tirirote
Merge branch 'develop' of https://github.com/statuscompliance/status-…
f0dac7a 
…backend into develop
@tirirote
Merge branch 'develop' of https://github.com/statuscompliance/status-…
1b3379a 
…backend into develop
@tirirote
Merge branch 'develop' of https://github.com/statuscompliance/status-…
9bf5842 
…backend into develop
@tirirote
Merge branch 'develop' of https://github.com/statuscompliance/status-…
3992ac1 
…backend into develop
@tirirote
Merge branch 'develop' of https://github.com/statuscompliance/status-…
d8a5eec 
…backend into develop
@tirirote
Merge branch 'develop' of https://github.com/statuscompliance/status-…
3c0103a 
…backend into develop
@tirirote
chore(config): added benchmark tests
848b854
@github-actions github-actions Bot added the 📦 Dependencies Pull requests that update a dependency file label May 19, 2025
@github-actions github-actions Bot added the 😎 size/m Pull request size M label May 20, 2025
@coveralls

coveralls commented May 20, 2025 •
edited
Loading

Copy link
Copy Markdown

Coverage Status

coverage: 68.541%. remained the same
when pulling 1d647d3 on config-test-benchmark
into 4c3deeb on develop.

@github-actions github-actions Bot added 😖 size/l Pull request size L and removed 😎 size/m Pull request size M labels May 21, 2025
ferferga
ferferga requested changes May 22, 2025
View reviewed changes
Comment thread benchmark/nodered.benchmark.ts Outdated
Comment thread benchmark/utils/nodeRedUtils.ts Outdated
Comment thread benchmark/utils/nodeRedUtils.ts Outdated
Comment thread benchmark/utils/nodeRedUtils.ts Outdated
ferferga
ferferga approved these changes May 23, 2025
View reviewed changes

@ferferga ferferga left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM as an starting point

@ferferga
ferferga requested a review from alvarobernal2412 May 23, 2025 10:41
ferferga
ferferga requested changes Jun 2, 2025
View reviewed changes

@ferferga ferferga left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The benches aren't working. The beforeAll hooks are not getting executed and the displayed times are NaN

@github-actions github-actions Bot added the 😎 size/m Pull request size M label Jun 11, 2025
@sonarqubecloud

sonarqubecloud Bot commented Jun 11, 2025

Copy link
Copy Markdown

Quality Gate Passed Quality Gate passed

Issues
2 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@alvarobernal2412
alvarobernal2412 marked this pull request as draft June 16, 2025 06:49
@alvarobernal2412
alvarobernal2412 force-pushed the develop branch 11 times, most recently from 6579fc5 to 99e97b2 Compare June 23, 2025 08:57
@alvarobernal2412
alvarobernal2412 force-pushed the develop branch 10 times, most recently from 79a5bc3 to f989597 Compare September 29, 2025 09:25
@alvarobernal2412
alvarobernal2412 force-pushed the develop branch 2 times, most recently from 17117c5 to 743c84f Compare December 4, 2025 09:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ferferga ferferga ferferga requested changes
@alvarobernal2412 alvarobernal2412 Awaiting requested review from alvarobernal2412

Requested changes must be addressed to merge this pull request.

Assignees

No one assigned

Labels

⚙️ Configuration Indicates the scope is related to the configuration 📦 Dependencies Pull requests that update a dependency file 😖 size/l Pull request size L 😎 size/m Pull request size M

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@tirirote @coveralls @ferferga @alvarobernal2412