mirror of
https://github.com/hanzoai/o11y-foundry.git
synced 2026-10-30 18:26:45 +00:00
PandeyandGitHub
f6b600e4a0
## Summary
- Split material domain into writer-facing `Material` and
traversal-capable `StructuredMaterial` interfaces with concrete
`YAMLMaterial`, `JSONMaterial`, `INIMaterial`, and `BlobMaterial` types.
- Inline INI and YAML format helpers into their respective material
types and remove `format.go` and `ini.go`.
- Rename interface methods for clarity: `Contents` → `JSONContents`,
`WithContents` → `CloneWithJSONContents`, `IsMultiDoc` →
`HasMultipleDocuments`.
- Switch material constructors to `internal/errors` with
`TypeInvalidInput` for parse failures, `TypeInternal` for canonical-JSON
marshaling failures, and `TypeNotFound` for missing traversal paths.
- Add `MustNew{JSON,YAML,INI}Material` constructors and split tests into
per-type `*_material_test.go` files.
- Tighten interface doc comments to follow godoc convention (lead with
identifier, drop pure restatement, document the gjson dotted-key path
syntax).
- Move `ini.PrettyFormat = false` into a package `init()` so the global
is touched once instead of on every `FmtContents` call.
91 lines
2.5 KiB
Go
91 lines
2.5 KiB
Go
package domain
|
|
|
|
import (
|
|
"testing"
|
|
|
|
"github.com/stretchr/testify/assert"
|
|
)
|
|
|
|
func TestNewYAMLMaterial(t *testing.T) {
|
|
tests := []struct {
|
|
name string
|
|
contents []byte
|
|
path string
|
|
pass bool
|
|
expectedJSON []byte
|
|
expectedFmtContents []byte
|
|
expectedHasMultipleDocuments bool
|
|
}{
|
|
{
|
|
name: "SingleDocument_Valid",
|
|
contents: []byte("service:\n names:\n - query-service\n - frontend\n"),
|
|
path: "service.yaml",
|
|
pass: true,
|
|
expectedJSON: []byte(`{"service":{"names":["query-service","frontend"]}}`),
|
|
expectedFmtContents: []byte("service:\n names:\n - query-service\n - frontend\n"),
|
|
expectedHasMultipleDocuments: false,
|
|
},
|
|
{
|
|
name: "MultiDocument_Valid",
|
|
contents: []byte("---\nname: one\n---\nname: two\n"),
|
|
path: "service.yaml",
|
|
pass: true,
|
|
expectedJSON: []byte(`[{"name":"one"},{"name":"two"}]`),
|
|
expectedFmtContents: []byte("name: one\n---\nname: two\n"),
|
|
expectedHasMultipleDocuments: true,
|
|
},
|
|
{
|
|
name: "Invalid",
|
|
contents: []byte("service: [unterminated\n"),
|
|
path: "service.yaml",
|
|
pass: false,
|
|
},
|
|
}
|
|
|
|
for _, tt := range tests {
|
|
t.Run(tt.name, func(t *testing.T) {
|
|
material, err := NewYAMLMaterial(tt.contents, tt.path)
|
|
if !tt.pass {
|
|
assert.Error(t, err)
|
|
return
|
|
}
|
|
|
|
assert.NoError(t, err)
|
|
assert.Equal(t, tt.path, material.Path())
|
|
assert.Equal(t, tt.expectedHasMultipleDocuments, material.HasMultipleDocuments())
|
|
assert.JSONEq(t, string(tt.expectedJSON), string(material.JSONContents()))
|
|
assert.Equal(t, tt.expectedFmtContents, material.FmtContents())
|
|
})
|
|
}
|
|
}
|
|
|
|
func TestYAMLMaterialGetBytes(t *testing.T) {
|
|
tests := []struct {
|
|
name string
|
|
material YAMLMaterial
|
|
path string
|
|
pass bool
|
|
expected []byte
|
|
}{
|
|
{
|
|
name: "SingleDocument_Exists",
|
|
material: MustNewYAMLMaterial([]byte("service:\n name: query-service\n"), "service.yaml"),
|
|
path: "service.name",
|
|
pass: true,
|
|
expected: []byte("query-service"),
|
|
},
|
|
}
|
|
|
|
for _, tt := range tests {
|
|
t.Run(tt.name, func(t *testing.T) {
|
|
output, err := tt.material.GetBytes(tt.path)
|
|
if !tt.pass {
|
|
assert.Error(t, err)
|
|
return
|
|
}
|
|
|
|
assert.Equal(t, tt.expected, output)
|
|
})
|
|
}
|
|
}
|