Skip to content
This repository has been archived by the owner on Oct 11, 2024. It is now read-only.

[TT-6703] Added integration tests using Venom framework #520

Merged
merged 27 commits into from
Nov 30, 2022
Merged

[TT-6703] Added integration tests using Venom framework #520

merged 27 commits into from
Nov 30, 2022

Conversation

komalsukhani
Copy link
Collaborator

@komalsukhani komalsukhani commented Oct 13, 2022 •
edited
Loading

Description

Added integration tests using venom framework.

Related Issue

TT-6703

Motivation and Context

Test Coverage For This Change

Screenshots (if appropriate)

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • Make sure you are requesting to pull a topic/feature/bugfix branch (right side). If PRing from your fork, don't come from your master!
  • Make sure you are making a pull request against our master branch (left side). Also, it would be best if you started your change off our latest master.
  • Make sure you are updating CHANGELOG.md based on your changes.
  • My change requires a change to the documentation.
    • If you've changed APIs, describe what needs to be updated in the documentation.
  • I have updated the documentation accordingly.
  • If you've changed API models, please update CRDs.
    • make manifests
    • make helm
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • Check your code additions will not fail linting checks:
    • gofmt -s -w .
    • go vet ./...
    • golangci-lint run

@komalsukhani komalsukhani changed the title POC Venom framework [POC] Venom framework Oct 13, 2022
komalsukhani and others added 20 commits October 13, 2022 17:10
@komalsukhani
Fix indentation
7bae7a3
commiting
c16799a
commiting
e44cba9
@komalsukhani
Fix header transform tests
6d0a2b0
reverting-original-changes-in-crds.yaml-file
5973278
Merge branch 'poc/venom' of github.com:TykTechnologies/tyk-operator i…
d47ce4d 
…nto poc/venom

reverting back changes to its original form in crds.yaml file
@komalsukhani
Merge branch 'master' into poc/venom
190171d
@komalsukhani
Fix circuit breaker tests
5527193
@komalsukhani
Fix blacklist tests
03a459d
@komalsukhani
Fix request transform test
182f2a2
@komalsukhani
Remove tests for cache plugins
0523a05
@komalsukhani
Add samples in venom-tests folder
46d6b01
@komalsukhani
Remove readme file
f8e845c
@komalsukhani
Update make file
0acba17
@komalsukhani
Remove venom tests from test-all target for now
d412500
@komalsukhani
Try to run venom tests in GithubActions
d550b7e
@komalsukhani
Fix check
711d5a8
@komalsukhani
Do not run tests on PR
9800305
@komalsukhani
Merge branch 'master' into poc/venom
92421c7
@komalsukhani
Update changelog
6e36bcf
@komalsukhani komalsukhani changed the title [POC] Venom framework [TT-6703] Added integration tests using Venom framework Nov 2, 2022
@komalsukhani komalsukhani marked this pull request as ready for review November 2, 2022 12:21
@komalsukhani komalsukhani requested a review from a team as a code owner November 2, 2022 12:21
@komalsukhani komalsukhani requested review from buraksekili and removed request for a team November 2, 2022 12:21
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

buraksekili
buraksekili approved these changes Nov 15, 2022
View reviewed changes
Copy link
Collaborator

@buraksekili buraksekili left a comment

Choose a reason for hiding this comment

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

thank you!

@komalsukhani komalsukhani merged commit c4b8507 into master Nov 30, 2022
@komalsukhani komalsukhani deleted the poc/venom branch November 30, 2022 07:29
buger pushed a commit that referenced this pull request May 22, 2024
@komalsukhani
[TT-6703] Added integration tests using Venom framework (#520)
8ab2444 
* Add test suit for json valiation

* Rewrite test

* Add readme

* Add executor for applying manifests

* Fix indentation

* commiting

* commiting

* Fix header transform tests

* reverting-original-changes-in-crds.yaml-file

* Fix circuit breaker tests

* Fix blacklist tests

* Fix request transform test

* Remove tests for cache plugins

* Add samples in venom-tests folder

* Remove readme file

* Update make file

* Remove venom tests from test-all target for now

* Try to run venom tests in GithubActions

* Fix check

* Do not run tests on PR

* Update changelog

* Delete duplicate tests

* Delete duplicate sample

* Remove comments from sample file

Co-authored-by: Pranshu Singh <pranshusingh@Pranshus-MacBook-Pro.local>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@buraksekili buraksekili buraksekili approved these changes

@andrei-tyk andrei-tyk andrei-tyk left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@komalsukhani @buraksekili @andrei-tyk