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.

Sign up for GitHub

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

Jump to bottom

Add unit tests for FirebaseDB methods #80

Merged
merged 16 commits into from
Apr 2, 2024
Merged

Add unit tests for FirebaseDB methods #80

merged 16 commits into from
Apr 2, 2024

Conversation

mario-bermonti
Copy link
Owner

Description

Includes new dependencies to mock Firebase and be able to run unittests and the unittests.

Related Issue

closes # 79

Type of Change

  • 📚 Examples / docs / tutorials / dependencies update
  • 🐞 Bug fix (non-breaking change that fixes an issue)
  • 🔧 Maintenance (non-breaking change that improves code)
  • 🥂 Improvement (non-breaking change which improves an existing feature)
  • 🚀 New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to change)
  • 🔐 Security fix
  • 🔐 Improvements to the CI workflow

Checklist

mario-bermonti and others added 16 commits April 1, 2024 09:59
@mario-bermonti
Merge pull request #76 from mario-bermonti:mario-bermonti/issue70
5cda062 
Add functionality for automatically converting base models to json (Map<String, dynamic>)
@mario-bermonti
improve: Use toJson() in FirebaseDB.addDevice()
1ea6c4f
@mario-bermonti
improve: Use toJson() in FirebaseDB.addSession()
fd3a18c
@mario-bermonti
improve: Use toJson() in FirebaseDB.addTrial()
307e4da
@mario-bermonti
improve: Use toJson() in FirebaseDB.addTrials()
ab0c3b1
@mario-bermonti
Merge pull request #78 from mario-bermonti:mario-bermonti/issue77
9492ff6 
Mario-bermonti/issue77
@mario-bermonti
deps: Add fake_cloud_firestore deps
36a5ca1
@mario-bermonti
improve(firebase): Make FirebaseDB.db public
199e439
@mario-bermonti
test(firebase): Add setup() to init FirebaseDB
62db466 
test(firebase): Add setup() to FirebaseDB tests

test(firebase): Init db in setup()
@mario-bermonti
test(firebase): Define firebase session path var
07dbe03
@mario-bermonti
test(firebase): Add test for FirebaseDB.addSession
f71cd4e
@mario-bermonti
test(firebase): Add test for FirebaseDB.addDevice
c64f081
@mario-bermonti
test(firebase): Add test for FirebaseDB.addTrial
4f3ff37 
test(firebase): Add test for FirebaseDB.addTrial

test(firebase): Use path variable

improve name for trials collection ref

improve name of ref to query snapshot - trial

fix name trial data from firebase

Merge getting collection ref and doc

Remove print() from test
@mario-bermonti
test(firebase): Add test for FirebaseDB.addTrials
2e0e73c 
test(firebase): Add test for FirebaseDB.addTrials

test(firebase): Add test for FirebaseDB.addTrials

improve names local trials

Remove print() from tests
@mario-bermonti
test(firebase): Rename data from firebase
5467e5e
@mario-bermonti
Merge branch 'main' into mario-bermonti/issue79
bc4943c
@mario-bermonti mario-bermonti merged commit d467ccd into main Apr 2, 2024
2 checks passed
@mario-bermonti mario-bermonti deleted the mario-bermonti/issue79 branch April 2, 2024 21:08
@mario-bermonti mario-bermonti mentioned this pull request Apr 3, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@mario-bermonti