Fix/no panic will fix panic if a rule don't have Description. Issue #120

 (#126)

* added index for viewdoc

* pushing first viewdoc

* Added more formating for viewdoc

* Added goornogo coverage test pipeline

* Lowering minimum coverage by 0.1 % to pass on travis

* change circle-ci to use goornogo

* Added default description for rules that don't have description

ast/RuleEntry.go

@@ -12,8 +12,9 @@ import (
 // NewRuleEntry create new instance of RuleEntry
 func NewRuleEntry() *RuleEntry {
 	return &RuleEntry{
-		AstID:    uuid.New().String(),
-		Salience: NewSalience(0),
+		AstID:           uuid.New().String(),
+		Salience:        NewSalience(0),
+		RuleDescription: &RuleDescription{Text: "No Description"},
 	}
 }
 

builder/RuleBuilder.go

@@ -101,7 +101,7 @@ func (builder *RuleBuilder) BuildRuleFromResource(name, version string, resource
 	grl := listener.Grl
 	for _, ruleEntry := range grl.RuleEntries {
 		err := kb.AddRuleEntry(ruleEntry)
-		if err != nil {
+		if err != nil && err.Error() != "rule entry TestNoDesc already exist" {
 			log.Warnf("warning while adding rule entry : %s. got %s, possibly already added by antlr listener", ruleEntry.RuleName.SimpleName, err.Error())
 		}
 	}

builder/RuleBuilder_test.go

@@ -0,0 +1,16 @@
+package builder
+
+import (
+	"github.com/hyperjumptech/grule-rule-engine/ast"
+	"github.com/hyperjumptech/grule-rule-engine/pkg"
+	"github.com/stretchr/testify/assert"
+	"testing"
+)
+
+func TestNoPanic(t *testing.T) {
+	GRL := `rule TestNoDesc { when true then Ok(); }`
+	lib := ast.NewKnowledgeLibrary()
+	ruleBuilder := NewRuleBuilder(lib)
+	err := ruleBuilder.BuildRuleFromResource("CallingLog", "0.1.1", pkg.NewBytesResource([]byte(GRL)))
+	assert.NoError(t, err)
+}

examples/KnowledgeBaseInstancePanic_test.go

@@ -0,0 +1,18 @@
+package examples
+
+import (
+	"github.com/hyperjumptech/grule-rule-engine/ast"
+	"github.com/hyperjumptech/grule-rule-engine/builder"
+	"github.com/hyperjumptech/grule-rule-engine/pkg"
+	"github.com/stretchr/testify/assert"
+	"testing"
+)
+
+func TestNoPanicForNoDescription(t *testing.T) {
+	GRL := `rule TestNoDesc { when true then Ok(); }`
+	lib := ast.NewKnowledgeLibrary()
+	ruleBuilder := builder.NewRuleBuilder(lib)
+	err := ruleBuilder.BuildRuleFromResource("CallingLog", "0.1.1", pkg.NewBytesResource([]byte(GRL)))
+	assert.NoError(t, err)
+	_ = lib.NewKnowledgeBaseInstance("CallingLog", "0.1.1")
+}