Skip to content
This repository was archived by the owner on Jul 8, 2023. It is now read-only.

Adds test for licensing_config#57

Merged
clintoncwolfe merged 3 commits into
inspec-6from
vasundhara/unit-test-for-license-config
Jul 7, 2023
Merged

Adds test for licensing_config#57
clintoncwolfe merged 3 commits into
inspec-6from
vasundhara/unit-test-for-license-config

Conversation

@Vasu1105

@Vasu1105 Vasu1105 commented Jun 14, 2023

Copy link
Copy Markdown
Contributor

Description

Adds a simple test for the static values encoded in the licensing config block.

Related Issue

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New content (non-breaking change)
  • Breaking change (a content change which would break existing functionality or processes)

Checklist:

  • I have read the CONTRIBUTING document.

@Vasu1105 Vasu1105 requested a review from a team as a code owner June 14, 2023 10:26
@Vasu1105 Vasu1105 force-pushed the vasundhara/unit-test-for-license-config branch 4 times, most recently from 5f88bac to 3b81286 Compare June 14, 2023 12:45
Comment thread test/unit/utils/license_config_test.rb Outdated
@Vasu1105 Vasu1105 force-pushed the vasundhara/unit-test-for-license-config branch from 3b81286 to d3a3e31 Compare June 15, 2023 14:24
Vasu1105 and others added 2 commits July 6, 2023 19:35
Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io>

Fix lint

Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io>

Read the value of license server url from env for test

Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io>
Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com>
@clintoncwolfe clintoncwolfe force-pushed the vasundhara/unit-test-for-license-config branch from d3a3e31 to 44bfcfa Compare July 6, 2023 23:40
Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com>
@clintoncwolfe clintoncwolfe merged commit dfb0172 into inspec-6 Jul 7, 2023
@clintoncwolfe clintoncwolfe deleted the vasundhara/unit-test-for-license-config branch July 7, 2023 01:24
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.

3 participants