Shardingsphere: Use YAML file instead of inline string in test cases

[terrymanu]

Some cases include YAML inline string for testing orchestration persistent which are hard to read.
It is better to put the YAML inline string into YAML file.

Some test cases which need to be refactored:

• [x] ConfigCenterTest
• [x] SchemaChangedListenerTest
• [x] OrchestrationSpringBootRegistryShardingTest
• [x] OrchestrationSpringBootRegistryEncryptTest
• [ ] OrchestrationSpringBootRegistryMasterSlaveTest
• [x] HttpServerHandlerTest
• [ ] Still to be found...

[sukumaar]

Can I pick this issue? This is my first issue so is it ok if I work on ConfigCenterTest file first and raise PR ?

[menghaoranss]

@terrymanu We are optimizing the unit test coverage of the orchestration module now，I will optimize the above test classes at the same time.

[sukumaar]

@menghaoranss hey if you are busy with something and need some hand then please let me work on this , I also want to contribute to open source community :-)

[yu199195]

@sukumaar good ，you can pick a task to respond to.

[binxia5kpch]

what can i do for this case?

[binxia5kpch]

can i try the test ConfigCenterTest ?

[menghaoranss]

can i try the test ConfigCenterTest ?

of course，looking forward to your pull request.

[sukumaar]

can i try the test ConfigCenterTest ?

hey is it possible for you to pick other test ? , because I have kinda started working on it.

[hitnewbee]

May I try SchemaChangedListenerTest this one？

[binxia5kpch]

can i try the test ConfigCenterTest ?

hey is it possible for you to pick other test ? , because I have kinda started working on it.

yeah i can pick other @sukumaar

[terrymanu]

Please ref #5629, use

@SneakyThrows
    private String readYAML(final String yamlFile) {
        return Files.readAllLines(Paths.get(ClassLoader.getSystemResource(yamlFile).toURI())).stream().map(each -> each + System.lineSeparator()).collect(Collectors.joining());
    }

to load file content

[tomakemyself]

what can i do for this question?

[zengzhihong]

can i try the test:OrchestrationSpringBootRegistryEncryptTest

[terrymanu]

@tomakemyself @zengzhihong, welcome

[big-mountain-z]

Can I try the test: OrchestrationSpringBootRegistryMasterSlaveTest