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

[ISSUE #4990] unit tests for HttpConvertsUtils.java #5047

Open
wants to merge 4,618 commits into
base: master
Choose a base branch
from

Conversation

Abhichede
Copy link
Contributor

Fixes #4990

Motivation
Increase test coverage

Modifications
Added new tests for HttpConvertsUtils.java located at [eventmesh-runtime/src/main/java/org/apache/eventmesh/common/util]

Documentation
Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)
If a feature is not applicable for documentation, explain why?
If a feature is not documented yet in this PR, please create a followup issue for adding the documentation

piyush280599 and others added 30 commits November 12, 2023 17:50
…ache#3883)

* Update HttpProtocolAdaptor.java

1. 
imported `java.util.Collections`
returns `Collections.emptyList()` at line 81.

2. Used `Objects.requireNonNull()` at line 103

* Update HttpProtocolAdaptor.java
* Add unit test for RedisSourceConnector.

* Cleanup redisson.
* Feat: Add example for spring.

* fix: Remove exclude configuration.

* fix: fix log
…et, RocketMQ consumption is slow (apache#4544)

* Provide MQ-storage-based retry strategy in http protocol.

* Remove unused method.

* fix

* fix

* Add retry, rocketmq-retry module.

* Do some optimization.

* Use interface instead of HttpHandleMsgContext.

* optimize

* fix

* Remove unused methods.

* fix log

* Remove retry-rocketmq dependency.

* rollback dependency changes.

* Use plugin to load spi.

* Remove unused methods.

* fix: remove storage dependency

* fix: log

* fix: change 'standalone' to 'default'

* fix: codereview
* Update build file to push docker image only on release

Fix apache#913

* Update build.yaml

---------

Co-authored-by: mike_xwm <mike_xwm@126.com>
* Add dingding sink connector.

* fix: Remove unused config.

* fix: Remove unused config.

* fix: review

* Add example config.

* fix: review

* fix: continue when record data is null.
* [ISSUE apache#4420] Add Feishu Sink connector

* [ISSUE apache#4420] Add Feishu Sink connector

* [ISSUE apache#4420] Add Feishu Sink connector

* [ISSUE apache#4420] Add Feishu Sink connector

* [ISSUE apache#4420] Add Feishu Sink connector,code style

* [ISSUE apache#4420] Add Feishu Sink connector,check style

* [ISSUE apache#4420] Add Feishu Sink connector,mock test

* [ISSUE apache#4420] Add Feishu Sink connector,mock test

* [ISSUE apache#4420] Add Feishu Sink connector,mock test
* Add WeCom sink connector.

* Refactor: use robot webhook instead of app notification.

* fix: Add test resources.

* fix: code review

* fix code review
* Add slack sink connector.

* fix code review
apache#4569)

* feat() : Support adding extensions from ConnectRcord to CloudEvent

* feat() : Determine whether there are primitive types

* fix() : Resolve ambiguity in method names in AbstractTCPServer

* feat : more put and get method in KeyValue

* feat : simplify Number type put method
…pache#4561)

* Added test for OpenFunction Source Connector

* Removed Redundant @SPY annnotation

* WIP resolved checkstyle errors
* [ISSUE apache#4564]
add url validation to avoid SSRF

* [ISSUE apache#4564] correct log level
* remove duplicated moudle

* refacotr moudle dingding to dingtalk

---------

Co-authored-by: easoncchen(陈广胜) <easoncchen@webank.com>
* fix : http demo rely on tcp runtime

* fix : grpc demo rely on tcp runtime
…che#4578)

* update ci.yml

* add filters

* add filters

* add filters

* apply filters under http processor

* update filterEngine

* [ISSUE apache#4577] Implement FilterEngine for EventMesh Filters

* refactor ci.yml

* fix ci check error

* fix ci check error

* fix ci check error

* fix ci check error

* remove duplicate method in PatternBuilder
…s. (apache#4582)

* Connector extension supports spring environment variables.

* fix: cr
* add wechat connector module and dependency

* add wechat connector

* add unit test

* fix check style error and downgrade okhttp version to 3.14.9

* add stop method after test end

* fix check style
…rts ConnectRecord to CloudEvent throw NPE (apache#4597)

* [ISSUE apache#4596]Fix SourceWorker#convertRecordToEvent method converts ConnectRecord  to CloudEvent throw NPE

* fix CloudEventUtil#convertRecordToEvent method converts ConnectRecord to CloudEvent throw NPE
* operator init

* delete MD

* update

* add spec status controller

* update eventmesh-operator.md

* connectors operator

* update

* update controller

* update

* fix bug and test controller

* update

* add eventmesh_v1_runtime.yaml eventmesh_v1_connectors_rocketmq.yaml eventmesh_v1_runtime_cluster.yaml

* delete unnecessary files

* add license and add README.md

* update

* update

* update go.sum

* update

* update

* update

* update README.md and merge command

* update README.md and merge command
apache#4620)

* [ISSUE apache#4619]Fix Grpc request reply can not revice reply message

* fix codestyle

* fomat codetyle
…astSubClient] (apache#4617)

* Issue Fix: apache#4088

* Issue Fix: apache#4088

Modified changes as suggested

* formatted

* Issue Fix: apache#4088

fixed codestyle
…rmer (apache#4622)

* [ISSUE apache#4621] Implement TransformerEngine for EventMesh Transformer

* fix ci check error
dependabot bot and others added 20 commits July 15, 2024 22:39
* Bump org.apache.kafka:kafka-clients from 3.6.2 to 3.7.1

Bumps org.apache.kafka:kafka-clients from 3.6.2 to 3.7.1.

---
updated-dependencies:
- dependency-name: org.apache.kafka:kafka-clients
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

* feat: increase the upgrade interval

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Pil0tXia <xiatian@apache.org>
---
updated-dependencies:
- dependency-name: com.alibaba.fastjson2:fastjson2
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
apache#5036)

Bumps [io.spring.gradle:dependency-management-plugin](https://github.com/spring-gradle-plugins/dependency-management-plugin) from 1.1.5 to 1.1.6.
- [Release notes](https://github.com/spring-gradle-plugins/dependency-management-plugin/releases)
- [Commits](spring-gradle-plugins/dependency-management-plugin@v1.1.5...v1.1.6)

---
updated-dependencies:
- dependency-name: io.spring.gradle:dependency-management-plugin
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps `jacksonVersion` from 2.17.1 to 2.17.2.

Updates `com.fasterxml.jackson.core:jackson-databind` from 2.17.1 to 2.17.2
- [Commits](https://github.com/FasterXML/jackson/commits)

Updates `com.fasterxml.jackson.core:jackson-core` from 2.17.1 to 2.17.2
- [Commits](FasterXML/jackson-core@jackson-core-2.17.1...jackson-core-2.17.2)

Updates `com.fasterxml.jackson.core:jackson-annotations` from 2.17.1 to 2.17.2
- [Commits](https://github.com/FasterXML/jackson/commits)

Updates `com.fasterxml.jackson.dataformat:jackson-dataformat-yaml` from 2.17.1 to 2.17.2
- [Commits](FasterXML/jackson-dataformats-text@jackson-dataformats-text-2.17.1...jackson-dataformats-text-2.17.2)

Updates `com.fasterxml.jackson.datatype:jackson-datatype-jsr310` from 2.17.1 to 2.17.2

---
updated-dependencies:
- dependency-name: com.fasterxml.jackson.core:jackson-databind
  dependency-type: direct:production
  update-type: version-update:semver-patch
- dependency-name: com.fasterxml.jackson.core:jackson-core
  dependency-type: direct:production
  update-type: version-update:semver-patch
- dependency-name: com.fasterxml.jackson.core:jackson-annotations
  dependency-type: direct:production
  update-type: version-update:semver-patch
- dependency-name: com.fasterxml.jackson.dataformat:jackson-dataformat-yaml
  dependency-type: direct:production
  update-type: version-update:semver-patch
- dependency-name: com.fasterxml.jackson.datatype:jackson-datatype-jsr310
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [com.baomidou:mybatis-plus](https://github.com/baomidou/mybatis-plus) from 3.5.6 to 3.5.7.
- [Release notes](https://github.com/baomidou/mybatis-plus/releases)
- [Changelog](https://github.com/baomidou/mybatis-plus/blob/3.0/CHANGELOG.md)
- [Commits](baomidou/mybatis-plus@v3.5.6...v3.5.7)

---
updated-dependencies:
- dependency-name: com.baomidou:mybatis-plus
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [org.mybatis.spring.boot:mybatis-spring-boot-starter](https://github.com/mybatis/spring-boot-starter) from 2.3.1 to 2.3.2.
- [Release notes](https://github.com/mybatis/spring-boot-starter/releases)
- [Commits](mybatis/spring-boot-starter@mybatis-spring-boot-2.3.1...mybatis-spring-boot-2.3.2)

---
updated-dependencies:
- dependency-name: org.mybatis.spring.boot:mybatis-spring-boot-starter
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [org.redisson:redisson](https://github.com/redisson/redisson) from 3.31.0 to 3.32.0.
- [Release notes](https://github.com/redisson/redisson/releases)
- [Changelog](https://github.com/redisson/redisson/blob/master/CHANGELOG.md)
- [Commits](redisson/redisson@redisson-parent-3.31.0...redisson-3.32.0)

---
updated-dependencies:
- dependency-name: org.redisson:redisson
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [org.junit.jupiter:junit-jupiter](https://github.com/junit-team/junit5) from 5.10.2 to 5.10.3.
- [Release notes](https://github.com/junit-team/junit5/releases)
- [Commits](junit-team/junit5@r5.10.2...r5.10.3)

---
updated-dependencies:
- dependency-name: org.junit.jupiter:junit-jupiter
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [org.projectlombok:lombok](https://github.com/projectlombok/lombok) from 1.18.32 to 1.18.34.
- [Changelog](https://github.com/projectlombok/lombok/blob/master/doc/changelog.markdown)
- [Commits](projectlombok/lombok@v1.18.32...v1.18.34)

---
updated-dependencies:
- dependency-name: org.projectlombok:lombok
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [com.alibaba.nacos:nacos-client](https://github.com/alibaba/nacos) from 2.3.2 to 2.3.3.
- [Release notes](https://github.com/alibaba/nacos/releases)
- [Changelog](https://github.com/alibaba/nacos/blob/develop/CHANGELOG.md)
- [Commits](https://github.com/alibaba/nacos/commits)

---
updated-dependencies:
- dependency-name: com.alibaba.nacos:nacos-client
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [org.assertj:assertj-core](https://github.com/assertj/assertj) from 3.26.0 to 3.26.3.
- [Release notes](https://github.com/assertj/assertj/releases)
- [Commits](assertj/assertj@assertj-build-3.26.0...assertj-build-3.26.3)

---
updated-dependencies:
- dependency-name: org.assertj:assertj-core
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [docker/build-push-action](https://github.com/docker/build-push-action) from 5 to 6.
- [Release notes](https://github.com/docker/build-push-action/releases)
- [Commits](docker/build-push-action@v5...v6)

---
updated-dependencies:
- dependency-name: docker/build-push-action
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* test: adds unit tests for HttpRequestUtil.java

* fix: fixes checkstyle warnings
* use conf instead of resource

* prepare for full and check

* more and more

* mysql type for read

* mysql type for read

* close to finish full read and begin full write

* do full write

* prepare for full write

* close to finish full read and begin full write

* close to finish full read and begin full write

* close to finish full read and begin full write

* fix commit execute

* fix checkstyle and license

* fix checkstyle and license

* fix checkstyle and license
…e#5041)

* [ISSUE apache#5040] Support gtid mode for sync data with mysql

* fix conflicts with master

* fix checkstyle error
…e#5043)

* feat: Add a new deniedLicense

* feat: change implementation namespace to compileOnly
…db gtid mode (apache#5045)

* [ISSUE apache#5040] Support gtid mode for sync data with mysql

* fix conflicts with master

* fix checkstyle error

* [ISSUE apache#5044] Data synchronization strong verification in mariadb gtid mode

* fix checkstyle error
@@ -0,0 +1,18 @@
package org.apache.eventmesh.common.stubs;
Copy link
Member

Choose a reason for hiding this comment

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

no apache license headers.

@harshithasudhakar harshithasudhakar added the waiting for contributor PR is awaiting the contributor's response to the review for further evaluation. label Sep 9, 2024
@xwm1992
Copy link
Contributor

xwm1992 commented Dec 12, 2024

@Abhichede please resolve the conflicts first

@github-actions github-actions bot removed the Stale label Dec 12, 2024
Copy link
Contributor

It has been 60 days since the last activity on this pull request. I am reaching out here to gently remind you that the Apache EventMesh community values every pull request, and please feel free to get in touch with the reviewers at any time. They are available to assist you in advancing the progress of your pull request and offering the latest feedback.

If you encounter any challenges during development, seeking support within the community is encouraged. We sincerely appreciate your contributions to Apache EventMesh.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
waiting for contributor PR is awaiting the contributor's response to the review for further evaluation.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Unit Test] Unit test for HttpConvertsUtils.java