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

[TT-7010] - Add unit tests #245

Merged
merged 6 commits into from
Nov 15, 2022
Merged

[TT-7010] - Add unit tests #245

merged 6 commits into from
Nov 15, 2022

Conversation

buraksekili
Copy link
Collaborator

@buraksekili buraksekili commented Nov 10, 2022

Description

This PR adds unit tests for tyk-headless gw component using helm-unittestframework

Related Issue

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.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

Signed-off-by: Burak Sekili <[email protected]>

Add workflow for unittests that run on each pr

Signed-off-by: Burak Sekili <[email protected]>

Update tests

Signed-off-by: Burak Sekili <[email protected]>
@buraksekili buraksekili marked this pull request as ready for review November 14, 2022 11:20
@buraksekili buraksekili requested a review from a team as a code owner November 14, 2022 11:20
@buraksekili buraksekili requested review from komalsukhani and removed request for a team November 14, 2022 11:20
@komalsukhani komalsukhani merged commit 544f590 into master Nov 15, 2022
@zalbiraw zalbiraw deleted the TT-7010/unit-tests branch November 28, 2022 22:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants