Refactor: Eliminate Global Variable to Enhance Testability #4685

andoriyaprashant · 2024-06-03T18:15:21Z

Fixes #4217

Proposed changes

This pull request refactors the codebase by removing a global variable to improve testability.

Types of changes

What types of changes does your code introduce to Litmus? Put an x in the boxes that apply

New feature (non-breaking change which adds functionality)
Bugfix (non-breaking change which fixes an issue)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
Documentation Update (if none of the other choices applies)

Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

I have read the CONTRIBUTING doc
I have signed the commit for DCO to be passed.
Lint and unit tests pass locally with my changes
I have added tests that prove my fix is effective or that my feature works (if appropriate)
I have added necessary documentation (if appropriate)

andoriyaprashant · 2024-06-03T18:15:56Z

Hello @namkyu1999 Please review my pull request

namkyu1999 · 2024-06-04T00:18:40Z

can you check DCO?

andoriyaprashant · 2024-06-04T04:18:11Z

@namkyu1999 Sir, as you instructed, I've tried to complete the checks. Could you please verify them? If there are any changes needed in the code, please let me know, and I'll try to fix them

namkyu1999 · 2024-06-04T05:09:09Z

Hi @andoriyaprashant, some checks are failing now (build-pipeline, CIFuzz). can you check them?

andoriyaprashant · 2024-06-04T06:37:37Z

Hi @andoriyaprashant, some checks are failing now (build-pipeline, CIFuzz). can you check them?

i am working on those 👍🏻

Jonsy13 · 2024-06-07T06:43:37Z

chaoscenter/graphql/server/pkg/database/mongodb/operations.go

@@ -32,8 +32,8 @@ type MongoOperations struct {
 }

 var (


Can we remove this var block itself as well.

Saranya-jena · 2024-06-07T06:49:12Z

chaoscenter/graphql/server/pkg/database/mongodb/config/operations.go

@@ -10,20 +10,20 @@ import (
 )

 // CreateConfig creates a new server config with unique key
-func CreateConfig(ctx context.Context, config *ServerConfig) error {
-	err := mongodb.Operator.Create(ctx, mongodb.ServerConfigCollection, config)
+func CreateConfig(ctx context.Context, config *ServerConfig, mongodbOperator mongodb.MongoOperator) error {


Is it possible to create an interface and receivers here similar to what we have in other places?

andoriyaprashant · 2024-06-08T06:58:42Z

Hello @Saranya-jena @Jonsy13 @namkyu1999
I made some changes to the files as @Jonsy13 suggested removing the var() block, so I removed it completely. Also, as per @Saranya-jena suggestion, I made some changes to the config file. Please review them, and if there are any errors or further changes needed, please let me know. I'll do my best to fix them.
Thank you

Saranya-jena · 2024-08-01T09:34:18Z

The checks are failing can you fix those?

Saranya-jena · 2024-08-23T06:24:38Z

@andoriyaprashant are you still working on this PR?

andoriyaprashant · 2024-08-23T06:37:03Z

@andoriyaprashant are you still working on this PR?

Sorry @Saranya-jena I forgot to check the pull request checks. I'll fix them as soon as possible 👍🏻

Saranya-jena · 2024-09-06T06:43:54Z

@andoriyaprashant the checks are still failing, PTAL.

andoriyaprashant · 2024-09-07T03:09:55Z

@andoriyaprashant the checks are still failing, PTAL.

Ma'am @Saranya-jena , can you please help me or guide me in fixing the CIFuzz workflow error? I tried, but I am not able to understand it

updated-dependencies: - dependency-name: github.com/99designs/gqlgen 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> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

Bumps [github.com/vektah/gqlparser/v2](https://github.com/vektah/gqlparser) from 2.5.11 to 2.5.12. - [Release notes](https://github.com/vektah/gqlparser/releases) - [Commits](vektah/gqlparser@v2.5.11...v2.5.12) --- updated-dependencies: - dependency-name: github.com/vektah/gqlparser/v2 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> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

Signed-off-by: Saranya-jena <saranya.jena@harness.io> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

Signed-off-by: Shubham Chaudhary <shubham.chaudhary@harness.io> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

* upgrade go version in all the modules Signed-off-by: Saranya-jena <saranya.jena@harness.io> * upgrade go version in dockerfiles Signed-off-by: Saranya-jena <saranya.jena@harness.io> * upgrade go version in github actions Signed-off-by: Saranya-jena <saranya.jena@harness.io> * updated go sum Signed-off-by: Saranya-jena <saranya.jena@harness.io> * updated subscriber dockerfile Signed-off-by: Saranya-jena <saranya.jena@harness.io> --------- Signed-off-by: Saranya-jena <saranya.jena@harness.io> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

* Added env support to enable/disable gql introspection Signed-off-by: Saranya-jena <saranya.jena@harness.io> * resolved comments Signed-off-by: Saranya-jena <saranya.jena@harness.io> * fixed imports Signed-off-by: Saranya-jena <saranya.jena@harness.io> * fixed build issues Signed-off-by: Saranya-jena <saranya.jena@harness.io> --------- Signed-off-by: Saranya-jena <saranya.jena@harness.io> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

Signed-off-by: Saranya-jena <saranya.jena@harness.io> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

Signed-off-by: Jongwoo Han <jongwooo.han@gmail.com> Co-authored-by: Namkyu Park <53862866+namkyu1999@users.noreply.github.com> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

Signed-off-by: Jongwoo Han <jongwooo.han@gmail.com> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

…os#4830) Signed-off-by: Shubham Chaudhary <shubham.chaudhary@harness.io> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

…os#4666) * fix: Invalid version details in backend. Signed-off-by: JanhaviAlekar <janhavialekar@gmail.com> * Checking CIVersion at start Signed-off-by: JanhaviAlekar <janhavialekar@gmail.com> --------- Signed-off-by: JanhaviAlekar <janhavialekar@gmail.com> Co-authored-by: Vedant Shrotria <vedant.shrotria@harness.io> Co-authored-by: Amit Kumar Das <amit.das@harness.io> Co-authored-by: Namkyu Park <53862866+namkyu1999@users.noreply.github.com> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com> Co-authored-by: Namkyu Park <53862866+namkyu1999@users.noreply.github.com> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

Bumps [github.com/99designs/gqlgen](https://github.com/99designs/gqlgen) from 0.17.47 to 0.17.49. - [Release notes](https://github.com/99designs/gqlgen/releases) - [Changelog](https://github.com/99designs/gqlgen/blob/master/CHANGELOG.md) - [Commits](99designs/gqlgen@v0.17.47...v0.17.49) --- updated-dependencies: - dependency-name: github.com/99designs/gqlgen 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> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

…uschaos#4692) Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com> Co-authored-by: Saranya Jena <saranya.jena@harness.io> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

…haos#4762) Signed-off-by: Denish Tomar <denishtomar3436@gmail.com> Co-authored-by: Saranya Jena <saranya.jena@harness.io> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

Signed-off-by: JanhaviAlekar <janhavialekar@gmail.com> Co-authored-by: Namkyu Park <53862866+namkyu1999@users.noreply.github.com> Co-authored-by: Sahil <sahil.kumar@harness.io> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

…itmuschaos#4813) Bumps [golang.org/x/crypto](https://github.com/golang/crypto) from 0.25.0 to 0.26.0. - [Commits](golang/crypto@v0.25.0...v0.26.0) --- updated-dependencies: - dependency-name: golang.org/x/crypto 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> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

Bumps [go.mongodb.org/mongo-driver](https://github.com/mongodb/mongo-go-driver) from 1.15.0 to 1.16.1. - [Release notes](https://github.com/mongodb/mongo-go-driver/releases) - [Commits](mongodb/mongo-go-driver@v1.15.0...v1.16.1) --- updated-dependencies: - dependency-name: go.mongodb.org/mongo-driver 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> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

Signed-off-by: Baalekshan <baalekshan@gmail.com> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

Signed-off-by: Suyeon Jung <suyeonjungdev@gmail.com> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

Bumps [go.mongodb.org/mongo-driver](https://github.com/mongodb/mongo-go-driver) from 1.15.1 to 1.16.1. - [Release notes](https://github.com/mongodb/mongo-go-driver/releases) - [Commits](mongodb/mongo-go-driver@v1.15.1...v1.16.1) --- updated-dependencies: - dependency-name: go.mongodb.org/mongo-driver 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> Co-authored-by: Saranya Jena <saranya.jena@harness.io> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

Signed-off-by: hursit <hursit.topal@enuygun.com> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

Signed-off-by: JanhaviAlekar <janhavialekar@gmail.com> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

Signed-off-by: JanhaviAlekar <janhavialekar@gmail.com> Co-authored-by: Namkyu Park <53862866+namkyu1999@users.noreply.github.com> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

…os#4779) * Adding environment filter in Infrastructure selection modal Signed-off-by: JanhaviAlekar <janhavialekar@gmail.com> * setting initialAllInfrastructureLength as 0 initially Signed-off-by: JanhaviAlekar <janhavialekar@gmail.com> * making InitialAllInfrastructureLength consistent Signed-off-by: JanhaviAlekar <janhavialekar@gmail.com> * Scroll env list and Pagination(infra) Signed-off-by: JanhaviAlekar <janhavialekar@gmail.com> * added preSelectedEnvironmentID Signed-off-by: JanhaviAlekar <janhavialekar@gmail.com> * Minor changes in code Signed-off-by: JanhaviAlekar <janhavialekar@gmail.com> * Refactored code Signed-off-by: JanhaviAlekar <janhavialekar@gmail.com> * Changing height of infralist section Signed-off-by: JanhaviAlekar <janhavialekar@gmail.com> * Minor improvements Signed-off-by: JanhaviAlekar <janhavialekar@gmail.com> --------- Signed-off-by: JanhaviAlekar <janhavialekar@gmail.com> Co-authored-by: Sahil <sahil.kumar@harness.io> Co-authored-by: Saranya Jena <saranya.jena@harness.io> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

Signed-off-by: 잉퓨 <dayeon620@kakao.com> Co-authored-by: Namkyu Park <53862866+namkyu1999@users.noreply.github.com> Co-authored-by: Saranya Jena <saranya.jena@harness.io> Co-authored-by: Vedant Shrotria <vedant.shrotria@harness.io> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

litmuschaos#4866) Signed-off-by: DongYoung Kim <kwx4957@gmail.com> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

Bumps [github.com/tidwall/gjson](https://github.com/tidwall/gjson) from 1.17.1 to 1.17.3. - [Commits](tidwall/gjson@v1.17.1...v1.17.3) --- updated-dependencies: - dependency-name: github.com/tidwall/gjson 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> Co-authored-by: Saranya Jena <saranya.jena@harness.io> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

* test: add fuzz test to GetChartsPath function in handler Signed-off-by: Soyeon Park <sypark9646@gmail.com> * test: add fuzz test to FuzzReadExperimentFile function in handler * Removed the ./types.go example in unit test handler_test.go/TestReadExperimentFile because it returns a file does not exist error, not the file is not a yaml error that the test is intended to return. Signed-off-by: Soyeon Park <sypark9646@gmail.com> * * test: Add the FuzzReadExperimentYAMLFile test in the handler_fuzz_test.go file Signed-off-by: Soyeon Park <sypark9646@gmail.com> * test: add fuzz test to FuzzIsFileExisting function in handler Signed-off-by: Soyeon Park <sypark9646@gmail.com> * test: add fuzz test to FuzzGetExperimentData, FuzzUnzipRemoteHub function in handler Signed-off-by: Soyeon Park <sypark9646@gmail.com> * refactor: remove unused imported library Signed-off-by: Soyeon Park <sypark9646@gmail.com> * fix: check yaml: control characters are not allowed Signed-off-by: Soyeon Park <sypark9646@gmail.com> * refactor: save goimport order Signed-off-by: Soyeon Park <sypark9646@gmail.com> --------- Signed-off-by: Soyeon Park <sypark9646@gmail.com> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

* add handler_fuzz_test file Signed-off-by: weeeeestern <liz0824@naver.com> * add service_fuzz_test file Signed-off-by: weeeeestern <liz0824@naver.com> * test: add fuzz test Signed-off-by: weeeeestern <liz0824@naver.com> * test: add fuzz_test Signed-off-by: weeeeestern <liz0824@naver.com> * test: modify fuzz-test Signed-off-by: weeeeestern <liz0824@naver.com> * test: modify fuzz-test Signed-off-by: weeeeestern <liz0824@naver.com> --------- Signed-off-by: weeeeestern <liz0824@naver.com> Co-authored-by: Namkyu Park <53862866+namkyu1999@users.noreply.github.com> Co-authored-by: Saranya Jena <saranya.jena@harness.io> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

…only a single namespace (litmuschaos#4680) * Create a seperate function to get Namespace in subscriber Signed-off-by: Calvin Audier <calvinaudier@gmail.com> * Update graphql server to include KubeNamespace call (model not generated) Signed-off-by: Calvin Audier <calvinaudier@gmail.com> * Update graphqls to include KubeNamespace model Update objectmodel and subscriber to incldue KubeNamespace Signed-off-by: Calvin Audier <calvinaudier@gmail.com> * Fix issue in graphqls and adding KubeNamespace type. Regenerating model with gqlgen. Signed-off-by: Calvin Audier <calvinaudier@gmail.com> * Update model to includude missing field in KubeNamespaceData Rename getKubeNamespace function that didn't match graphqls operation in Subscription Add missing function for the graphql server to retrieve list of namespace Signed-off-by: Calvin Audier <calvinaudier@gmail.com> * Add remaining function in k8s pkg and requests to return list of namespace Signed-off-by: Calvin Audier <calvinaudier@gmail.com> * Update graphql/subscriber to transform KubeObject to not be a array since subscriber will only return one element. Signed-off-by: Calvin Audier <calvinaudier@gmail.com> * Update web server to seperate call for KubeObject and KubeNamespace. Signed-off-by: Calvin Audier <calvinaudier@gmail.com> * Fix import with goimports Signed-off-by: Calvin Audier <calvinaudier@gmail.com> * Reverting upgrade of webpack-dev-server so it's compatible with github workflow Signed-off-by: Calvin Audier <calvinaudier@gmail.com> * Run gofmt with correct version. Signed-off-by: Calvin Audier <calvinaudier@gmail.com> * Bumping ubi-minimal:8.8 to 8.10 to fix some HIGH CVE severity detected by trivy. Signed-off-by: Calvin Audier <calvinaudier@gmail.com> * Updating chaos_infrastructure mock to include KubeNamespace. Fix a comment Signed-off-by: Calvin Audier <calvinaudier@gmail.com> * Fix issue from Codacy Removing some unused code Signed-off-by: Calvin Audier <calvinaudier@gmail.com> * Fix codacy issue: - Add some trailing comma - Trying transform subscription function to resolve "non-arrow functions are forbidden" - Add null check Signed-off-by: Calvin Audier <calvinaudier@gmail.com> * Continue fix Codacy issue - Adding trailing comma - Re-ordering by alphabetic order some parameters Signed-off-by: Calvin Audier <calvinaudier@gmail.com> --------- Signed-off-by: Calvin Audier <calvinaudier@gmail.com> Signed-off-by: Calvinaud <calvinaudier@gmail.com> Co-authored-by: Namkyu Park <53862866+namkyu1999@users.noreply.github.com> Co-authored-by: Saranya Jena <saranya.jena@harness.io> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

Signed-off-by: Prashant Andoriya <121665385+andoriyaprashant@users.noreply.github.com>

S-ayanide · 2024-11-22T07:25:28Z

Hi @andoriyaprashant , can you please rebase with main? The current commit history is incorrect which is why it's pulling a lot of extra commits too.

Signed-off-by: Prashant Andoriya <121665385+andoriyaprashant@users.noreply.github.com>

andoriyaprashant force-pushed the branch1 branch from aff6467 to c471726 Compare June 4, 2024 02:14

namkyu1999 assigned andoriyaprashant Jun 4, 2024

andoriyaprashant force-pushed the branch1 branch from bac85b7 to f50208d Compare June 6, 2024 10:24

Jonsy13 reviewed Jun 7, 2024

View reviewed changes

Saranya-jena reviewed Jun 7, 2024

View reviewed changes

andoriyaprashant force-pushed the branch1 branch from f808557 to 37c96c5 Compare June 8, 2024 06:39

Saranya-jena approved these changes Aug 1, 2024

View reviewed changes

andoriyaprashant force-pushed the branch1 branch 2 times, most recently from 5bc31c5 to 30cea9a Compare September 7, 2024 08:45

dependabot bot and others added 10 commits September 7, 2024 14:16

Updating email id of Oum kale (litmuschaos#4658)

08a877f

Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

fixed nil pointer issue with default chaoshubs (litmuschaos#4683)

1f1d36d

Signed-off-by: Saranya-jena <saranya.jena@harness.io> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

fix(logs): fix the rank warning in logs (litmuschaos#4682)

a13ed01

Signed-off-by: Shubham Chaudhary <shubham.chaudhary@harness.io> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

Refactor: Eliminate Global Variable to Enhance Testability

f10da7b

Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

checks

b251240

Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

backend-checks

1ea0aa0

Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

Saranya-jena and others added 26 commits September 7, 2024 14:16

Added installation manifests for 3.10.0 (litmuschaos#4827)

6042dc3

Signed-off-by: Saranya-jena <saranya.jena@harness.io> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

Fix image links in README (litmuschaos#4811)

f832f95

Signed-off-by: Jongwoo Han <jongwooo.han@gmail.com> Co-authored-by: Namkyu Park <53862866+namkyu1999@users.noreply.github.com> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

Rename env to EC2_INSTANCE_TAG (litmuschaos#4815)

6c33724

Signed-off-by: Jongwoo Han <jongwooo.han@gmail.com> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

chore(3.10.0): Adding the installation manifest for 3.10.0 (litmuscha…

1bc1285

…os#4830) Signed-off-by: Shubham Chaudhary <shubham.chaudhary@harness.io> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

e2e.yml fixed (litmuschaos#4696)

03bf77d

Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com> Co-authored-by: Namkyu Park <53862866+namkyu1999@users.noreply.github.com> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

Fix Horizontal Scroll Bar in "Enable Chaos Environments" Pop-up (litm…

96130d5

…uschaos#4692) Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com> Co-authored-by: Saranya Jena <saranya.jena@harness.io> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

Improved documentation with spelling and grammar corrections (litmusc…

28d75cc

…haos#4762) Signed-off-by: Denish Tomar <denishtomar3436@gmail.com> Co-authored-by: Saranya Jena <saranya.jena@harness.io> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

Allow older infras to connect with chaos centre (litmuschaos#4823)

f791ded

Signed-off-by: Baalekshan <baalekshan@gmail.com> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

fix: Correct swagger.json file versions (litmuschaos#4838)

26b0157

Signed-off-by: Suyeon Jung <suyeonjungdev@gmail.com> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

Create wingie-enuygun.md (litmuschaos#4861)

6ff3eb5

Signed-off-by: hursit <hursit.topal@enuygun.com> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

Changed schema chaoshub (litmuschaos#4842)

0753bb6

Signed-off-by: JanhaviAlekar <janhavialekar@gmail.com> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

added '#' as valid Password Character (litmuschaos#4833)

eab3f3b

Signed-off-by: JanhaviAlekar <janhavialekar@gmail.com> Co-authored-by: Namkyu Park <53862866+namkyu1999@users.noreply.github.com> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

Comment response invalidCredentials status has changed from 400 to 401 (

4e5bc4f

litmuschaos#4866) Signed-off-by: DongYoung Kim <kwx4957@gmail.com> Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>

Merge branch 'master' into branch1

c8b7184

Signed-off-by: Prashant Andoriya <121665385+andoriyaprashant@users.noreply.github.com>

namkyu1999 added the need-approvers-review Reminder label to the codeowners/maintainers for stale PRs that are more that 2 weeks old label Nov 15, 2024

Merge branch 'master' into branch1

ab65c75

Merge branch 'master' into branch1

65b33ff

Signed-off-by: Prashant Andoriya <121665385+andoriyaprashant@users.noreply.github.com>

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Refactor: Eliminate Global Variable to Enhance Testability #4685

Refactor: Eliminate Global Variable to Enhance Testability #4685

andoriyaprashant commented Jun 3, 2024

andoriyaprashant commented Jun 3, 2024

namkyu1999 commented Jun 4, 2024

andoriyaprashant commented Jun 4, 2024

namkyu1999 commented Jun 4, 2024

andoriyaprashant commented Jun 4, 2024

Jonsy13 Jun 7, 2024

Saranya-jena Jun 7, 2024

andoriyaprashant commented Jun 8, 2024

Saranya-jena commented Aug 1, 2024

Saranya-jena commented Aug 23, 2024

andoriyaprashant commented Aug 23, 2024

Saranya-jena commented Sep 6, 2024

andoriyaprashant commented Sep 7, 2024

S-ayanide commented Nov 22, 2024 •

edited

Loading

Refactor: Eliminate Global Variable to Enhance Testability #4685

Are you sure you want to change the base?

Refactor: Eliminate Global Variable to Enhance Testability #4685

Conversation

andoriyaprashant commented Jun 3, 2024

Fixes #4217

Proposed changes

Types of changes

Checklist

andoriyaprashant commented Jun 3, 2024

namkyu1999 commented Jun 4, 2024

andoriyaprashant commented Jun 4, 2024

namkyu1999 commented Jun 4, 2024

andoriyaprashant commented Jun 4, 2024

Jonsy13 Jun 7, 2024

Choose a reason for hiding this comment

Saranya-jena Jun 7, 2024

Choose a reason for hiding this comment

andoriyaprashant commented Jun 8, 2024

Saranya-jena commented Aug 1, 2024

Saranya-jena commented Aug 23, 2024

andoriyaprashant commented Aug 23, 2024

Saranya-jena commented Sep 6, 2024

andoriyaprashant commented Sep 7, 2024

S-ayanide commented Nov 22, 2024 • edited Loading

S-ayanide commented Nov 22, 2024 •

edited

Loading