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

Add int tests for test lab, remote config, and fireperf #5200

Merged

Conversation

colerogers
Copy link
Contributor

@colerogers colerogers commented Nov 1, 2022

  • Bumps the sdk to the latest version
  • Adds in preserveExternalChanges to tests
  • Adds test lab, remote config, and fireperf integration tests

@codecov-commenter
Copy link

codecov-commenter commented Nov 1, 2022

Codecov Report

Base: 56.27% // Head: 56.27% // No change to project coverage 👍

Coverage data is based on head (c97ff0f) compared to base (e984a6c).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #5200   +/-   ##
=======================================
  Coverage   56.27%   56.27%           
=======================================
  Files         309      309           
  Lines       20820    20820           
  Branches     4224     4224           
=======================================
  Hits        11716    11716           
  Misses       8092     8092           
  Partials     1012     1012           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@colerogers colerogers force-pushed the colerogers.add-int-tests-remoteconfig-testlab-fireperf branch from 2234dd4 to eac3e92 Compare November 2, 2022 17:39
@colerogers colerogers marked this pull request as ready for review November 21, 2022 21:28
@colerogers
Copy link
Contributor Author

Passing workflow run on this branch - https://github.com/firebase/firebase-tools/actions/runs/3517527484

Copy link
Contributor

@taeold taeold left a comment

Choose a reason for hiding this comment

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

🔥

Can you add a link to the successful test run on this branch to leave proof that this change has at least one passing result?

package.json Outdated
@@ -93,6 +93,7 @@
]
},
"dependencies": {
"@firebase/logger": "^0.3.4",
Copy link
Contributor

Choose a reason for hiding this comment

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

Hmm this is adding new dep to the CLI. Is this intentional or should this be a dev dep?

@colerogers
Copy link
Contributor Author

@colerogers colerogers enabled auto-merge (squash) December 2, 2022 15:55
@colerogers colerogers merged commit 7b070b0 into master Dec 2, 2022
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.

None yet

4 participants