Skip to content

Conversation

@CodeMonkeyLeet
Copy link
Contributor

Description

Fix existing failures in components integration tests

Issue reference

N/A

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

@CodeMonkeyLeet CodeMonkeyLeet requested review from a team as code owners September 28, 2021 16:49
@CodeMonkeyLeet
Copy link
Contributor Author

@artursouza, refactor of #1142 to unblock @halspang

@codecov
Copy link

codecov bot commented Sep 28, 2021

Codecov Report

Merging #1173 (08679ba) into master (905cbc0) will increase coverage by 0.68%.
The diff coverage is 71.22%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1173      +/-   ##
==========================================
+ Coverage   33.77%   34.45%   +0.68%     
==========================================
  Files         138      140       +2     
  Lines       11686    11890     +204     
==========================================
+ Hits         3947     4097     +150     
- Misses       7329     7375      +46     
- Partials      410      418       +8     
Impacted Files Coverage Δ
bindings/azure/blobstorage/blobstorage.go 14.61% <0.00%> (-0.21%) ⬇️
bindings/azure/eventgrid/eventgrid.go 3.84% <0.00%> (-0.10%) ⬇️
bindings/azure/eventhubs/eventhubs.go 16.66% <0.00%> (-0.66%) ⬇️
...indings/azure/servicebusqueues/servicebusqueues.go 14.66% <0.00%> (-0.41%) ⬇️
bindings/azure/storagequeues/storagequeues.go 37.61% <0.00%> (-1.81%) ⬇️
pubsub/azure/eventhubs/eventhubs.go 23.40% <0.00%> (-0.51%) ⬇️
pubsub/azure/servicebus/servicebus.go 30.19% <0.00%> (-0.30%) ⬇️
state/azure/tablestorage/tablestorage.go 11.86% <0.00%> (-0.21%) ⬇️
state/mongodb/mongodb.go 16.29% <0.00%> (-0.48%) ⬇️
state/alicloud/tablestore/tablestore.go 78.72% <78.72%> (ø)
... and 6 more

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 e41acc4...08679ba. Read the comment docs.

@artursouza artursouza merged commit 9a075cb into dapr:master Sep 29, 2021
Taction added a commit to Taction/components-contrib that referenced this pull request Oct 6, 2021
* master:
  fix 'durable' setting in RabbitMQ (dapr#1183)
  Support setting up Azure IoT Hub for integration testing with EventHubs components (dapr#1174)
  gcp bucket update  (dapr#1128)
  ok-to-test changes for components-contrib (dapr#1091)
  version stored procedure due to previous changes made (dapr#1176)
  Fix existing integration tests (dapr#1173)
  Add state alicloud.tablestore (dapr#1116)
  Add dapr user agent for Azure Components (dapr#1152)
  Support using setup-azure-conf-test.sh with existing Service Principal (dapr#1158)
  Add Event Hubs conformance tests to conformance.yml workflow (dapr#1164)
  restructure code and add comments per code review
  cover all concurrency + etag cases
  Fix EOF linebreaks
  Adds first-write conformance test
  update metadata key
  fix linter warnings
  TableStorage conformance tests pass
  TableStore config
  tablestorage conformance WIP
  ci: standard linter config (dapr#1102)

# Conflicts:
#	secretstores/hashicorp/vault/vault.go
@CodeMonkeyLeet CodeMonkeyLeet deleted the fix_integration_tests branch November 3, 2021 18:53
amimimor pushed a commit to amimimor/components-contrib that referenced this pull request Dec 9, 2021
* Fix servicebusqueues_integration_test.go compile errors

* Fix postgresql_integration_test.go deleteItemThatDoesNotExist expectations

* Fix mysql_integration_test.go deleteItemThatDoesNotExist expectations

* Fix mysql_integration_test.go for bindings

* Increase TTL in ServiceBusQueues tests for stability
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