这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@ashleyfrieze
Copy link
Contributor

No description provided.

@ashleyfrieze ashleyfrieze force-pushed the BAEL-5025-java-convert-camel-case-to-words branch 2 times, most recently from 180c53b to 574d5a7 Compare June 26, 2021 16:40
Copy link
Collaborator

@KevinGilmore KevinGilmore left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A couple of lines indented too much

@Test
void givenStopWords_thenTitleHasThemInLowerCase() {
assertThat(capitalizeMyTitle(Arrays.asList("this", "is", "A", "title", "with", "a", "stop", "word", "or", "two")))
.isEqualTo("This Is a Title With a Stop Word or Two");
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

make indentation like the above methods

@Test
void givenStopWordIsFirstWord_thenTitleHasItCapitalized() {
assertThat(capitalizeMyTitle(Arrays.asList("a", "stop", "word", "first")))
.isEqualTo("A Stop Word First");
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

indentation

@ashleyfrieze ashleyfrieze force-pushed the BAEL-5025-java-convert-camel-case-to-words branch from 574d5a7 to 20d7ec4 Compare July 3, 2021 13:33
@eric-martin eric-martin merged commit 0c1b0bf into eugenp:master Jul 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants