Improve col tests (#341)

README.md

@@ -28,7 +28,7 @@ always when a new page appear, in this case, a header may have many rows, lines
 * With `go get`:
 
 ```bash
-go get github.com/johnfercher/maroto/v2/[email protected].38
+go get github.com/johnfercher/maroto/v2/[email protected].39
 ```
 
 

docs/README.md

@@ -2,12 +2,12 @@
 
 [![Mentioned in Awesome Go](https://awesome.re/mentioned-badge.svg)](https://github.com/avelino/awesome-go#template-engines) [![Branch](https://img.shields.io/badge/V2-Branch-pink)](https://github.com/johnfercher/maroto/tree/v2)  [![Roadmap](https://img.shields.io/badge/V2-Roadmap-purple)](https://github.com/users/johnfercher/projects/1) [![Discussion](https://img.shields.io/badge/V2-Discussion-blue)](https://github.com/johnfercher/maroto/issues/257) [![Release Notes](https://img.shields.io/badge/Release-Notes-cyan)](https://github.com/johnfercher/maroto/releases) [![Visits Badge](https://badges.pufler.dev/visits/johnfercher/maroto)](https://badges.pufler.dev)
 
-#### Maroto`v2.0.0-alpha.38`is here! Try out:
+#### Maroto`v2.0.0-alpha.39`is here! Try out:
 
 * Installation with`go get`:
 
 ```bash
-go get github.com/johnfercher/maroto/v2/[email protected].38
+go get github.com/johnfercher/maroto/v2/[email protected].39
 ```
 
 The public API was completely redesigned with the aim of enhancing the 

pkg/components/col/col.go

@@ -45,6 +45,13 @@ func (c *col) GetStructure() *node.Node[core.Structure] {
 		Details: c.style.ToMap(),
 	}
 
+	if c.isMax {
+		if len(str.Details) == 0 {
+			str.Details = make(map[string]interface{})
+		}
+		str.Details["is_max"] = true
+	}
+
 	node := node.New(str)
 
 	for _, c := range c.components {

pkg/components/col/col_test.go

@@ -1,23 +1,41 @@
 package col_test
 
 import (
-	"fmt"
 	"testing"
 
+	"github.com/johnfercher/maroto/v2/internal/fixture"
+	"github.com/johnfercher/maroto/v2/mocks"
+	"github.com/johnfercher/maroto/v2/pkg/components/code"
 	"github.com/johnfercher/maroto/v2/pkg/core/entity"
+	"github.com/johnfercher/maroto/v2/pkg/props"
+	"github.com/johnfercher/maroto/v2/pkg/test"
+	"github.com/stretchr/testify/assert"
 
 	"github.com/johnfercher/maroto/v2/pkg/components/col"
-
-	"github.com/stretchr/testify/assert"
 )
 
 func TestNew(t *testing.T) {
-	// Act
-	c := col.New()
+	t.Run("when size is not defined, should use is as max", func(t *testing.T) {
+		// Act
+		c := col.New()
+
+		// Assert
+		test.New(t).Assert(c.GetStructure()).Equals("components/cols/new_zero_size.json")
+	})
+	t.Run("when size is defined, should use not use max", func(t *testing.T) {
+		// Act
+		c := col.New(12)
+
+		// Assert
+		test.New(t).Assert(c.GetStructure()).Equals("components/cols/new_defined_size.json")
+	})
+	t.Run("when has component, should retrieve components", func(t *testing.T) {
+		// Act
+		c := col.New(12).Add(code.NewQr("code"))
 
-	// Assert
-	assert.NotNil(t, c)
-	assert.Equal(t, "*col.col", fmt.Sprintf("%T", c))
+		// Assert
+		test.New(t).Assert(c.GetStructure()).Equals("components/cols/new_with_components.json")
+	})
 }
 
 func TestCol_GetSize(t *testing.T) {
@@ -43,3 +61,31 @@ func TestCol_GetSize(t *testing.T) {
 		assert.Equal(t, 14, size)
 	})
 }
+
+func TestCol_Render(t *testing.T) {
+	t.Run("should call provider correctly", func(t *testing.T) {
+		// Arrange
+		cfg := &entity.Config{}
+		cell := fixture.CellEntity()
+		style := &props.Cell{}
+
+		provider := &mocks.Provider{}
+		provider.EXPECT().CreateCol(cell.Width, cell.Height, cfg, style)
+
+		component := &mocks.Component{}
+		component.EXPECT().Render(provider, &cell)
+		component.EXPECT().SetConfig(cfg)
+
+		sut := col.New(12).Add(component)
+		sut.WithStyle(style)
+		sut.SetConfig(cfg)
+
+		// Act
+		sut.Render(provider, cell, true)
+
+		// Assert
+		provider.AssertNumberOfCalls(t, "CreateCol", 1)
+		component.AssertNumberOfCalls(t, "Render", 1)
+		component.AssertNumberOfCalls(t, "SetConfig", 1)
+	})
+}

test/maroto/components/codes/new_bar_row_custom_prop.json

@@ -5,6 +5,9 @@
 		{
 			"value": 0,
 			"type": "col",
+			"details": {
+				"is_max": true
+			},
 			"nodes": [
 				{
 					"value": "code",

test/maroto/components/codes/new_bar_row_default_prop.json

@@ -5,6 +5,9 @@
 		{
 			"value": 0,
 			"type": "col",
+			"details": {
+				"is_max": true
+			},
 			"nodes": [
 				{
 					"value": "code",

test/maroto/components/codes/new_matrix_row_custom_prop.json

@@ -5,6 +5,9 @@
 		{
 			"value": 0,
 			"type": "col",
+			"details": {
+				"is_max": true
+			},
 			"nodes": [
 				{
 					"value": "code",

test/maroto/components/codes/new_matrix_row_default_prop.json

@@ -5,6 +5,9 @@
 		{
 			"value": 0,
 			"type": "col",
+			"details": {
+				"is_max": true
+			},
 			"nodes": [
 				{
 					"value": "code",

test/maroto/components/codes/new_qr_row_custom_prop.json

@@ -5,6 +5,9 @@
 		{
 			"value": 0,
 			"type": "col",
+			"details": {
+				"is_max": true
+			},
 			"nodes": [
 				{
 					"value": "code",

test/maroto/components/codes/new_qr_row_default_prop.json

@@ -5,6 +5,9 @@
 		{
 			"value": 0,
 			"type": "col",
+			"details": {
+				"is_max": true
+			},
 			"nodes": [
 				{
 					"value": "code",

test/maroto/components/cols/new_defined_size.json

@@ -0,0 +1,4 @@
+{
+	"value": 12,
+	"type": "col"
+}

test/maroto/components/cols/new_with_components.json

@@ -0,0 +1,13 @@
+{
+	"value": 12,
+	"type": "col",
+	"nodes": [
+		{
+			"value": "code",
+			"type": "qrcode",
+			"details": {
+				"prop_percent": 100
+			}
+		}
+	]
+}

test/maroto/components/cols/new_zero_size.json

@@ -0,0 +1,7 @@
+{
+	"value": 0,
+	"type": "col",
+	"details": {
+		"is_max": true
+	}
+}

test/maroto/components/images/new_image_from_bytes_row_custom_prop.json

@@ -5,6 +5,9 @@
 		{
 			"value": 0,
 			"type": "col",
+			"details": {
+				"is_max": true
+			},
 			"nodes": [
 				{
 					"value": "AQID",

test/maroto/components/images/new_image_from_bytes_row_default_prop.json

@@ -5,6 +5,9 @@
 		{
 			"value": 0,
 			"type": "col",
+			"details": {
+				"is_max": true
+			},
 			"nodes": [
 				{
 					"value": "AQID",

test/maroto/components/images/new_image_from_file_row_custom_prop.json

@@ -5,6 +5,9 @@
 		{
 			"value": 0,
 			"type": "col",
+			"details": {
+				"is_max": true
+			},
 			"nodes": [
 				{
 					"value": "path",

test/maroto/components/images/new_image_from_file_row_default_prop.json

@@ -5,6 +5,9 @@
 		{
 			"value": 0,
 			"type": "col",
+			"details": {
+				"is_max": true
+			},
 			"nodes": [
 				{
 					"value": "path",

test/maroto/components/lines/new_line_row_custom_prop.json

@@ -5,6 +5,9 @@
 		{
 			"value": 0,
 			"type": "col",
+			"details": {
+				"is_max": true
+			},
 			"nodes": [
 				{
 					"type": "lineStyle",

test/maroto/components/lines/new_line_row_default_prop.json

@@ -5,6 +5,9 @@
 		{
 			"value": 0,
 			"type": "col",
+			"details": {
+				"is_max": true
+			},
 			"nodes": [
 				{
 					"type": "lineStyle",

test/maroto/components/signatures/new_signature_row_custom_prop.json

@@ -5,6 +5,9 @@
 		{
 			"value": 0,
 			"type": "col",
+			"details": {
+				"is_max": true
+			},
 			"nodes": [
 				{
 					"value": "signature",

test/maroto/components/signatures/new_signature_row_default_prop.json

@@ -5,6 +5,9 @@
 		{
 			"value": 0,
 			"type": "col",
+			"details": {
+				"is_max": true
+			},
 			"nodes": [
 				{
 					"value": "signature",

test/maroto/components/texts/new_text_row_custom_prop.json

@@ -5,6 +5,9 @@
 		{
 			"value": 0,
 			"type": "col",
+			"details": {
+				"is_max": true
+			},
 			"nodes": [
 				{
 					"value": "code",

test/maroto/components/texts/new_text_row_default_prop.json

@@ -5,6 +5,9 @@
 		{
 			"value": 0,
 			"type": "col",
+			"details": {
+				"is_max": true
+			},
 			"nodes": [
 				{
 					"value": "code",