Skip to content

Conversation

@akkie
Copy link
Contributor

@akkie akkie commented Jun 24, 2021

Description

This PR implements E2E tests for the Zeebe binding.

Checklist

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

@akkie akkie marked this pull request as draft June 24, 2021 15:43
@akkie
Copy link
Contributor Author

akkie commented Jun 24, 2021

@pkedy Would be great if you could give some early feedback if this goes in the right direction.

@pkedy
Copy link
Member

pkedy commented Jun 25, 2021

@akkie You've got some linter errors breaking the build but this look like its structured well. You've got a README and docker compose for running it which is great.

@akkie
Copy link
Contributor Author

akkie commented Jun 28, 2021

@pkedy Do you have a suggestion how I can resolve the issue regarding the global variables.

`ZeebeVersion` is a global variable (gochecknoglobals)

I would like to share them between the tests.

@akkie
Copy link
Contributor Author

akkie commented Jun 29, 2021

I have created a getter for the environment variables to fix the global lint issue. I have also created a new entry in the makefile to run the tests from CI. I have not created a new workflow for E2E tests, because I'm not sure if each test should run separate or if it makes sense to create a shell script that runs all the E2E tests.

From my perspective the PR is finished and can be reviewed.

@akkie akkie marked this pull request as ready for review June 29, 2021 11:35
@akkie akkie mentioned this pull request Jun 29, 2021
pkedy
pkedy previously approved these changes Jun 30, 2021
Copy link
Member

@pkedy pkedy 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 doing this!

@pkedy pkedy self-assigned this Jun 30, 2021
@yaron2
Copy link
Member

yaron2 commented Jul 8, 2021

@akkie please resolve conflicts and we can merge.

@akkie akkie requested review from a team as code owners July 9, 2021 06:08
@akkie
Copy link
Contributor Author

akkie commented Jul 9, 2021

@yaron2 Done

@codecov
Copy link

codecov bot commented Jul 12, 2021

Codecov Report

Merging #973 (e987cb1) into master (01147e5) will increase coverage by 3.72%.
The diff coverage is 37.74%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #973      +/-   ##
==========================================
+ Coverage   30.59%   34.32%   +3.72%     
==========================================
  Files          85      132      +47     
  Lines        7148    10550    +3402     
==========================================
+ Hits         2187     3621    +1434     
- Misses       4724     6552    +1828     
- Partials      237      377     +140     
Impacted Files Coverage Δ
bindings/alicloud/oss/oss.go 11.11% <ø> (ø)
bindings/alicloud/rocketmq/rocketmq.go 0.00% <0.00%> (ø)
bindings/apns/apns.go 88.00% <ø> (ø)
bindings/aws/dynamodb/dynamodb.go 10.52% <ø> (ø)
bindings/aws/kinesis/kinesis.go 2.61% <ø> (ø)
bindings/aws/s3/s3.go 10.81% <ø> (ø)
bindings/aws/sns/sns.go 10.52% <ø> (ø)
bindings/aws/sqs/sqs.go 6.55% <0.00%> (ø)
bindings/azure/blobstorage/blobstorage.go 7.54% <0.00%> (-0.30%) ⬇️
bindings/azure/cosmosdb/cosmosdb.go 25.71% <ø> (ø)
... and 169 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 c3cf138...e987cb1. Read the comment docs.

Copy link
Member

@yaron2 yaron2 left a comment

Choose a reason for hiding this comment

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

lgtm

@yaron2 yaron2 assigned akkie and unassigned pkedy Jul 12, 2021
@yaron2 yaron2 added this to the v1.3 milestone Jul 12, 2021
@yaron2 yaron2 merged commit cb9a510 into dapr:master Jul 12, 2021
@akkie akkie deleted the zeebe_e2e_tests branch July 13, 2021 07:05
Taction added a commit to Taction/components-contrib that referenced this pull request Jul 15, 2021
* master: (23 commits)
  Adds support for TTL in Redis State Store (dapr#990)
  Adds support for TTL in Memcached State Store (PR Fixup) (dapr#1011)
  Adds support for TTL in Cassandra State Store (dapr#996)
  Return blob metadata and add list operation (dapr#999)
  Added new output binding for Cosmos Graph DB - Gremlin (dapr#885)
  Adds support for TTL in CosmosDB State Store (dapr#991)
  Delete unused code and update test (dapr#924)
  Return SystemProperties on events from EventHubs bindings and pubsub (dapr#1009)
  Implement E2E tests for zeebe (dapr#973)
  rebase (dapr#982)
  Update CODEOWNERS (dapr#987)
  ci: add test skip case (dapr#922)
  fix: Mysql should support more data types. dapr#923 (dapr#926)
  fix: Dapr runtime panic when handle Pub/Sub (dapr#3281) (dapr#967)
  switch to golang-jwt (dapr#993)
  unmarshal getBlobRetryCount as int (dapr#919)
  secretstores: support more format of azure key vault (dapr#944)
  Optimize vault secret component error output (dapr#909)
  Convert AZURE_KEYVAULT in conformance.yml a GitHub secret (dapr#1002)
  Use AzureKeyVaultName param for Azure Keyvault conformance test (dapr#975)
  ...

# Conflicts:
#	secretstores/hashicorp/vault/vault.go
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