/*
 * Licensed to the Apache Software Foundation (ASF) under one or more contributor license
 * agreements. See the NOTICE file distributed with this work for additional information regarding
 * copyright ownership. The ASF licenses this file to You under the Apache License, Version 2.0 (the
 * "License"); you may not use this file except in compliance with the License. You may obtain a
 * copy of the License at
 *
 * http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software distributed under the License
 * is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express
 * or implied. See the License for the specific language governing permissions and limitations under
 * the License.
 */
package org.apache.geode.management.internal.cli.result;

import static org.assertj.core.api.Assertions.assertThat;

import java.nio.file.Path;
import java.nio.file.Paths;

import org.junit.Test;
import org.junit.experimental.categories.Category;

import org.apache.geode.test.junit.categories.UnitTest;

@Category(UnitTest.class)
public class CommandResultTest {

  @Test
  public void emptyResultHasOneEmptyLine() {
    CommandResult commandResult = new CommandResult(new InfoResultData());

    assertThat(commandResult.nextLine()).isEqualTo("");
    assertThat(commandResult.hasNextLine()).isFalse();
  }

  @Test
  public void resultWithOneLineHasOneLine() {
    CommandResult commandResult = new CommandResult(new InfoResultData("oneLine"));

    assertThat(commandResult.nextLine()).isEqualTo("oneLine\n");
    assertThat(commandResult.hasNextLine()).isFalse();
  }

  @Test
  public void resultWithTwoLinesHasTwoLines() {
    InfoResultData resultData = new InfoResultData();
    resultData.addLine("lineOne");
    resultData.addLine("lineTwo");
    CommandResult commandResult = new CommandResult(resultData);

    assertThat(commandResult.nextLine()).isEqualTo("lineOne\nlineTwo\n");
    assertThat(commandResult.hasNextLine()).isFalse();
  }

  @Test
  public void emptyResultDoesNotHaveFileToDownload() {
    CommandResult commandResult = new CommandResult(new InfoResultData());

    assertThat(commandResult.hasFileToDownload()).isFalse();
  }

  @Test
  public void resultWithFileDoesHaveFileToDownload() {
    Path fileToDownload = Paths.get(".").toAbsolutePath();
    CommandResult commandResult = new CommandResult(fileToDownload);

    assertThat(commandResult.hasFileToDownload()).isTrue();
    assertThat(commandResult.nextLine()).isEqualTo(fileToDownload.toString() + '\n');
    assertThat(commandResult.getFileToDownload()).isEqualTo(fileToDownload);
  }


}

@@ -837,24 +837,90 @@
 li.result;%0A%0A
+import static org.apache.commons.lang.SystemUtils.LINE_SEPARATOR;%0A
 import stati
@@ -965,16 +965,16 @@
 rtThat;%0A
-
 %0Aimport 
@@ -1657,19 +1657,34 @@
 %22oneLine
-%5Cn%22
+%22 + LINE_SEPARATOR
 );%0A    a
@@ -2016,32 +2016,41 @@
 sult.nextLine())
+%0A        
 .isEqualTo(%22line
@@ -2056,20 +2056,55 @@
 eOne
-%5CnlineTwo%5Cn%22
+%22 + LINE_SEPARATOR + %22lineTwo%22 + LINE_SEPARATOR
 );%0A 
@@ -2702,12 +2702,22 @@
 ) + 
-'%5Cn'
+LINE_SEPARATOR
 );%0A 

Use system independent expected strings