Add not equals validation

parser/ast.go

@@ -175,6 +175,9 @@ func (n *NodeEqual) LeftValue() NodeExpr {
 
 // NOT EQUAL
 func (n *NodeNotEqual) Assertion(lvalue string, rvalue string) bool {
+	if len(lvalue) == 40 {
+		return lvalue[0:len(rvalue)] != rvalue
+	}
 	return lvalue != rvalue
 }
 

runtime/commits_test.go

@@ -1,6 +1,7 @@
 package runtime
 
 import (
+	"fmt"
 	"path/filepath"
 	"testing"
 
@@ -82,3 +83,14 @@ func TestSelectedFieldsCount(t *testing.T) {
 		t.Errorf("Selected 'author' and 'hash'. Got %v", table.fields)
 	}
 }
+
+func TestNotEqualsInWhere(t *testing.T) {
+	queryData := "select committer, hash from commits limit 1"
+	table := getTableForQuery(queryData, "../", t)
+	firstCommitter := table.rows[0]["committer"].(string)
+	query := fmt.Sprintf("select committer, hash from commits where committer != '%s' limit 1", firstCommitter)
+	table = getTableForQuery(query, "../", t)
+	if firstCommitter == table.rows[0]["committer"].(string) {
+		t.Errorf("Strill got the same committer as the first one. - %s", firstCommitter)
+	}
+}

runtime/visitor.go

@@ -91,7 +91,9 @@ func (v *RuntimeVisitor) VisitExpr(n parser.NodeExpr) error {
 	case reflect.TypeOf(new(parser.NodeIn)):
 		g := n.(*parser.NodeIn)
 		return v.VisitIn(g)
-
+	case reflect.TypeOf(new(parser.NodeNotEqual)):
+		g := n.(*parser.NodeNotEqual)
+		return v.VisitNotEqual(g)
 	}
 
 	return nil
@@ -101,7 +103,13 @@ func (v *RuntimeVisitor) VisitEqual(n *parser.NodeEqual) error {
 	lvalue := n.LeftValue().(*parser.NodeId).Value()
 	rvalue := n.RightValue().(*parser.NodeLiteral).Value()
 	boolRegister = n.Assertion(metadata(lvalue), rvalue)
+	return nil
+}
 
+func (v *RuntimeVisitor) VisitNotEqual(n *parser.NodeNotEqual) error {
+	lvalue := n.LeftValue().(*parser.NodeId).Value()
+	rvalue := n.RightValue().(*parser.NodeLiteral).Value()
+	boolRegister = n.Assertion(metadata(lvalue), rvalue)
 	return nil
 }