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

db tests for hardware create events #401

Merged
merged 3 commits into from
Dec 24, 2020

Conversation

gauravgahlot
Copy link
Contributor

Description

The PR adds DB tests for hardware events (CUD).

Checklist:

I have:

  • updated the documentation and/or roadmap (if required)
  • added unit or e2e tests
  • provided instructions on how to upgrade

@codecov
Copy link

codecov bot commented Dec 21, 2020

Codecov Report

Merging #401 (1b8cb2e) into master (06519be) will increase coverage by 0.41%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #401      +/-   ##
==========================================
+ Coverage   20.82%   21.24%   +0.41%     
==========================================
  Files          24       24              
  Lines        2170     2170              
==========================================
+ Hits          452      461       +9     
+ Misses       1678     1667      -11     
- Partials       40       42       +2     
Impacted Files Coverage Δ
db/events.go 36.00% <0.00%> (+36.00%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 06519be...1b8cb2e. Read the comment docs.

@gianarb
Copy link
Contributor

gianarb commented Dec 21, 2020

This is amazing @gauravgahlot !! Thank you for doing that! Do you mind adding another test that similarly to how we do for the template creates 10 hardware asynchronously?! I am very curious to see what happens under "load" :)

Signed-off-by: Gaurav Gahlot <gauravgahlot0107@gmail.com>
@gianarb
Copy link
Contributor

gianarb commented Dec 24, 2020

In order to make CI happy here you have to format JSON with prettier. You can run this command:

$ prettier --write db/testdata/hardware.json

Thanks

@gauravgahlot gauravgahlot marked this pull request as ready for review December 24, 2020 11:19
Signed-off-by: Gaurav Gahlot <gauravgahlot0107@gmail.com>
Signed-off-by: Gianluca Arbezzano <gianarb92@gmail.com>
@gianarb gianarb added the ready-to-merge Signal to Mergify to merge the PR. label Dec 24, 2020
@mergify mergify bot merged commit 65a446c into tinkerbell:master Dec 24, 2020
@gauravgahlot gauravgahlot deleted the events-tests branch December 24, 2020 16:06
@gauravgahlot gauravgahlot changed the title db tests for hardware events db tests for hardware create events Dec 24, 2020
@mmlb mmlb removed the ready-to-merge Signal to Mergify to merge the PR. label Jan 25, 2021
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

Successfully merging this pull request may close these issues.

3 participants