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

Fix storage gcs conformance tests #4869

Merged
merged 54 commits into from
Aug 16, 2022
Merged

Fix storage gcs conformance tests #4869

merged 54 commits into from
Aug 16, 2022

Conversation

tonyjhuang
Copy link
Contributor

@tonyjhuang tonyjhuang commented Aug 13, 2022

Description

Scenarios Tested

Sample Commands

@codecov-commenter
Copy link

codecov-commenter commented Aug 13, 2022

Codecov Report

Merging #4869 (3f1f3cd) into master (5728643) will decrease coverage by 0.19%.
The diff coverage is 4.29%.

@@            Coverage Diff             @@
##           master    #4869      +/-   ##
==========================================
- Coverage   56.93%   56.73%   -0.20%     
==========================================
  Files         288      288              
  Lines       19062    19110      +48     
  Branches     3775     3792      +17     
==========================================
- Hits        10853    10843      -10     
- Misses       7290     7350      +60     
+ Partials      919      917       -2     
Impacted Files Coverage Δ
src/emulator/storage/apis/firebase.ts 4.05% <0.00%> (-0.04%) ⬇️
src/emulator/storage/apis/gcloud.ts 5.33% <0.00%> (-1.09%) ⬇️
src/emulator/storage/files.ts 23.75% <0.00%> (-1.15%) ⬇️
src/emulator/storage/upload.ts 72.72% <33.33%> (-5.28%) ⬇️
src/emulator/storage/metadata.ts 42.05% <71.42%> (-4.62%) ⬇️

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

Berlioz and others added 7 commits August 16, 2022 12:09
* deploy working with using the default compute sa & running a GET request

* fix test cases

* formatter

* cleaning up code post merge

* add changelog entry
…ckend (#4873)

* Only start eventarc emulator when there is at least one emulatable backend

* Enable automatic port finding

* Add changelog

* Format changelog
Copy link
Contributor

@bkendall bkendall left a comment

Choose a reason for hiding this comment

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

This seems to touch a lot of files... is that intended?

package.json Outdated Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
Copy link
Contributor

@bkendall bkendall left a comment

Choose a reason for hiding this comment

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

Thanks for addressing those bits :)

Copy link
Contributor

@Yuangwang Yuangwang left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for the tests and refactor!

@tonyjhuang tonyjhuang enabled auto-merge (squash) August 16, 2022 19:53
@tonyjhuang tonyjhuang merged commit 3c35d8f into master Aug 16, 2022
@tonyjhuang tonyjhuang deleted the tonyjhuang/test-gcs branch August 16, 2022 20:33
christhompsongoogle pushed a commit that referenced this pull request Sep 30, 2022
* move

* copy

* move

* save

* pull out firebase and gcs tests

* internal

* env

* gcs sdk tests

* clean

* remove tests.ts

* lint

* remove redundant config

* persistence

* remove creds

* lint

* gcs endpoint auth

* fix firebase conformance tests

* revert

* clean

* shrinkwrap

* no async

* merge

* remove

* fix test and lint

* lint

* multipart

* fix tests

* save

* fix tests

* fix metadata

* revert package

* fix

* lint

* Support CEL expressions in the v1alpha1 wire format provided by the SDK (#4809)

* Add v2 Schedule Triggers to deploy (#4764)

* deploy working with using the default compute sa & running a GET request

* fix test cases

* formatter

* cleaning up code post merge

* add changelog entry

* Only start eventarc emulator when there is at least one emulatable backend (#4873)

* Only start eventarc emulator when there is at least one emulatable backend

* Enable automatic port finding

* Add changelog

* Format changelog

* Persist CF3 param values to disk, and read them if available during deploy (#4787)

* 11.7.0

* [firebase-release] Removed change log and reset repo after 11.7.0 release

* revert js sdk tests

* fix tests

* comments

* remove mocha

* remove deep eql

* revert shrinkwrap

* changelog

* lint

Co-authored-by: Victor Fan <vsfan@google.com>
Co-authored-by: Cole Rogers <colerogers@users.noreply.github.com>
Co-authored-by: huangjeff5 <64040981+huangjeff5@users.noreply.github.com>
Co-authored-by: Google Open Source Bot <firebase-oss-bot@google.com>
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

9 participants