🐛 compute checksum for properties of queries

We had missed to process checksums for properties of queries with this call. Since semantically it is supposed to refresh the entire checksum of the Mquery object, it needs to cover properties as well.

Signed-off-by: Dominik Richter <[email protected]>

examples/complex.mql.yaml

@@ -52,7 +52,7 @@ packs:
           # This MQL uses the property defined above. You can override it via
           # e.g. --props "home='/home/user'"
           mql: |
-            file( props.home ) { * }
+            file( props.home ) { basename user group }
 
 # These are shared queries that can be used in any querypack
 queries:

explorer/mquery.go

@@ -118,6 +118,9 @@ func (m *Mquery) RefreshChecksum(
 
 	for i := range m.Props {
 		prop := m.Props[i]
+		if _, err := prop.RefreshChecksumAndType(schema); err != nil {
+			return err
+		}
 		if prop.Checksum == "" {
 			return errors.New("referenced property '" + prop.Mrn + "' checksum is empty")
 		}

explorer/mquery_test.go

@@ -4,13 +4,33 @@
 package explorer
 
 import (
+	"context"
 	"encoding/json"
 	"testing"
 
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/require"
+	"go.mondoo.com/cnquery/providers-sdk/v1/testutils"
 )
 
+func TestMquery_RefreshChecksum(t *testing.T) {
+	a := &Mquery{
+		Mql:   "mondoo.version != props.world",
+		Props: []*Property{{Mql: "'hi'", Uid: "world"}},
+	}
+	x := testutils.LinuxMock()
+	err := a.RefreshChecksum(
+		context.Background(),
+		x.Schema(),
+		func(ctx context.Context, mrn string) (*Mquery, error) {
+			return nil, nil
+		},
+	)
+	require.NoError(t, err)
+	assert.Equal(t, "t2slvX2k58s=", a.Checksum)
+	assert.Equal(t, "0xgs2tmERsM=", a.Props[0].Checksum)
+}
+
 func TestMqueryMerge(t *testing.T) {
 	a := &Mquery{
 		Mql:   "base",