forked from prospero238/liquibase-runner
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
JENKINS-37699: Allow expressions in configuration fields.
General cleanup of properties assemblage
- Loading branch information
1 parent
3643a97
commit e4bfd70
Showing
9 changed files
with
245 additions
and
56 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
88 changes: 88 additions & 0 deletions
88
src/test/java/org/jenkinsci/plugins/liquibase/common/PropertiesAssemblerTest.java
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,88 @@ | ||
| package org.jenkinsci.plugins.liquibase.common; | ||
|
|
||
| import hudson.EnvVars; | ||
| import hudson.model.AbstractBuild; | ||
| import hudson.model.BuildListener; | ||
| import hudson.model.Descriptor; | ||
| import hudson.tasks.Builder; | ||
| import liquibase.Contexts; | ||
| import liquibase.Liquibase; | ||
| import liquibase.exception.LiquibaseException; | ||
|
|
||
| import java.io.IOException; | ||
| import java.util.Properties; | ||
|
|
||
| import org.jenkinsci.plugins.liquibase.evaluator.AbstractLiquibaseBuilder; | ||
| import org.jenkinsci.plugins.liquibase.evaluator.ExecutedChangesetAction; | ||
| import org.junit.Test; | ||
| import org.junit.runner.RunWith; | ||
| import org.mockito.Mock; | ||
| import org.mockito.runners.MockitoJUnitRunner; | ||
|
|
||
| import static org.hamcrest.Matchers.is; | ||
| import static org.junit.Assert.assertThat; | ||
| import static org.mockito.Mockito.when; | ||
|
|
||
| @RunWith(MockitoJUnitRunner.class) | ||
| public class PropertiesAssemblerTest { | ||
|
|
||
| @Mock | ||
| private AbstractBuild build; | ||
| @Mock | ||
| private BuildListener listener; | ||
| protected Properties properties = new Properties(); | ||
| protected LiquibaseProperty changelogFile = LiquibaseProperty.CHANGELOG_FILE; | ||
| protected EnvVars envVars = new EnvVars(); | ||
|
|
||
|
|
||
| @Test | ||
| public void should_resolve_env_variables() throws IOException, InterruptedException { | ||
| String environmentProperty = "token"; | ||
| String environmentValue = "resolvedValue"; | ||
| envVars.put(environmentProperty, environmentValue); | ||
|
|
||
| when(build.getEnvironment(listener)).thenReturn(envVars); | ||
|
|
||
| String expression = "${" + environmentProperty + "}"; | ||
| BuilderStub liquibaseBuilder = new BuilderStub(); | ||
| liquibaseBuilder.setChangeLogFile(expression); | ||
| PropertiesAssembler.assembleFromProjectConfiguration(liquibaseBuilder, properties, | ||
| build.getEnvironment(listener)); | ||
|
|
||
| assertThat(properties.getProperty(changelogFile.propertyName()), is(environmentValue)); | ||
|
|
||
| } | ||
|
|
||
| @Test | ||
| public void should_not_resolve_expression() { | ||
| Properties properties = new Properties(); | ||
| String value = "${not_in_env}"; | ||
| PropertiesAssembler.addPropertyIfDefined(properties, changelogFile, value, envVars); | ||
|
|
||
| assertThat(properties.getProperty(changelogFile.propertyName()), is(value)); | ||
| } | ||
|
|
||
|
|
||
|
|
||
| private static class BuilderStub extends AbstractLiquibaseBuilder { | ||
| @Override | ||
| public void doPerform(AbstractBuild<?, ?> build, | ||
| BuildListener listener, | ||
| Liquibase liquibase, | ||
| Contexts contexts, | ||
| ExecutedChangesetAction executedChangesetAction, Properties configProperties) | ||
| throws InterruptedException, IOException, LiquibaseException { | ||
|
|
||
| } | ||
|
|
||
| @Override | ||
| public Descriptor<Builder> getDescriptor() { | ||
| return null; | ||
| } | ||
|
|
||
| @Override | ||
| public String getChangeLogFile() { | ||
| return "${token}"; | ||
| } | ||
| } | ||
| } |
Oops, something went wrong.