Merge branch 'master' into feature/gh-716-cli-success-message

.travis.yml

@@ -23,17 +23,17 @@ install: true
 script:
   # Unit-test and build server
   - cd server/
-  - ./gradlew build --info
+  - ./gradlew --console=plain build --info
   - cd ..
 
   # Unit-test and build integration server
   - cd github-integration/
-  - ./gradlew build --info
+  - ./gradlew --console=plain build --info
   - cd ..
 
   # Launch local zally server
   - cd server/
-  - ./gradlew bootRun > /dev/null &
+  - ./gradlew --console=plain bootRun > /dev/null &
   - echo $! > /tmp/zally_server.pid
   - cd ..
 

cli/zally/domain/violation.go

@@ -6,5 +6,6 @@ type Violation struct {
 	Decription    string   `json:"description"`
 	ViolationType string   `json:"violation_type"`
 	RuleLink      string   `json:"rule_link"`
+	Pointer       string   `json:"pointer"`
 	Paths         []string `json:"paths"`
 }

cli/zally/domain/violations_test.go

@@ -12,6 +12,7 @@ func TestViolations(t *testing.T) {
 	mustViolation.RuleLink = "http://example.com/mustViolation"
 	mustViolation.ViolationType = "MUST"
 	mustViolation.Decription = "Must Description"
+	mustViolation.Pointer = "/pointer/1"
 	mustViolation.Paths = []string{"/path/one", "/path/two"}
 
 	var shouldViolation Violation
@@ -33,7 +34,7 @@ func TestViolations(t *testing.T) {
 	hintViolation.RuleLink = "http://example.com/hintViolation"
 	hintViolation.ViolationType = "HINT"
 	hintViolation.Decription = "Hint Description"
-	hintViolation.Paths = []string{"/path/seven", "/path/eight"}
+	hintViolation.Pointer = "/pointer/2"
 
 	var violationsCount ViolationsCount
 	violationsCount.Must = 1

cli/zally/utils/formatters/markdown_formatter.go

@@ -68,9 +68,15 @@ func (f *MarkdownFormatter) formatViolation(violation *domain.Violation) string
 
 	fmt.Fprintf(&buffer, "### `%s` [%s](%s)\n\n", violation.ViolationType, violation.Title, violation.RuleLink)
 	fmt.Fprintf(&buffer, "> %s\n\n", violation.Decription)
-	for _, path := range violation.Paths {
-		fmt.Fprintf(&buffer, "- %s\n", path)
+
+	if violation.Pointer != "" {
+		fmt.Fprintf(&buffer, "- %s\n", violation.Pointer)
+	} else {
+		for _, path := range violation.Paths {
+			fmt.Fprintf(&buffer, "- %s\n", path)
+		}
 	}
+
 	fmt.Fprintf(&buffer, "\n")
 
 	return buffer.String()

cli/zally/utils/formatters/markdown_formatter_test.go

@@ -126,13 +126,14 @@ func TestMarkdownFormatServerMessage(t *testing.T) {
 func TestMarkdownFormatViolation(t *testing.T) {
 	var markdownFormatter MarkdownFormatter
 
-	t.Run("Converts violation to string in pretty format", func(t *testing.T) {
+	var violation domain.Violation
+	violation.Title = "Test Title"
+	violation.RuleLink = "http://example.com/violation"
+	violation.ViolationType = "MUST"
+	violation.Decription = "Test Description"
+
+	t.Run("Converts violation to string in pretty format with just paths", func(t *testing.T) {
 
-		var violation domain.Violation
-		violation.Title = "Test Title"
-		violation.RuleLink = "http://example.com/violation"
-		violation.ViolationType = "MUST"
-		violation.Decription = "Test Description"
 		violation.Paths = []string{"/path/one", "/path/two"}
 
 		actualResult := markdownFormatter.formatViolation(&violation)
@@ -143,6 +144,31 @@ func TestMarkdownFormatViolation(t *testing.T) {
 
 		tests.AssertEquals(t, expectedResult, actualResult)
 	})
+
+	t.Run("Converts violation to string in pretty format with paths and pointer", func(t *testing.T) {
+
+		violation.Pointer = "/pointer/1"
+
+		actualResult := markdownFormatter.formatViolation(&violation)
+		expectedResult := "### `MUST` [Test Title](http://example.com/violation)\n\n" +
+			"> Test Description\n\n" +
+			"- /pointer/1\n\n"
+
+		tests.AssertEquals(t, expectedResult, actualResult)
+	})
+
+	t.Run("Converts violation to string in pretty format with just pointer", func(t *testing.T) {
+
+		violation.Paths = nil
+
+		actualResult := markdownFormatter.formatViolation(&violation)
+		expectedResult := "### `MUST` [Test Title](http://example.com/violation)\n\n" +
+			"> Test Description\n\n" +
+			"- /pointer/1\n\n"
+
+		tests.AssertEquals(t, expectedResult, actualResult)
+	})
+
 }
 
 func TestMarkdownFormatHeader(t *testing.T) {

cli/zally/utils/formatters/pretty_formatter.go

@@ -90,8 +90,12 @@ func (f *PrettyFormatter) formatViolation(violation *domain.Violation) string {
 	fmt.Fprintf(&buffer, "\t%s\n", violation.Decription)
 	fmt.Fprintf(&buffer, "\t%s\n", violation.RuleLink)
 
-	for _, path := range violation.Paths {
-		fmt.Fprintf(&buffer, "\t\t%s\n", path)
+	if violation.Pointer != "" {
+		fmt.Fprintf(&buffer, "\t\t%s\n", violation.Pointer)
+	} else {
+		for _, path := range violation.Paths {
+			fmt.Fprintf(&buffer, "\t\t%s\n", path)
+		}
 	}
 
 	fmt.Fprintf(&buffer, "\n")

cli/zally/utils/formatters/pretty_formatter_test.go

@@ -112,13 +112,14 @@ func TestPrettyFormatServerMessage(t *testing.T) {
 func TestPrettyFormatViolationInPrettyFormat(t *testing.T) {
 	prettyFormatter := NewPrettyFormatter(NewPrettyColorizer(true))
 
-	t.Run("Converts violation to string in pretty format", func(t *testing.T) {
+	var violation domain.Violation
+	violation.Title = "Test Title"
+	violation.RuleLink = "http://example.com/violation"
+	violation.ViolationType = "MUST"
+	violation.Decription = "Test Description"
+
+	t.Run("Converts violation to string in pretty format with just paths", func(t *testing.T) {
 
-		var violation domain.Violation
-		violation.Title = "Test Title"
-		violation.RuleLink = "http://example.com/violation"
-		violation.ViolationType = "MUST"
-		violation.Decription = "Test Description"
 		violation.Paths = []string{"/path/one", "/path/two"}
 
 		actualResult := prettyFormatter.formatViolation(&violation)
@@ -130,6 +131,32 @@ func TestPrettyFormatViolationInPrettyFormat(t *testing.T) {
 
 		tests.AssertEquals(t, expectedResult, actualResult)
 	})
+
+	t.Run("Converts violation to string in pretty format with paths and pointer", func(t *testing.T) {
+
+		violation.Pointer = "/pointer/1"
+
+		actualResult := prettyFormatter.formatViolation(&violation)
+		expectedResult := "\x1b[31mMUST\x1b[0m \x1b[31mTest Title\x1b[0m\n" +
+			"\tTest Description\n" +
+			"\thttp://example.com/violation\n" +
+			"\t\t/pointer/1\n\n"
+
+		tests.AssertEquals(t, expectedResult, actualResult)
+	})
+
+	t.Run("Converts violation to string in pretty format with just pointer", func(t *testing.T) {
+
+		violation.Paths = nil
+
+		actualResult := prettyFormatter.formatViolation(&violation)
+		expectedResult := "\x1b[31mMUST\x1b[0m \x1b[31mTest Title\x1b[0m\n" +
+			"\tTest Description\n" +
+			"\thttp://example.com/violation\n" +
+			"\t\t/pointer/1\n\n"
+
+		tests.AssertEquals(t, expectedResult, actualResult)
+	})
 }
 
 func TestPrettyFormatHeader(t *testing.T) {

server/build.gradle

@@ -78,6 +78,7 @@ dependencies {
     compile("org.zalando:twintip-spring-web:1.1.0")
     compile("org.jetbrains.kotlin:kotlin-stdlib:$kotlin_version")
     compile("com.typesafe:config:1.3.3")
+    compile("de.mpg.mpi-inf:javatools:1.0.1")
 
     testCompile("net.jadler:jadler-core:1.3.0")
     testCompile("net.jadler:jadler-jdk:1.3.0")

server/src/main/java/de/zalando/zally/apireview/ApiViolationsController.java

@@ -95,7 +95,8 @@ private ViolationDTO toDto(Result violation) {
             violation.getDescription(),
             violation.getViolationType(),
             violation.getRuleSet().url(violation.getRule()).toString(),
-            violation.getPaths()
+            violation.getPaths(),
+            violation.getPointer() == null ? null : violation.getPointer().toString()
         );
     }
 

server/src/main/java/de/zalando/zally/dto/ViolationDTO.kt

@@ -3,7 +3,6 @@ package de.zalando.zally.dto
 import com.fasterxml.jackson.annotation.JsonInclude
 import com.fasterxml.jackson.annotation.JsonInclude.Include
 import de.zalando.zally.rule.api.Severity
-import java.util.Arrays.asList
 
 data class ViolationDTO(
     var title: String? = null,
@@ -12,28 +11,4 @@ data class ViolationDTO(
     var ruleLink: String? = null,
     @JsonInclude(Include.NON_NULL) @Deprecated("Use `pointer` instead.") var paths: List<String>? = null,
     @JsonInclude(Include.NON_NULL) var pointer: String? = null
-) {
-    constructor(
-        title: String?,
-        description: String?,
-        violationType: Severity?,
-        ruleLink: String?,
-        vararg paths: String
-    ) : this(title, description, violationType, ruleLink, asList(*paths))
-
-    @Deprecated("Use `pointer` instead.")
-    constructor(
-        title: String?,
-        description: String?,
-        violationType: Severity?,
-        ruleLink: String?,
-        paths: List<String>?
-    ) : this(
-        title,
-        description,
-        violationType,
-        ruleLink,
-        if (paths?.size == 1) null else paths,
-        if (paths?.size == 1) paths[0] else null
-    )
-}
+)