cloud-barista / cb-ladybug

Cloud-Barista Multi-Cloud Application Runtime Framework : Support Multi-Cloud Kubernetes Service
Apache License 2.0
11 stars 11 forks source link

Move common const var in test scripts to one file #72

Closed jihoon-seo closed 3 years ago

jihoon-seo commented 3 years ago

What would you like to be enhanced : cb-ladybug/docs/test 아래에 있는 스크립트들에

# const
c_URL_LADYBUG="http://localhost:8080/ladybug"
c_CT="Content-Type: application/json"

등의 라인이 반복적으로 존재합니다.

각 테스트 스크립트 파일에 존재하는 것 보다는 하나의 파일에 모아 놓는 것이 좋을 것 같습니다.

Why is this needed :

Proposed solution : PR #48 에서 docs/test/conf.env 이 추가될 예정이므로 위의 라인들을 docs/test/conf.env 파일로 옮기고 각 테스트 스크립트 파일에서는 source ./conf.env 하여 해당 변수 read

sykim-etri commented 3 years ago

새롭게 conf.env가 추가될 예정이니 그것도 좋은 방법 같습니다. #48 이 통합되면 추진하도록 하겠습니다. 감사합니다.